[FFmpeg-devel] [PATCH] Fix a valgrind error in h264.c

[Vitor Sessak]

Hi,

I was wondering why MinGW fails decoding CVMAPAQP3_Sony_E.jsv in FATE 
(http://fate.multimedia.cx/index.php?build_record=61129), so I tested it 
with valgrind in my box:

> ==10039== Conditional jump or move depends on uninitialised value(s)
> ==10039==    at 0x8146DD5: ff_print_debug_info (mpegvideo.c:1092)
> ==10039== 
> ==10039== Conditional jump or move depends on uninitialised value(s)
> ==10039==    at 0x82A3971: h264_parse (h264_parser.c:55)
> ==10039== 
> ==10039== Invalid read of size 1
> ==10039==    at 0x828BFBE: filter_mb (h264.c:6471)
> ==10039==  Address 0x49374a3 is not stack'd, malloc'd or (recently) free'd

The attached patch fixes the last one, but I have no familiarity with 
h264 code (nor $bignum hours of free time to acquire it) so I don't know 
if it is the right way to fix it...

-Vitor

PS: I don't know either if this actually fixes MinGW or not...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: h264_valg.diff
Type: text/x-diff
Size: 650 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090303/e485fb86/attachment.diff>