[FFmpeg-devel] [PATCH] avformat/movenc: Fix invalid check

Jan Ekström jeebjp at gmail.com
Thu Jun 30 23:18:34 EEST 2022


On Thu, Jun 30, 2022 at 11:15 PM Andreas Rheinhardt
<andreas.rheinhardt at outlook.com> wrote:
>
> Regression since c9de096851803d45444ae9dfe3a390a2d53ac71b.
> Fixes Coverity ID 1506839.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavformat/movenc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/movenc.c b/libavformat/movenc.c
> index 0ef6b3c76e..a4dace7c1d 100644
> --- a/libavformat/movenc.c
> +++ b/libavformat/movenc.c
> @@ -424,7 +424,7 @@ static int handle_eac3(MOVMuxContext *mov, AVPacket *pkt, MOVTrack *track)
>      if (!info->pkt && !(info->pkt = av_packet_alloc()))
>          return AVERROR(ENOMEM);
>
> -    if ((ret = avpriv_ac3_parse_header(&hdr, pkt->data, pkt->size) < 0)) {
> +    if ((ret = avpriv_ac3_parse_header(&hdr, pkt->data, pkt->size)) < 0) {

Geez. Sorry for missing this and thanks for noticing it.

LGTM.

Jan


More information about the ffmpeg-devel mailing list