[FFmpeg-devel] [PATCH] jpeg2000dec: make cblk->length(inc)/data use consistent
Nicolas BERTRAND
nicoinattendu at gmail.com
Mon Jun 17 17:21:40 CEST 2013
My previous comment is wrong.
I forgot the case of several quality layers in image so the cblk length
can change in case of a cblk dispatched in several layers.
the image have 4 quality layers
Is the +2 necessary in :
if ( bytestream2_get_bytes_left(&s->g) < cblk->lengthinc
+ || sizeof(cblk->data) < cblk->length + cblk->lengthinc + 2
)
To avoid a en empty read in the following bytestream2_get_bufferu?
So, its LGTM
More information about the ffmpeg-devel
mailing list