[FFmpeg-devel] [PATCH v1] avformat/imf: Fix error handling in set_context_streams_from_tracks()

Zane van Iperen zane at zanevaniperen.com
Mon Jan 3 13:34:16 EET 2022


On Monday, 3 January 2022 3:22:26 AM AEST pal at sandflow.com wrote:
> From: Pierre-Anthony Lemieux <pal at palemieux.com>
> 
> Signed-off-by: Pierre-Anthony Lemieux <pal at palemieux.com>
> ---
>  libavformat/imfdec.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavformat/imfdec.c b/libavformat/imfdec.c
> index f17064cfcd..da8c6cddea 100644
> --- a/libavformat/imfdec.c
> +++ b/libavformat/imfdec.c
> @@ -564,9 +564,8 @@ static int set_context_streams_from_tracks(AVFormatContext *s)
>          /* Copy stream information */
>          asset_stream = avformat_new_stream(s, NULL);
>          if (!asset_stream) {
> -            ret = AVERROR(ENOMEM);
>              av_log(s, AV_LOG_ERROR, "Could not create stream\n");
> -            break;
> +            return AVERROR(ENOMEM);
>          }
>          asset_stream->id = i;
>          ret = avcodec_parameters_copy(asset_stream->codecpar, first_resource_stream->codecpar);
> 

lgtm




More information about the ffmpeg-devel mailing list