[FFmpeg-devel] [libav-devel] [PATCH 3/4] dds: make sure pallete frame buffer exists before use
Andreas Cadhalpun
andreas.cadhalpun at gmail.com
Fri Nov 13 22:01:01 CET 2015
On 13.11.2015 02:08, Vittorio Giovara wrote:
> oh I see, that can happen for a special crafted file, DDPF_FOURCC has
> been introduced recently while DDPF_PALETTE has been removed, so a
> normal file should not have both set.
OK, that makes sense.
> Because of that, and how rare palette dds are, I think it's quite safe
> to unset ctx->paletted if ctx->compressed is set, and be done with it.
Patch doing that is attached.
> That is the only case I can see that happen, right?
Yes, that's the only case.
> Thanks for the catch, how did you find it btw? :)
Like all the other: with afl [1].
It's really good at finding weird cases. ;)
Best regards,
Andreas
1: http://lcamtuf.coredump.cx/afl/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-dds-disable-palette-flag-for-compressed-dds.patch
Type: text/x-diff
Size: 1106 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151113/e729b0db/attachment.patch>
More information about the ffmpeg-devel
mailing list