[xiph-commits] r13164 - trunk/Tremor

[xiphmont at svn.xiph.org]

Author: xiphmont
Date: 2007-06-21 16:07:33 -0700 (Thu, 21 Jun 2007)
New Revision: 13164

Modified:
   trunk/Tremor/Makefile.am
   trunk/Tremor/framing.c
Log:
Eliminate memory leak bugs in Tremor mailine (closes bug 1127)
Eliminate electric fence from debug Makefile; valgrind supercedes.



Modified: trunk/Tremor/Makefile.am
===================================================================
--- trunk/Tremor/Makefile.am	2007-06-21 14:57:34 UTC (rev 13163)
+++ trunk/Tremor/Makefile.am	2007-06-21 23:07:33 UTC (rev 13164)
@@ -32,7 +32,7 @@
 	$(MAKE) ivorbisfile_example
 
 debug:
-	$(MAKE) all CFLAGS="@DEBUG@" LDFLAGS="-lefence"	
+	$(MAKE) all CFLAGS="@DEBUG@" 
 
 profile:
 	$(MAKE) all CFLAGS="@PROFILE@"

Modified: trunk/Tremor/framing.c
===================================================================
--- trunk/Tremor/framing.c	2007-06-21 14:57:34 UTC (rev 13163)
+++ trunk/Tremor/framing.c	2007-06-21 23:07:33 UTC (rev 13164)
@@ -53,11 +53,6 @@
     bt=bs->unused_buffers;
     rt=bs->unused_references;
 
-    if(!bs->outstanding){
-      _ogg_free(bs);
-      return;
-    }
-
     while(bt){
       ogg_buffer *b=bt;
       bt=b->ptr.next;
@@ -71,6 +66,10 @@
       _ogg_free(r);
     }
     bs->unused_references=0;
+
+    if(!bs->outstanding)
+      _ogg_free(bs);
+
   }
 }
 
@@ -834,6 +833,7 @@
     ogg_buffer_release(os->header_tail);
     ogg_buffer_release(os->body_tail);
     memset(os,0,sizeof(*os));    
+    _ogg_free(os);
   }
   return OGG_SUCCESS;
 }