Re: [libav-devel] [PATCH 1/3] rtpdec_qt: Add an accidentally removed allocation return value check

2013-09-27 Thread Anton Khirnov

On Thu, 26 Sep 2013 17:13:38 +0200, Luca Barbato lu_z...@gentoo.org wrote:
 On 26/09/13 15:43, Martin Storsjö wrote:
  This check was mistakenly removed in 5626f994f.
  ---
   libavformat/rtpdec_qt.c |2 ++
   1 file changed, 2 insertions(+)
  
 
 The set looks fine, thanks for fixing all of them.
 

+1

-- 
Anton Khirnov
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 1/3] rtpdec_qt: Add an accidentally removed allocation return value check

2013-09-26 Thread Martin Storsjö
This check was mistakenly removed in 5626f994f.
---
 libavformat/rtpdec_qt.c |2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavformat/rtpdec_qt.c b/libavformat/rtpdec_qt.c
index 146002a..8898f05 100644
--- a/libavformat/rtpdec_qt.c
+++ b/libavformat/rtpdec_qt.c
@@ -180,6 +180,8 @@ static int qt_rtp_parse_packet(AVFormatContext *s, 
PayloadContext *qt,
 av_freep(qt-pkt.data);
 av_init_packet(qt-pkt);
 qt-pkt.data = av_malloc(alen + FF_INPUT_BUFFER_PADDING_SIZE);
+if (!qt-pkt.data)
+return AVERROR(ENOMEM);
 qt-pkt.size = 0;
 qt-timestamp = *timestamp;
 }
-- 
1.7.9.4

___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel


Re: [libav-devel] [PATCH 1/3] rtpdec_qt: Add an accidentally removed allocation return value check

2013-09-26 Thread Luca Barbato
On 26/09/13 15:43, Martin Storsjö wrote:
 This check was mistakenly removed in 5626f994f.
 ---
  libavformat/rtpdec_qt.c |2 ++
  1 file changed, 2 insertions(+)
 

The set looks fine, thanks for fixing all of them.

lu

___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel