Browse Source

Fixed a memory leak (thanks, Zack!).

We should probably rework this piece of code a little more after 2.0.4 ships,
though.

Fixes Bugzilla #3004.
Ryan C. Gordon 10 years ago
parent
commit
699f879a8f
1 changed files with 2 additions and 0 deletions
  1. 2 0
      src/core/linux/SDL_ibus.c

+ 2 - 0
src/core/linux/SDL_ibus.c

@@ -461,10 +461,12 @@ SDL_IBus_Init(void)
             return SDL_FALSE;
         }
         
+        /* !!! FIXME: if ibus_addr_file != NULL, this will overwrite it and leak (twice!) */
         ibus_addr_file = SDL_strdup(addr_file);
         
         addr = IBus_ReadAddressFromFile(addr_file);
         if (!addr) {
+            SDL_free(addr_file);
             return SDL_FALSE;
         }