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 C94F942BAF for ; Wed, 12 Jan 2022 09:51:34 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5971368AD5F; Wed, 12 Jan 2022 11:51:33 +0200 (EET) Received: from EUR03-VE1-obe.outbound.protection.outlook.com (mail-oln040092072020.outbound.protection.outlook.com [40.92.72.20]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 5602F68A72C for ; Wed, 12 Jan 2022 11:51:27 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=H1AUku6sWbP54XkD7T+hyC405/JwB4V6ujeiJx8VsYP1izhKWM3UjODTl9qmVtZBzaMIno2DHvL5SaFYiSo+A50L8q/Vij1R4M35V+xUpkssKIPU4d539IFZw6IHI4VklFgLuf18DS8QHwVywDNgMl/kfEI6g3Tp+/lUybqwbnAGaaLD0wsfiPw3ZMUyjfHRwKwgA3y/UpX42oQrG2bi3wAidYuOppa+t2gp4d3BAdW3W97kZF4NKs232w+xuClIphuFnXG6JLGcgbhxuGN2B/WYnn5XITO02oJCfd8b7MBImwbDzrMNRLbimCdoSOsOCsi7EtxoglYdQJ5cnwmliw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=t5SOs/+/rO4rFu26gVFDmyyC43teH1BRJjCa4Ri3pOs=; b=XNbmt9K3i7vPvuUtx80JlSQrNdlNc4aILVvQFp8Tv9vsccC9K/npzj5Wp09nJ5UxKJeovrWIjw3WKw2qSmNyfpD2Poqlp3pRiznvugUsodg4anrajEVhhGKbjoksIAN4WfMtREo/ctOi31fvNYbEVigTS5r0U32DNLdHIUA5eQGuEnU7gZOhOX7c/h8fZczRqk3gizfifleqJVpKuggL/+YiGkRHoE6QIl/NmXUuHCjTb3LHGdbbyCz0RLWrC9SKdLmLgcSW+3+5zVvo0Jg+eJ+ElJRAIo103nTVAl87OH9rha2OT8dC9pFfgBIonvNEPu6rF1ZeVDdfsO/zlvQ13Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=t5SOs/+/rO4rFu26gVFDmyyC43teH1BRJjCa4Ri3pOs=; b=UwRyM1tpPbjx/qG0UTvsotQA9PakfyEUR+weRvSZufoYVtpqpacsQ4pSi3E69HF7U4RPAYM/NdHqyCr7ME3U3I9Lm2eq7v5xB/nmncTbxGquAlap4X9AdDcMDDeCeE3su6eR4Rf7k7yvs83CAwX2Qh4tWIlNsK9BNYQoC3M/DbiBWznG5EDEptn+6NC1mfBY6sEPSWzIzkQccJaT5a9WaBXnIonQCrlAE1iRSZ2cwfILgcTUoAUXa9vOc9Drq/95Vf4hv7xbPUghTKp2O032avK/X6Rk8LZoackkg95Ek1jD6FnUVj6iwCrAxdRGu9fQpjHwZBzyYfGGk775MfsDyQ== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by AS8PR03MB7144.eurprd03.prod.outlook.com (2603:10a6:20b:29e::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4867.11; Wed, 12 Jan 2022 09:51:25 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::19fc:be9f:2c9c:53f5]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::19fc:be9f:2c9c:53f5%9]) with mapi id 15.20.4888.010; Wed, 12 Jan 2022 09:51:25 +0000 Message-ID: Date: Wed, 12 Jan 2022 10:51:23 +0100 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: From: Andreas Rheinhardt In-Reply-To: X-TMN: [llWPmUAQ4BmoT2TtA7kjkxsVkRAe7FqS] X-ClientProxiedBy: AM0PR03CA0077.eurprd03.prod.outlook.com (2603:10a6:208:69::18) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: b87a397f-82d0-4f10-c541-08d9d5b118b5 X-MS-TrafficTypeDiagnostic: AS8PR03MB7144:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: J/2jepiSqtau2J96/QaOZWzBHSYzheYcVvcSVMcAX5khzIELH9vHtiQ32ex0oPAq5rDP8K2BXNcQG6uaJc5otf8AlSYfHUIDp+rMxI4xPLG4iRwE/5Cbd/tfiQ3sLJs7uQHlGP0C5uvzHPeAsI/oEe0ZcdOyJg4pOrkwPX43fiPIKmHWQiv6uGPcVD8kBiPzAnSsYDJGODz3NXuRNlHwkC/mS7cC9OdGC39fiE2iFadanYsH5T3RpNaRWGKV1KwPzoJLwcIi5SqrBOo3CHglQ9cdOBGJ2x5ClFSW14c4aCyObuE0FDJiK5g+XD26SBMm3yoAIT4rBrIy0010cY9WE9/9hOGvb+GSlSG+6vwBdNWAIdg+ib+wn1eCh4yQgKtU7Qsh1sftRzgbzXRYfvZGPTFT+nrtNu1W8yWtFdihEHRDMoNoKG+gyAkmHJV4vIgDaHbiNid7gsunY/ZTy+LW/WhV1V0atFH8pX01XbreGBGmPcYpiKJaicTRSC+2RHmuPft0kR5PF7XK/2Gec1HE/fQJZorH244zrffImBKFKBkLJXm/CAV3hRICNuOM4HcYaWFFPwKq8zoTBGYuj+wh1Q== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MGpMSHFDTTV5SHJ0RFEvTzVvVkw5VStuM1Y2YzJpRTJLSUZSbjRIQncxMmh0?= =?utf-8?B?TkRuQkRMVlNpd2RwZ1lpS3NOS0k1dFJaU25nQWFGajE1UnRMSUtGS2MrU1JJ?= =?utf-8?B?SksxS0ZiQXI5UmRjZEhIRWhCUE0va1grTHNWdlhxbVNDVlJaekcyWmtQeFZh?= =?utf-8?B?S3Nta3Y2UE1udHNjNnlwRit5K3A4VElYV2pESGE4VVlpdXl0VUs4TExaYytq?= =?utf-8?B?YVlQVk9qbUN2dmJRbjZ6ZjVidFlJRU4yVkduVFp1VmpSa2YyaytxL2dwUlVw?= =?utf-8?B?blhBOW5OWllkeXN6aFlHSUsyTEkvbDM1QmFhcjhwYkw4RVFPbXo3cFU1L0xu?= =?utf-8?B?OWI5R3A2WnZ1NWJWRUhMck9Od0NFeVlKOHY5VktTb2Z3Z3NFYk03aGh1TTNF?= =?utf-8?B?MXFoU2J6TFhjeGR5OUV0T2FHa2FhY2VHMkxrY3ZGVngvY2w0RkFyQmw5bDRm?= =?utf-8?B?Ujl5bkJHK3hDaGZUWmhsdDgyKzBIbnN3d2F0TVpQZ3ZRYzhvSEdtZE1ld3JK?= =?utf-8?B?VjVzcVFESW1nNHlleU1LdmlOWnVXSER2MEREYVQrRmFiaURpZ0tJMHhJazdL?= =?utf-8?B?QlpMdCtaVHJybk9iNkZrYUNWaVB0aWxyK2xRbmx4QVg5UTlYaGlzYkRxQlZW?= =?utf-8?B?dDFRNUV4SkQzZjJBdldGOS92ODdLY2M0bndyS3h4T0NyWWE1bVF2dFVOcmY1?= =?utf-8?B?V29RK3NJRlVZQUt6VHJmR2VjazJTUnZxeU8vRGlkQ1N3N2J2N1FyKzczaXpx?= =?utf-8?B?NDZuUmV1ejExWm9XUm1GTnVHWnFFTSsrS2VEaitVRW80d1M2eExIVW9KRSs3?= =?utf-8?B?RDIyL1RWeWhab3FZeHZmbFBuZGx0NDB2NGNZZEVpYkp4NzNjYm4xN3hZUG0v?= =?utf-8?B?dFBIcDRHN1FFNXdkWGF2WTdHUTRtdVo2TXN4c3FGTDYxL0pYdGxnTGZJUG9z?= =?utf-8?B?MVFVc0d5cXRUQ2hCMnhuUHlRU3RuL1pMS3ZYYkk1aXZhQ0JXRTlTaXdaTU80?= =?utf-8?B?KzJlcEJyeHVUbWtPVUswZGF0d25kR2xrR2pxTkhHVTFVRldoYVNPQjIrbS94?= =?utf-8?B?YjRMY1V6dHhXcFFOSUdqWXVGZFZhdHdPYTNINE4yZ3FUUWVmRTVMZTNVa2pE?= =?utf-8?B?cFlReFB5OWtXL0xIdWxBTzdBYUVNc3pZTGl6OUhvRUhnbG5yVFFLais2bFU0?= =?utf-8?B?Z0VlNUJtMTJES1g4aExWV1N2RGlEYTROY3VRTFBnUWhIYTIxa0ptM2FtcTFT?= =?utf-8?B?d28zZ0l1TkpzZU0zcDJIcE5rMVNaallWWlp5N2cvalJwN292QT09?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: b87a397f-82d0-4f10-c541-08d9d5b118b5 X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jan 2022 09:51:25.2003 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR03MB7144 Subject: Re: [FFmpeg-devel] [PATCH 1/3] avcodec/vp3: Don't output bogus warning 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: Andreas Rheinhardt: > It is perfectly fine to have from one to seven bits left > at the end of parsing. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/vp3.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c > index 5b9ba60f49..791e531862 100644 > --- a/libavcodec/vp3.c > +++ b/libavcodec/vp3.c > @@ -3149,10 +3149,10 @@ static av_cold int theora_decode_init(AVCodecContext *avctx) > "Unknown Theora config packet: %d\n", ptype & ~0x80); > break; > } > - if (ptype != 0x81 && 8 * header_len[i] != get_bits_count(&gb)) > + if (ptype != 0x81 && get_bits_left(&gb) >= 8U) > av_log(avctx, AV_LOG_WARNING, > "%d bits left in packet %X\n", > - 8 * header_len[i] - get_bits_count(&gb), ptype); > + get_bits_left(&gb), ptype); > if (s->theora < 0x030200) > break; > } > Will apply this patchset tomorrow unless there are objections. - Andreas _______________________________________________ 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".