GetLine Funktion

[Klaus (Gast)]

Hallo,

ich habe eine Funktion geschrieben um eine Zeile einer Datei auszulesen.
Manchmal kommt es zu Abstürzen, und ich finde den Fehler nicht.
Kann mal jemand über diese Funktion schauen ob das so passt?

S8 GetLine(int fd, char *buf, U8 n)
{
  U8 charsRead=0;
  U8 readChar;
  
  if(n==0) return (-1);
  if (fd < 0) return (-1);

  nav_select( fd );

  if ( file_eof() )  return (-1);

  while(charsRead<n-1)
  {
    readChar=file_getc();
    if(readChar == FS_EOF) return (-1);//break;
    if((readChar == '\n')||(readChar == '\r')) {if(charsRead>0) break;}
    else
    {
      *buf++ = readChar;
      charsRead++;
    }

  }

  *buf = '\0';

  return charsRead;
}

n = maximal 100

[Peter II (Gast)]

wieviel platz ist ein buf?

[gaast (Gast)]

>n = maximal 100
Bau da mal einen Test ein oder nimm asser.h zum Testen.

[gaast (Gast)]

asserT.h

[Karl H. (kbuchegg)]

Jetzt 3 mal in Gedanken durchgegangen. IMHO ist diese Funktion ok. 
Fehler beim aufrufenden Code?

[DirkB (Gast)]

Wie ist denn FS_EOF definiert?

[Rolf M. (rmagnus)]

DirkB schrieb:
> Wie ist denn FS_EOF definiert?

Der Typ von readChar läßt vermuten, daß es sich um einen 8-Bit-Typ 
handelt. Wie wird denn sichergestellt, daß FS_EOF keinem gültigen Wert 
entspricht?