summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorTony Theodore <tonyt@logyst.com>2013-02-09 07:26:37 (GMT)
committerTony Theodore <tonyt@logyst.com>2013-02-09 07:26:37 (GMT)
commitf7133d3145115d7d4fed45ab895d96c1e4ac51ad (patch)
treed94968de1102e1eb24ea7c5e53f5ccde4e5fb970 /src
parent36555723f57064fb8be79aefe43e8de024d6b58f (diff)
downloadmxe-f7133d3145115d7d4fed45ab895d96c1e4ac51ad.zip
mxe-f7133d3145115d7d4fed45ab895d96c1e4ac51ad.tar.gz
mxe-f7133d3145115d7d4fed45ab895d96c1e4ac51ad.tar.bz2
package sdl_mixer: fix for double-freeing resources with WAV files
Diffstat (limited to 'src')
-rw-r--r--src/sdl_mixer-1-fix-double-free.patch35
1 files changed, 35 insertions, 0 deletions
diff --git a/src/sdl_mixer-1-fix-double-free.patch b/src/sdl_mixer-1-fix-double-free.patch
new file mode 100644
index 0000000..a987ec1
--- /dev/null
+++ b/src/sdl_mixer-1-fix-double-free.patch
@@ -0,0 +1,35 @@
+This file is part of MXE.
+See index.html for further information.
+
+https://bugs.gentoo.org/show_bug.cgi?id=406739
+Nikos Chantziaras 2012-03-03 03:13:39 EST
+
+There's a bug in the current version of SDL_mixer where failing to load a WAV sample will result in freeing a resource twice, resulting in an application crash. There is an upstream fix for this:
+
+http://hg.libsdl.org/SDL_mixer/rev/ea53a85853dd
+
+Upstream bug: http://bugzilla.libsdl.org/show_bug.cgi?id=1418
+
+--- a/mixer.c
++++ b/mixer.c
+@@ -610,13 +610,15 @@
+ break;
+ default:
+ SDL_SetError("Unrecognized sound file type");
+- return(0);
++ if ( freesrc ) {
++ SDL_RWclose(src);
++ }
++ loaded = NULL;
++ break;
+ }
+ if ( !loaded ) {
++ /* The individual loaders have closed src if needed */
+ SDL_free(chunk);
+- if ( freesrc ) {
+- SDL_RWclose(src);
+- }
+ return(NULL);
+ }
+
+