[FFmpeg-devel] [PATCH] avpacket: fix size check in packet_alloc

Michael Niedermayer michael at niedermayer.cc
Tue Jan 5 21:39:47 CET 2016


On Tue, Jan 05, 2016 at 01:05:50PM +0100, Andreas Cadhalpun wrote:
> The previous check only caught sizes from -AV_INPUT_BUFFER_PADDING_SIZE
> to -1.
> 
> This fixes ubsan runtime error: signed integer overflow: 2147483647 + 32
> cannot be represented in type 'int'
> 
> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
> ---
>  libavcodec/avpacket.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

LGTM

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160105/f67224b7/attachment.sig>


More information about the ffmpeg-devel mailing list