X-Git-Url: https://pd.if.org/git/?a=blobdiff_plain;f=functions%2Fstdio%2Ffopen.c;h=14ce3a8f57b2fe8911baba2c941f0529c76050a0;hb=d9e48b611b63bcfc55727463cb9d9d0e87a7a405;hp=f22287da4669bf8b15441b8076089103aea2a5ad;hpb=b08f4b52b1cd1f7a9553c0f357a7c90859fa3e73;p=pdclib

diff --git a/functions/stdio/fopen.c b/functions/stdio/fopen.c
index f22287d..14ce3a8 100644
--- a/functions/stdio/fopen.c
+++ b/functions/stdio/fopen.c
@@ -12,59 +12,34 @@
 #ifndef REGTEST
 #include <_PDCLIB_glue.h>
 
-static const FILE * _PDCLIB_filelist = NULL;
+/* FIXME: This approach is a possible attack vector. */
+struct _PDCLIB_file_t * _PDCLIB_filelist = NULL;
 
-static int filemode( char const * const mode )
+struct _PDCLIB_file_t * fopen( const char * _PDCLIB_restrict filename, const char * _PDCLIB_restrict mode )
 {
-    int rc = 0;
-    switch ( mode[0] )
-    {
-        case 'r':
-            rc |= _PDCLIB_FREAD;
-            break;
-        case 'w':
-            rc |= _PDCLIB_FWRITE;
-            break;
-        case 'a':
-            rc |= _PDCLIB_FAPPEND;
-            break;
-        default:
-            return -1;
-    }
-    for ( size_t i = 1; i < 4; ++i )
+    struct _PDCLIB_file_t * rc;
+    if ( mode == NULL || filename == NULL || filename[0] == '\0' )
     {
-        switch ( mode[1] )
-        {
-            case '+':
-                if ( rc & _PDCLIB_FRW ) return -1;
-                rc |= _PDCLIB_FRW;
-                break;
-            case 'b':
-                if ( rc & _PDCLIB_FBIN ) return -1;
-                rc |= _PDCLIB_FBIN;
-                break;
-            case '\0':
-                return rc;
-            default:
-                return -1;
-        }
+        /* Mode or filename invalid */
+        return NULL;
     }
-    return -1;
-}
-
-FILE * fopen( const char * _PDCLIB_restrict filename, const char * _PDCLIB_restrict mode )
-{
-    FILE * rc;
-    if ( mode == NULL || filename == NULL || filename[0] == '\0' )
+    if ( ( rc = calloc( 1, sizeof( struct _PDCLIB_file_t ) ) ) == NULL )
     {
+        /* no memory for another FILE */
         return NULL;
     }
-    if ( ( rc = calloc( 1, sizeof( FILE ) ) ) == NULL ) return rc; /* no space for another FILE */
-    if ( ( rc->status = filemode( mode ) ) == -1 ) goto fail; /* invalid mode given */
-    if ( ( rc->handle = _PDCLIB_open( filename, rc->status ) ) == -1 ) goto fail; /* OS "open" failed */
+    if ( ( rc->status = _PDCLIB_filemode( mode ) ) == 0 ) goto fail; /* invalid mode */
+    rc->handle = _PDCLIB_open( filename, rc->status );
+    if ( rc->handle == _PDCLIB_NOHANDLE ) goto fail; /* OS open() failed */
+    /* Adding to list of open files */
     rc->next = _PDCLIB_filelist;
     _PDCLIB_filelist = rc;
-    /* TODO: Continue here: Set up PDCLib FILE contents */
+    /* Setting buffer, and mark as internal. TODO: Check for unbuffered */
+    if ( ( rc->buffer = malloc( BUFSIZ ) ) == NULL ) goto fail;
+    rc->bufsize = BUFSIZ;
+    rc->bufidx = 0;
+    rc->status |= ( _PDCLIB_LIBBUFFER | _PDCLIB_VIRGINSTR );
+    /* TODO: Setting mbstate */
     return rc;
 fail:
     free( rc );
@@ -78,7 +53,15 @@ fail:
 
 int main( void )
 {
-    TESTCASE( NO_TESTDRIVER );
+    TESTCASE( fopen( NULL, NULL ) == NULL );
+    TESTCASE( fopen( NULL, "w" ) == NULL );
+    TESTCASE( fopen( "", NULL ) == NULL );
+    TESTCASE( fopen( "", "w" ) == NULL );
+    TESTCASE( fopen( "foo", "" ) == NULL );
+    TESTCASE( fopen( "testfile", "wq" ) == NULL ); /* Illegal mode */
+    TESTCASE( fopen( "testfile", "wr" ) == NULL ); /* Illegal mode */
+    TESTCASE( fopen( "testfile", "w" ) != NULL );
+    system( "rm testfile" );
     return TEST_RESULTS;
 }