Re: [Lost] [Patch] Bugfix für stat und fstat

[Antoine Kaufmann <toni@xxxxxxxxxxxxxxxx>]

Am Montag, 10. März 2008 22.09:40 schrieb Kevin Wolf:
> Am Montag, 10. März 2008 21:31:20 schrieb Antoine Kaufmann:
> > ! libc: Bugfix in stat beim aufrufen mit einem Verzeichnis als Pfad
> > (wurde bis jetzt nicht richtig erkannt)
> > + libc: Implementation fuer fstat
>
> Gibt es einen Grund, warum du jetzt opendir statt dem nativen (und bisher
> benutzten) directory_open benutzt?

Nein, eigentlich nicht...

> Außerdem denke ich, daß der Zusammenhang zwischen fstat und stat genau
> falschrum implementiert ist. Wenn eine Datei schon geöffnet ist, sollte man
> nicht den Pfad holen und die Datei ein zweites Mal öffnen, sondern diesen
> Dateideskriptor eben benutzen. stat kann dann die Datei öffnen und fstat
> aufrufen.
Jo das denke ich auch. Da sollte ich LostIO mal mit einem RPC nachrüsten und 
da darf ich wieder alle Treiber nachbessern, deswegen habe ich das noch 
gelassen. Aber irgendwo habe ich eh noch eine Ladung LostIO-Patches am 
rumliegen...

=== src/modules/lib/posix/stat.c
==================================================================
--- src/modules/lib/posix/stat.c	(revision 1409)
+++ src/modules/lib/posix/stat.c	(local)
@@ -36,6 +36,7 @@
 #include <sys/stat.h>
 #include <stdio.h>
 #include <dir.h>
+#include <errno.h>
 
 /**
  * Modus einer Datei Aendern. Der Modus entscheidet unter anderem ueber
@@ -83,32 +84,31 @@
 
     FILE* file = fopen(filename, "r");
     
-    // Wenn die Datei nicht geoeffnet werden kann wird sofort abgebrochen
+    // Wenn die Datei nicht geoeffnet werden kann, koennte es sich um ein
+    // Verzeichnis handeln, falls nicht wird sofort abgebrochen
     if (file == NULL) {
-        return -1;
-    }
+        io_resource_t* dir = directory_open(filename);
+        if (dir == NULL) {
+            errno = ENOENT;
+            return -1;
+        }
+        directory_close(dir);
 
-    // TODO im Moment wird nur die Groesse eingetragen
-
-    fseek(file, 0, SEEK_END);
-    stat_buf->st_size = ftell(file);
-
-    stat_buf->st_blksize = 512;
-    stat_buf->st_blocks = stat_buf->st_size / stat_buf->st_blksize;
-    if (stat_buf->st_size % stat_buf->st_blksize != 0) {
-        stat_buf->st_blocks++;
-    }
-    
-    // Versuchen den Pfad als verzeichnis zu oeffen. Falls dies klappt wird der
-    // entsprehende Modus gesetzt
-    io_resource_t* dir = directory_open(filename);
-    if (dir != NULL) {
-        directory_close(dir);
         stat_buf->st_mode |= S_IFDIR;
     } else {
         stat_buf->st_mode |= S_IFREG;
+
+        // TODO im Moment wird nur die Groesse eingetragen
+
+        fseek(file, 0, SEEK_END);
+        stat_buf->st_size = ftell(file);
+
+        stat_buf->st_blksize = 512;
+        stat_buf->st_blocks = stat_buf->st_size / stat_buf->st_blksize;
+        if (stat_buf->st_size % stat_buf->st_blksize != 0) {
+            stat_buf->st_blocks++;
+        }
     }
-
     stat_buf->st_uid = 0;
     stat_buf->st_gid = 0;
 
@@ -141,8 +141,12 @@
  */
 int fstat(int file, struct stat* stat_buf)
 {
-    // TODO
-    return -1;
+    FILE* f = fdopen(file, "r");
+    if (f == NULL) {
+        errno = EBADF;
+        return -1;
+    }
+    return stat(f->path, stat_buf);
 }
 
 /**