# HG changeset patch
# User Sam Lantinga <slouken@libsdl.org>
# Date 1184083550 0
# Node ID d083a2e4433d858ea89f834da342ec62aa220e14
# Parent  5f96cf74b78250372a910f191763ba546a2e5780
Merged memory leak fix from SDL 1.2

diff -r 5f96cf74b782 -r d083a2e4433d src/file/SDL_rwops.c
--- a/src/file/SDL_rwops.c	Tue Jul 10 15:51:17 2007 +0000
+++ b/src/file/SDL_rwops.c	Tue Jul 10 16:05:50 2007 +0000
@@ -57,18 +57,7 @@
     int a_mode;
 
     if (!context)
-        return -1;
-
-    context->hidden.win32io.h = INVALID_HANDLE_VALUE;   /* mark this as unusable */
-
-    context->hidden.win32io.buffer.data =
-        (char *) SDL_malloc(READAHEAD_BUFFER_SIZE);
-    if (!context->hidden.win32io.buffer.data) {
-        SDL_OutOfMemory();
-        return -1;
-    }
-    context->hidden.win32io.buffer.size = 0;
-    context->hidden.win32io.buffer.left = 0;
+        return -1;              /* failed (invalid call) */
 
     /* "r" = reading, file must exist */
     /* "w" = writing, truncate existing, file may not exist */
@@ -126,6 +115,16 @@
     context->hidden.win32io.h = h;
     context->hidden.win32io.append = a_mode ? SDL_TRUE : SDL_FALSE;
 
+    context->hidden.win32io.buffer.data =
+        (char *) SDL_malloc(READAHEAD_BUFFER_SIZE);
+    if (!context->hidden.win32io.buffer.data) {
+        SDL_OutOfMemory();
+        CloseHandle(context->hidden.win32io.h);
+        return -1;
+    }
+    context->hidden.win32io.buffer.size = 0;
+    context->hidden.win32io.buffer.left = 0;
+
     return 0;                   /* ok */
 }
 static long SDLCALL
@@ -428,8 +427,7 @@
     rwops = SDL_AllocRW();
     if (!rwops)
         return NULL;            /* SDL_SetError already setup by SDL_AllocRW() */
-    rwops->hidden.win32io.h = INVALID_HANDLE_VALUE;
-    if (win32_file_open(rwops, file, mode)) {
+    if (win32_file_open(rwops, file, mode) < 0) {
         SDL_FreeRW(rwops);
         return NULL;
     }