[FFmpeg-cvslog] af_hdcd: check return value of av_frame_copy_props()
Burt P
git at videolan.org
Wed Aug 24 17:09:23 EEST 2016
ffmpeg | branch: master | Burt P <pburt0 at gmail.com> | Mon Aug 22 17:14:49 2016 -0500| [8a78fc5b015f34e0a0c877b4b22b33fb961f847b] | committer: Burt P
af_hdcd: check return value of av_frame_copy_props()
Anton Khirnov:
"[av_frame_copy_props()] potentially contains memory allocation,
so the return value needs to be checked."
Signed-off-by: Burt P <pburt0 at gmail.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=8a78fc5b015f34e0a0c877b4b22b33fb961f847b
---
libavfilter/af_hdcd.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/libavfilter/af_hdcd.c b/libavfilter/af_hdcd.c
index 1bcd279..2324dc3 100644
--- a/libavfilter/af_hdcd.c
+++ b/libavfilter/af_hdcd.c
@@ -1530,14 +1530,18 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
AVFrame *out;
const int16_t *in_data;
int32_t *out_data;
- int n, c;
+ int n, c, result;
out = ff_get_audio_buffer(outlink, in->nb_samples);
if (!out) {
av_frame_free(&in);
return AVERROR(ENOMEM);
}
- av_frame_copy_props(out, in);
+ result = av_frame_copy_props(out, in);
+ if (result) {
+ av_frame_free(&in);
+ return result;
+ }
out->format = outlink->format;
in_data = (int16_t*)in->data[0];
More information about the ffmpeg-cvslog
mailing list