Re: [FFmpeg-devel] [PATCH 2/2] avformat/mov_esds: check return value of ff_mp4_read_dec_config_descr

2017-11-25 Thread James Almer 
On 11/25/2017 11:09 PM, Michael Niedermayer wrote:
> On Sat, Nov 25, 2017 at 03:54:57PM -0300, James Almer wrote:
>> Signed-off-by: James Almer 
>> ---
>>  libavformat/mov_esds.c | 7 ---
>>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> should be ok
> 
> thx

Pushed, thanks.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mov_esds: check return value of ff_mp4_read_dec_config_descr

2017-11-25 Thread Michael Niedermayer 
On Sat, Nov 25, 2017 at 03:54:57PM -0300, James Almer wrote:
> Signed-off-by: James Almer 
> ---
>  libavformat/mov_esds.c | 7 ---
>  1 file changed, 4 insertions(+), 3 deletions(-)

should be ok

thx

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

"I am not trying to be anyone's saviour, I'm trying to think about the
 future and not be sad" - Elon Musk



signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 2/2] avformat/mov_esds: check return value of ff_mp4_read_dec_config_descr

2017-11-25 Thread James Almer 
Signed-off-by: James Almer 
---
 libavformat/mov_esds.c | 7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/libavformat/mov_esds.c b/libavformat/mov_esds.c
index 2ecf03742f..a444d969c6 100644
--- a/libavformat/mov_esds.c
+++ b/libavformat/mov_esds.c
@@ -23,7 +23,7 @@
 int ff_mov_read_esds(AVFormatContext *fc, AVIOContext *pb)
 {
 AVStream *st;
-int tag;
+int tag, ret = 0;
 
 if (fc->nb_streams < 1)
 return 0;
@@ -38,6 +38,7 @@ int ff_mov_read_esds(AVFormatContext *fc, AVIOContext *pb)
 
 ff_mp4_read_descr(fc, pb, &tag);
 if (tag == MP4DecConfigDescrTag)
-ff_mp4_read_dec_config_descr(fc, st, pb);
-return 0;
+ret = ff_mp4_read_dec_config_descr(fc, st, pb);
+
+return ret;
 }
-- 
2.15.0

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel