Fixed dereference of free()'d memory.

Thanks to Evgeny Podjachev for the fix!
icculus · Mar 13, 2012 · a42ada1 · a42ada1
1 parent c57bd94
commit a42ada1
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/src/archiver_dir.c b/src/archiver_dir.c
@@ -64,20 +64,21 @@ static PHYSFS_Io *doOpen(dvoid *opaque, const char *name,
     if (fileExists == NULL)
         fileExists = &existtmp;
 
-    *fileExists = 0;
-
     BAIL_IF_MACRO(f == NULL, NULL, NULL);
 
     io = __PHYSFS_createNativeIo(f, mode);
-    allocator.Free(f);
     if (io == NULL)
     {
         PHYSFS_Stat statbuf;  /* !!! FIXME: this changes the error message. */
         __PHYSFS_platformStat(f, fileExists, &statbuf);
-        return NULL;
     } /* if */
+    else
+    {
+        *fileExists = 1;
+    } /* else */
+
+    allocator.Free(f);
 
-    *fileExists = 1;
     return io;
 } /* doOpen */