int serial_read(int port){
     int buffer_size = 4       // Größe des Eingabepuffers
    char buffer[buffer_size];  // Eingabepuffer für die komplette Eingabe 
    int anz;         // gelesene Zeichen
    char c;          // Eingabepuffer für das aktuelle Byte 
    int  i;          // Zeichenposition bzw Index
    i = 0;
    do{              // Lesen bis zur Puffergröße (im Moment 4 Byte)
        anz = read(port, (void*)&c, 1);
        if (anz > 0){
            if (c != '\r'){
                buffer[i++] = c;      
            }
        }
    }while (c != '\r' && i < buffer_size && anz >= 0);       
   
    if (anz < 0) {
        perror("Lesen fehlgeschlagen");
    }
    else if (i == 0) {
        perror("Keine Daten empfangen");
    }
    else {
        buffer[i] = '\0';     
        // printf("%i Bytes: %s", i, buffer);    // zur Fehlersuche
    }
    // Rückgabewert 
    int ergebnis;
    if (strstr(buffer,"_OK_")){
        ergebnis = 1;
    }
    else{
        ergebnis = 0;
    }
return ergebnis; 
}

Meine C Kenntnisse sind noch nicht so gut ;). Ich habe es erst einmal so gelöst, dass ich einfach den eingehenden String einfach auf eine Zeichenkette überprüfe und dann etntweder eine 1 oder eine 0 zurück gebe.

Das ganze Programm soll zur Steuerung eines µC mit dem PC sein.

Ich hoffe das mir jemand weiterhelfen kann.

Schönen Karfreitag noch
Gruß Andreas

4 ist zu wenig um als C-String „OK“ und die '\0' aufzunehmen.
Das buffer[i] = 0; überschreibt dann andere Daten.

Buffer musst du im aufrufenden Kontext definieren, innerhalb von LowLevel Funktionen macht man keine Ausgaben auf stdout/stderr sondern ebenfalls allenfalls im aufrufenden Kontext.

int serial_read (int port, char *b, int buffermax)
{
  int anz;         // gelesene Zeichen
  int i = 0;       // Zeichenposition bzw Index

  do {				// Lesen bis zur Puffergroesse
      anz = read (port, b, 1);
      if (anz > 0)
	    if (*b != '\r')
	        b++,i++;
     }
  while (*b != '\r' && i < buffermax-1 && anz >= 0);
  *b = 0;

  if (i == 0)
    {
      perror ("Keine Daten empfangen");
    }

  // Rueckgabewert 
  int ergebnis;
  if (strstr (b, "_OK_"))
    {
      ergebnis = 1;
    }
  else
    {
      ergebnis = 0;
    }
  return ergebnis;
}

int main ()
{
  char buffer[BUFSIZ]={0};
  int r = serial_read (4711, buffer, BUFSIZ);
  printf ("%d\n%s", r, buffer);
  return 0;
}

Nein, versuche globale Variablen so weit wie möglich zu vermeiden. Es reicht aus, den Puffer in der aufrufenden Funktion zur Verfügung zu stellen. Genau so, wie wob das in seinem Beispiel schon gemacht hat.

Gruß Andreas