[tremor] memory leakage in lowmem-branch

[Roland Wintersteller]

Hi together, 

I recently found out, that not all memory allocated in framing.c (lowmem-branch) is freed again. I debugged the code and made the following changes:

function _ogg_buffer_destroy (framing.c line 58):
deleted the return statement. Otherwise memory for a number of ogg_buffer and ogg_reference is not freed.

function ogg_stream_destroy (framing.c line 827):
Similar to oy in the function ogg_sync_destroy at line 568, also in ogg_stream_destroy the memory for ogg_stream has to be freed. I introduced the statement _ogg_free(os);

After this changes the memory leakage problem was fixed. At the end of the decoding process, all memory which was allocated first is freed again. The decoder still works fine. 

Regards, Roland

<p>--- >8 ----
List archives:  http://www.xiph.org/archives/
Ogg project homepage: http://www.xiph.org/ogg/
To unsubscribe from this list, send a message to 'tremor-request at xiph.org'
containing only the word 'unsubscribe' in the body.  No subject is needed.
Unsubscribe messages sent to the list will be ignored/filtered.