On 30 March 2015 at 02:48, Michael Niedermayer <michae...@gmx.at> wrote:
> On Sun, Mar 29, 2015 at 11:05:41AM +0000, Donny Yang wrote: > > Signed-off-by: Donny Yang <w...@kota.moe> > > --- > > libavcodec/pngenc.c | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c > > index 3697dbb..bd3aae5 100644 > > --- a/libavcodec/pngenc.c > > +++ b/libavcodec/pngenc.c > > @@ -373,8 +373,6 @@ static int encode_frame(AVCodecContext *avctx, > AVPacket *pkt, > > enc_row_size + > > 12 * (((int64_t)enc_row_size + IOBUF_SIZE - 1) / > IOBUF_SIZE) // 12 * ceil(enc_row_size / IOBUF_SIZE) > > ); > > - if (max_packet_size > INT_MAX) > > - return AVERROR(ENOMEM); > > the check is neccessary to prevent potential integer overflows > Doesn't ffmpeg support memory allocations of greater than 4 GiB? I thought it did because the memory allocation functions either accept an int64_t or size_t... _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
