Re: [FFmpeg-devel] [PATCH 01/11] libavformat/mxfdec.c: klv_read_packet: Properly check klv_decode_ber_length return value.

2015-10-24 Thread Tomas Härdin
On Thu, 2015-10-22 at 19:25 +0200, Alexis Ballier wrote: > On Wed, 21 Oct 2015 23:31:29 +0200 > Tomas Härdin wrote: > > > On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > > > klv_decode_ber_length cannot return -1, but can return > > > AVERROR_INVALIDDATA.

Re: [FFmpeg-devel] [PATCH 01/11] libavformat/mxfdec.c: klv_read_packet: Properly check klv_decode_ber_length return value.

2015-10-22 Thread Alexis Ballier
On Wed, 21 Oct 2015 23:31:29 +0200 Tomas Härdin wrote: > On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > > klv_decode_ber_length cannot return -1, but can return > > AVERROR_INVALIDDATA. Store its return value in a signed integer > > (instead of unsigned

[FFmpeg-devel] [PATCH 01/11] libavformat/mxfdec.c: klv_read_packet: Properly check klv_decode_ber_length return value.

2015-10-21 Thread Alexis Ballier
klv_decode_ber_length cannot return -1, but can return AVERROR_INVALIDDATA. Store its return value in a signed integer (instead of unsigned KLVPacket.length) and forward the error appropriately. --- libavformat/mxfdec.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH 01/11] libavformat/mxfdec.c: klv_read_packet: Properly check klv_decode_ber_length return value.

2015-10-21 Thread Tomas Härdin
On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > klv_decode_ber_length cannot return -1, but can return AVERROR_INVALIDDATA. > Store its return value in a signed integer (instead of unsigned > KLVPacket.length) and forward the error appropriately. > --- > libavformat/mxfdec.c | 6