Compilerfehler oder Programmfehler AVR

[Markus (Gast)]

Hallo,
ich hab was sehr seltsames bei meinem ATmega88PA bemerkt.
Zum Hintergrund:
es ist ein Programm zum steuern eines GSM Modules, dass im Fehlerfall, 
sprich Heizung ist kaputt, eine SMS an max. 4 Nummern versendet.
Die erste SMS wird an Nummer1 ohne Probleme versendet.
Bei der zweiten SMS wird die Funktion sms_schreiben mit i=2 aufgerufen.
Nicht wie erwartet geht es nach Ende der Funktion zurück sondern beginnt 
wieder am Anfang des GESAMTEN Programmes. Zudem ist mir aufgefallen bei 
der 2ten SMS wird auch buff_schreibe_c Fehlerhaft bearbeitet.
Wie kann ich das Beheben oder den Fehler mehr eingrenzen. Ich habe einen 
ICE3 Debugger.


Program Memory Usage:  4738 bytes   57,8 % Full
Data Memory Usage:  756 bytes   73,8 % Full



hier wird Fall 1 aufgerufen

void sms_schreiben_handler(uint8_t fall)
{
  //Fall 1 = Heizung Fehler
  //Fall 2 = Bestätigung nach Nummer geändert
  
  uint8_t i =0;
  char buff_c [100];
  uint8_t save = 0;
  
  if (fall==1)
        {  
    
    for (i=1;i<=(eeprom_nummern.anzahl);i++)
    {
      save++;
      strcpy(sms.text,"Die Heizung meldet einen Fehler");
      sms_schreiben(i);
      if (save==4)
      {
        return;
      };
    };
    
    return;
  };
  


  if (fall==2)
  {
    
    strcpy( buff_c, "Es wurden folgende Nummern eingetragen:" );
    
    switch(eeprom_nummern.anzahl)
    {
      case 1:
      strcat( buff_c, eeprom_nummern.nummer1);
      break;
      
      case 2:
      strcat( buff_c, eeprom_nummern.nummer1);
      strcat( buff_c,";");
      strcat( buff_c, eeprom_nummern.nummer2);
      break;
      
      
      case 3:
      strcat( buff_c, eeprom_nummern.nummer1);
      strcat( buff_c,";");
      strcat( buff_c, eeprom_nummern.nummer2);
      strcat( buff_c,";");
      strcat( buff_c, eeprom_nummern.nummer3);
      break;
      
      case 4:
      strcat( buff_c, eeprom_nummern.nummer1);
      strcat( buff_c,";");
      strcat( buff_c, eeprom_nummern.nummer2);
      strcat( buff_c,";");
      strcat( buff_c, eeprom_nummern.nummer3);
      strcat( buff_c,";");
      strcat( buff_c, eeprom_nummern.nummer4);
      break;
      
      
    };  
    
    strcpy(sms.text,buff_c);
    
    sms_schreiben(1); //Null ist Nummer 1
    
    
    
  };

};

void sms_schreiben(uint8_t nummer)
{
  char buff_schreibe_c[20];
  
  
  
  
  EIMSK=0x00;
  
  cli();
  sim_guthaben();
  wait_sekunden(2);
  
  uart_puts("AT+CMGF=1"); // Text mode
  _delay_ms(100);
  

  
  
strcpy(buff_schreibe_c,"AT+CMGS="); // Ziel Nummer
  
  
  switch(nummer)
  {
    case 1:    
    strcat(buff_schreibe_c,eeprom_nummern.nummer1);
    break;
    
    case 2:
    strcat(buff_schreibe_c,eeprom_nummern.nummer2);
    break;
    
    case 3:
    strcat(buff_schreibe_c,eeprom_nummern.nummer3);
    break;
    
    case 4:
    strcat(buff_schreibe_c,eeprom_nummern.nummer4);
    break;
    
    
    
  };
  
  uart_puts(buff_schreibe_c);
  _delay_ms(100);

  uart_puts(sms.text);
  uart_puts(sms.guthaben);
// zu testzwecken auskommentiert  uart_putc(0x1A);
  
  wait_sekunden(5);
  
  
  

  return;
};

[Michael R. (Firma: Brainit GmbH) (fisa)]

Du kannst mit an Sicherheit grenzender Wahrscheinlichkeit davon ausgehen 
dass es sich nicht um einen Compilerfehler handelt.

Markus schrieb:
> Bei der zweiten SMS wird die Funktion sms_schreiben mit i=2 aufgerufen.
> Nicht wie erwartet geht es nach Ende der Funktion zurück sondern beginnt
> wieder am Anfang des GESAMTEN Programmes.

Reset aufgrund nicht existenten Interrupt-Handlers oder aufgrund eines 
Stack overflows, oder watchdog, oder...

Markus schrieb:
> cli();
>   sim_guthaben();
>   wait_sekunden(2);

das cli() ist zumindest verdächtig

Ich tippe aber auf stack overflow.

[sei (Gast)]

kein sei()

Aber auch: Mehrere Sekunden mit deaktivierten Interrupts zu blockieren 
riecht nach falschem Design.

[Markus (Gast)]

Ja, ich hoffe es les liegt an mir dann kann ich es beheben.
Wie soll ich das am besten mit Sei()/cli() lösen?
Ist es besser die Interrupts direkt zu deaktivieren?

[Markus (Gast)]

Ich brauch in der Funktion sim_guthaben() Timer1 um ein Timeout zu 
erzeugen.

[Rolf M. (rmagnus)]

Markus schrieb:
> Ja, ich hoffe es les liegt an mir dann kann ich es beheben.
> Wie soll ich das am besten mit Sei()/cli() lösen?

Die Interrupts immer nur so kurz wie irgend möglich sperren. cli() und 
sei() immer paarweise benutzen. Wo ist das sei() zu dem cli() in 
sms_schreiben?

> Ist es besser die Interrupts direkt zu deaktivieren?

Schwer zu sagen, da unbekannt ist, welche Interrupts es bei dir 
überhaupt gibt, was sie machen und wie sie mit dem restlichen Programm 
zusammenarbeiten.

Ansonsten: Einen Stackoverflow kann ich mir auch vorstellen, da dein 
Speicher ja mit statischen Variablen schon recht voll ist und die 
gezeigten Funktionen auch nicht sehr sparsam damit umgehen.

[g457 (Gast)]

Ein Stacküberlauf ist sogar sehr wahrscheinlich. Aber wie man schon an 
dem kurzen Codeauszug sehen kann, gibt es massenhaft Speicherbereiche, 
die man problemlos in den Flash verschieben kann.

[Marco S (Gast)]

strncpy/strncat können helfen, Fehler zu vermeiden.

[Markus (Gast)]

Cool vielen Dank hab mal wieder dazu gelernt :D
Also ich hab jetzt mal cli(); gelöscht dann gings.

[Marco S (Gast)]

Schau dir den code ruhig weiter an. Meine Handynummer lässt dein 
buff_schreibe_c überlaufen. Und mit so einem noblen Debugger solltest du 
auch mal spielen.