From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 4BE3D45DF7 for ; Tue, 11 Apr 2023 21:49:32 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 1374168BBE3; Wed, 12 Apr 2023 00:49:30 +0300 (EEST) Received: from mout.gmx.net (mout.gmx.net [212.227.17.20]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 4F8D2680817 for ; Wed, 12 Apr 2023 00:49:23 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=s31663417; t=1681249762; i=barsnick@gmx.net; bh=+RBvyokWAb/6LEKO1J5E0pa2CFJ6JExMUBiNM2BOhkQ=; h=X-UI-Sender-Class:Date:From:To:Subject:References:In-Reply-To; b=pXtnESTVRs8sBevNlwJF+6sWtROsGFK5LFqKzv86YfUjJ6e6YtAn2lzmAanZ4kSpj XUc1tKIVz/a6HpZGSr4+GkTiAvY2pa0kodIBaJO22MfjjDAiTnVZijWPFyPlNQV7ki faXOPdrNNNRCBgOl6zM4Y1upFY4md0d0UcuwGqAHriW7notJD4Vc5rmR5auSeA6UHQ 8ZWOtAdDdnHYXCkH8MEO6+57bRvFBf4OCYFX14vOSk+88y1pL10qKv0659Cx9XP5RD hPIXK3al7Ceh9r/T0OqW2kpGTko8aCihlLVsPW6jXNtxm9HK07tz65MjJtlYE4BLW2 bD5WlJnN1qwTQ== X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Received: from moonlight ([89.182.7.99]) by mail.gmx.net (mrgmx104 [212.227.17.168]) with ESMTPSA (Nemesis) id 1MIwz4-1q6ds80f6o-00KObG; Tue, 11 Apr 2023 23:49:22 +0200 Date: Tue, 11 Apr 2023 23:49:15 +0200 From: Moritz Barsnick To: FFmpeg development discussions and patches Message-ID: References: <20230405225044.DEFF3410810@natalya.videolan.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230405225044.DEFF3410810@natalya.videolan.org> X-Provags-ID: V03:K1:5EwgqRU2FreNlBpfeFRpnO1lFZ9LBs/T4pHol4XYGK/8PQwYNrO 03pIUCEpeSy8k2RPSknTN+MDfZAz7R0rRVQVMNtNvsV/G2WWlCp3koZv1K044dfAJnKamwM l09a+GM3UyphTuFe2zjfDNokp+wgAODc4GuhhG5l9YCKy6+rmfbzsahesR2+Oxfea99jDGV X5Qc+E4MEwg443h35k8vQ== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:Qn4mtw+hVi4=;vEskusXd/FF5mpAYhbLagWxl7WG WU+zjn6R0ZLSVs5I91yRjlEjblAX8AKPlppvlV3RUe+A9f+G0nkDm252L8Msf0+nYWMXDlTsL MU11QqswOdb4KPqvqRaJE4ZiYi5PYTujJ+eh/Hbup6lU2XRieH2OgfE6Btu9gVbPWc0JFWWE3 sQG19ihKThnhCY23nwN/i3p5x+1Rqe+s1L+s6jSeryoaBOVGMK5wFc6hNQhDhtySqExwJKtUu /vEV2it4yhqvJaGfM9yTgHQvUrvVsjMe1iGvPYS4pnQHZBFoaWFe/0TTSY0t/vPVvCUl5sJ4A Xg9PZH4qlx0RuHmJvuejtXqo7R38QwXVHnpu8OjA35Z6XtwZjU32diSo1sB0fWc6Xy+7x+UqK 11avVUfNM1qeChAIY7QDcY6kSg5rdec3RoEApRiiDSWuXyamWg69u1n73npIeaANg57/NSrE8 SGbONOPY9sxnX5bFbw98ssqZZH4hsPQ5kyU5IBYYmXfckZtzJMdUfUKKNsKTG0ydLvSSJ81PX hRgu0nDxyxn7y/an1WJOyxteJiRgbpnSPDIyz5X25EKrnulSJrn5Qc05lkk9mygccJZwKEURf Rjxtj9h6SZyytfYKyRjqH2SzOH4zDH0+cXKCAO9ixNTsa+QTObULQdmB80WCqrvIr0Ulp9auy O0Pik1If4gIbxq2m/Fu4dWsqoAMn4Ik2afmJrYHeDhAFYLJ6czGEUt0IyBu5U8mfge9dH6zYn H9dhaPBEgEGqJfv+smanUzbRDuZVBdSX+AiBUChD4mylDwho8BGuFWYSsm69BiJ2ROl7rdh3o KFsUHz8bx9HL1PCsIbKlJSlGTrdSsUpZX7ahwr1ZHn0SHsccSEcFL43ulR1ZbvdEy4TO2RHFy 3bmPEhxIVwi6JPSGVVDplLISfTj+rO5vPKGFe6d+dBI/3v9gL7EnjvaermqdIET8K0plBbka1 COEQPQ== Subject: Re: [FFmpeg-devel] [FFmpeg-cvslog] avformat/mxfenc: reject unsupported ffv1 versions X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Wed, Apr 05, 2023 at 22:50:43 +0000, Jerome Martinez wrote: > ffmpeg | branch: master | Jerome Martinez | Tue Apr 4 16:57:03 2023 +0200| [feeeefc3db04810286cd647f235ced0ec6c3eba8] | committer: Michael Niedermayer > > avformat/mxfenc: reject unsupported ffv1 versions > > Signed-off-by: Michael Niedermayer > > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=feeeefc3db04810286cd647f235ced0ec6c3eba8 > --- > > libavformat/mxfenc.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c > index 9eba208ffb..d8252ed68f 100644 > --- a/libavformat/mxfenc.c > +++ b/libavformat/mxfenc.c > @@ -2493,6 +2493,10 @@ static int mxf_parse_ffv1_frame(AVFormatContext *s, AVStream *st, AVPacket *pkt) > if (v > 4) { > return 0; > } > + if (v > 4) { > + av_log(s, AV_LOG_ERROR, "unsupported ffv1 version %d\n", v); > + return 0; > + } > sc->micro_version = get_ffv1_unsigned_symbol(&c, state); You inserted unreachable code. Cheers, Moritz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".