UART Fehlfunktion

[Jonas E. (jonas_e43)]

Hallo zusammen,

Mit folgenden Funktionen sende ich Strings über UART. Nach dem ~50-150 
Mal hängt sich der gesamte Controller auf, bis weitere Daten kommen. 
Könnte dies an den Funktionen liegen?

Empfangen:

uint8_t UART_getc(void)
{
  while (!(UCSRA & (1<<RXC)))   // warten bis Zeichen verfuegbar
  ;
  return UDR;                   // Zeichen aus UDR an Aufrufer zurueckgeben
}

uint8_t UART_gets(char* buffer, uint8_t maxLength)
{
  uint8_t index = 0;
  
  while ( ( buffer[index] = UART_getc() ) != '\0' && index < maxLength )
  {
    index++;
  }
  buffer[index] = '\0';
  
  // bytes read
  return index;
}

Senden:

void UART_putchar(char z)          // Zeichen in UART Ausgang geben
{
  while ( !(UCSRA & (1<<UDRE)) )        // Warte bis UART Ausgang frei
      ;
  UDR = z;
}
  
void UART_puts(char *s)            // Zeichenkette (String) in UART Ausgang geben
{
  while ( *s )
  {
    UART_putchar(*s);
    s++;
  }
  UART_putchar('\0');
}

Vielen Dank im Vorraus

Gruß Jonas

[Karlheinz (Gast)]

Hallo,

mit
> while ( ( buffer[index] = UART_getc() ) != '\0' && index < maxLength )
>   {
>     index++;
>   }

kann index den Wert von maxLength erreichen und das abschliessende

>   buffer[index] = '\0';

schreibt dann ein Byte über Buffer ... hinaus ins RAM

[Jonas E. (jonas_e43)]

Danke für die Antwort. Habe das jetzt mal so übernommen:

uint8_t UART_gets(char* buffer, uint8_t maxLength)
{
  uint8_t index = 0;
  
  while ( index < maxLength && ( buffer[index] = UART_getc() ) != '\0' )
  {
    index++;
  }
  if ( index < maxLength )
  buffer[index] = '\0';
  else
  buffer[maxLength - 1] = '\0';
  
  // bytes read
  return index;
}

Leider hilft das nichts.
Vielleicht fällt euch ja noch was ein :)
Gruß Jonas