diff options
author | Anton Khirnov <anton@khirnov.net> | 2022-11-27 10:47:53 +0100 |
---|---|---|
committer | Anton Khirnov <anton@khirnov.net> | 2022-11-28 10:34:10 +0100 |
commit | a53f26a4f2467002ec6ff75d7f31800da24bb245 (patch) | |
tree | 947d76bbad4adf8c9a8815ad82de02610025d5fa | |
parent | df24f4de714d74eacea2cc322c32b5122f9f55d0 (diff) |
lavc/libx264: do not ignore memory allocation errors
-rw-r--r-- | libavcodec/libx264.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index f776d65588..8b4b986f12 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -468,21 +468,22 @@ static int setup_frame(AVCodecContext *ctx, const AVFrame *frame, size_t sei_size; ret = ff_alloc_a53_sei(frame, 0, &sei_data, &sei_size); - if (ret < 0) { - av_log(ctx, AV_LOG_ERROR, "Not enough memory for closed captions, skipping\n"); - } else if (sei_data) { + if (ret < 0) + goto fail; + + if (sei_data) { pic->extra_sei.payloads = av_mallocz(sizeof(pic->extra_sei.payloads[0])); if (pic->extra_sei.payloads == NULL) { - av_log(ctx, AV_LOG_ERROR, "Not enough memory for closed captions, skipping\n"); - av_free(sei_data); - } else { + ret = AVERROR(ENOMEM); + goto fail; + } + pic->extra_sei.sei_free = av_free; pic->extra_sei.payloads[0].payload_size = sei_size; pic->extra_sei.payloads[0].payload = sei_data; pic->extra_sei.num_payloads = 1; pic->extra_sei.payloads[0].payload_type = 4; - } } } |