[FFmpeg-devel] [PATCH] Fix unaligned dsputil call in pngdec.c

Vitor Sessak vitor1001
Sat May 23 18:01:41 CEST 2009


Reimar D?ffinger wrote:
> On Sat, May 23, 2009 at 05:46:07PM +0200, Reimar D?ffinger wrote:
>> On Sat, May 23, 2009 at 05:32:10PM +0200, Vitor Sessak wrote:
>>> @@ -612,7 +613,7 @@
>>>      ret = s->bytestream - s->bytestream_start;
>>>   the_end:
>>>      inflateEnd(&s->zstream);
>>> -    av_freep(&s->crow_buf);
>>> +    av_freep(&s->crow_buf - 15);
>> still won't do what you want at all, I recommend testing with valgrind.
>> Might be best to decrement crwo_buf first.

Indeed, 1000l.

> Of course after checking that it is !NULL first. I think that's going to
> become a mess, maybe split the variable, making crow_buf_base the
> malloced one, i.e.

Good idea, it is more readable like this.

-Vitor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: png4.diff
Type: text/x-diff
Size: 1218 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090523/fffa50e7/attachment.diff>



More information about the ffmpeg-devel mailing list