Re: [FFmpeg-devel] [PATCH 1/2] avfilter/aresample: Check for mmemory alloc failure for out sample rates

2014-10-02 Thread Michael Niedermayer
On Thu, Oct 02, 2014 at 11:58:01AM +0100, Derek Buitenhuis wrote:
> On 10/2/2014 11:55 AM, Derek Buitenhuis wrote:
> > +av_log(ctx, AV_LOG_ERROR, "Cannot allocate output sampelrates.\n");
> 
> Also fixed locally.

patch LGTM

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

Those who are best at talking, realize last or never when they are wrong.


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


Re: [FFmpeg-devel] [PATCH 1/2] avfilter/aresample: Check for mmemory alloc failure for out sample rates

2014-10-02 Thread Derek Buitenhuis
On 10/2/2014 11:55 AM, Derek Buitenhuis wrote:
> +av_log(ctx, AV_LOG_ERROR, "Cannot allocate output sampelrates.\n");

Also fixed locally.

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


[FFmpeg-devel] [PATCH 1/2] avfilter/aresample: Check for mmemory alloc failure for out sample rates

2014-10-02 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis 
---
 libavfilter/af_aresample.c |5 +
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c
index 5f34321..550da54 100644
--- a/libavfilter/af_aresample.c
+++ b/libavfilter/af_aresample.c
@@ -103,6 +103,11 @@ static int query_formats(AVFilterContext *ctx)
 } else {
 out_samplerates = ff_all_samplerates();
 }
+if (!out_samplerates) {
+av_log(ctx, AV_LOG_ERROR, "Cannot allocate output sampelrates.\n");
+return AVERROR(ENOMEM);
+}
+
 ff_formats_ref(out_samplerates, &outlink->in_samplerates);
 
 if(out_format != AV_SAMPLE_FMT_NONE) {
-- 
1.7.1

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