[FFmpeg-devel] [PATCH] gifdec: drop pointless casting

Paul B Mahol onemda at gmail.com
Thu Dec 20 20:13:36 CET 2012


On 12/20/12, Piotr Bandurski <ami_stuff at o2.pl> wrote:
> Hi,
>
>> - if( (unsigned)s->screen_width > 32767
>> - || (unsigned)s->screen_height > 32767){
>> + if (s->screen_width > 32767 ||
>> + s->screen_height > 32767) {
>
> Would removing this check be enough to support gifs up to 65535x65535?

There are such files?

screen_width/height are ints and they are used to store 16 bit number so they
will always be >= 0.


More information about the ffmpeg-devel mailing list