[Ffmpeg-devel] [PATCH] Fix for memleak in dv.c

plaum at ipf.uni-stuttgart.de plaum
Fri Aug 26 00:58:19 CEST 2005


Another one:

This fixes a memleak (ca. 200 kB) in dv.c by allocating dv_vlc_map
via av_mallocz_static().

I also found something scary in dv.c:

/* MultiThreading */
static uint8_t** dv_anchor;

/* .. and a bit later in dvvideo_init()... */

/* dv_anchor lets each thread know its Id */
dv_anchor = av_malloc(12*27*sizeof(void*));

Shouldn't dv_anchor be moved to DVVideoContext?
I doubt that multiple instances of dv decoders would work with the
current code.

Cheers

Burkhard

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg_dv_memleak_fix.patch
Type: text/x-patch
Size: 1113 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20050826/b7e213dc/attachment.bin>



More information about the ffmpeg-devel mailing list