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 7264A47B4F for ; Mon, 2 Oct 2023 23:37:57 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id E0A7E68CADC; Tue, 3 Oct 2023 02:37:54 +0300 (EEST) Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-he1eur04olkn2094.outbound.protection.outlook.com [40.92.73.94]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 676FD68CA8F for ; Tue, 3 Oct 2023 02:37:48 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=OZoJ6lSSQHFVy0Q0YsymyAdGlykvF2IN/AuDXDk82iBwTb5tbyAIFOq53yEqu/ZnZvmr5ONZ4TpG4QQ7tppStKDwgQQKph5EqWZiyX7fRU2mk31HcdBOmDbfkrsX5Kl93SNM2PXLFHOxjO53UKt8oWcHDwAAAUe1QP5rzEh2XKafyOmh/lQQZwtwFIaAldaNPXHLhJEm9kU5FvecaUnfvxnEaK17JU6qECaqhHB0FiyXOB5BUKGxDpPXTb1lRylru9lq6pey/VvtPZtLe7+DcOjCNUxHYYoALESVQX/VRiy+KuF94elzzD97IGPZCf/c8eyDeeFjxYoMO4FKj2qG/A== 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=C/PTWvMPeuM1j8mxqIzNlgQr/Oile+XV61F+57SMlcg=; b=RxOauP26XTq07wDnTnmn7uxatYLy2WWrfM1JVnjktfo+zgO+RudHUNgiy20D5sOOslwKY1avTtXwMacTiNUOSMlAdNNZ2zBeYi4xNm+UjsMcd+72OY+6AjAfg/9UWF+k58CNLpaHS1DRih3GF5UNrH/BdvheybYd35MglytCe/QVpBRRsYM6kad4xkSIEIw55nlMGuGfgXuRlPszTdCvvn4CfmsIKarn8iY0rZJVQx0KIUHp1yMbBYwSysPaw0ZUFgKQg53L+Gf+Tg+ao7ESDWtxIcR/7sloKE1KH440R5EW7LmfzRVx6PRE7RhAiJ6RlJQKvP6SvRXu1XKTOZ2ewg== 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=C/PTWvMPeuM1j8mxqIzNlgQr/Oile+XV61F+57SMlcg=; b=RfFR1WMX2Zxmt/EoUndG9JdeoUhcp8rCAHHV2X9CXoQkTPJw365NiVmSSw7yC0CiQO268nM4Q4kTwVquWdMQy7eQs7h/hd9I1VGCV7TbGpP7wEPymZkg7vpDM9CnP8nGwxHDI7hojOQuFX57jFapU7Wr0oMtbSuuNbSjF1Fe1ZQq2vjOCFWl+GB3AsijpBdffKJzbQ2cQZ4aeEUM9NyQqlJIW4e18M9Tp1303bkoJCPF8Q1fwKTDFp7N1pbAXVklf7Tao8w9qQtZz3/4XypLdGLa2R2jfPXkV8DIODxtd/boQPnGtAYcuVRAmNpK6MHHEMTq6DYCRc2+HEwBHGH+Tw== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by PR3P250MB0049.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:14c::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6838.30; Mon, 2 Oct 2023 23:37:46 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::5e01:aea5:d3a8:cafa]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::5e01:aea5:d3a8:cafa%3]) with mapi id 15.20.6792.026; Mon, 2 Oct 2023 23:37:46 +0000 Message-ID: Date: Tue, 3 Oct 2023 01:39:01 +0200 User-Agent: Mozilla Thunderbird Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: From: Andreas Rheinhardt In-Reply-To: X-TMN: [90CtHbOspYKqGhYgQj2Oa9qi0uDyu7PR] X-ClientProxiedBy: FR3P281CA0049.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:4a::22) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <2d8130b6-aa32-4b65-8b96-e3cdd3c80d69@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8P250MB0744:EE_|PR3P250MB0049:EE_ X-MS-Office365-Filtering-Correlation-Id: 37caf681-fde6-47ae-0152-08dbc3a094ad X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: YQPAEn8O4WLA6SX39R07TL/7GHM801PJJrprxL/sUbLVUYhBh46NLNurgD4L4vZF+nMyvcUqpDcmB2VdXjmXrcF8tU73Hitmf0tzhJsAvuKBHAvQJ/mvctwpRKcoaaLPO3faEHiMf8pbjJ2kkRiDfE/zXkao48KQtbTCmj1jc/3n6onZ2eBP258MEc35XlGSfSpbLw/UhfyNNWnXGKkSXxbzyZTAyJWA/4qNq1FTY7dl55v5G0rdUAMpacEfySn0XrvLS9C2CMzD1GcoKu9+xBtoe+hAVFNCtZJE4682j+jiv+5msFpx/MDOv/ErxvaM7kmK4FNomtxixrQ20AFQ9mnvsfgXn0SkVix3Bg7/PHF1oNNb99PRMg9nDrJNitl10zFgDjFysHC4ssj2taowLOnrwOGIyJ/4FfIFHKiTpnQRlnU+py5EN+8Rz6DR7dJTdYUnLaU7WejFoLo/AcTBnqyeNtsLMIdzzuKft0cTSay20kC1gufxte4L9ifqY5rSsEGsb4sIoaLxrX2pDvcL/VhBW/qkOpqxUjds56YjLwA/DgZfYgcaFQwYACr9H1bG X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?T2hNL3p3TjVZMm81V3dxVzdCTHJySHFBVHZXRWplL1JZdmtxME5PS05ETVc2?= =?utf-8?B?cTk2eFZrSko3RUpUUTFrT2V6SDI1TGJMNU1lOVluSitQTVBRNkR3QU1ZcEpT?= =?utf-8?B?WHdlOXVnK00xOGd3azBNcVE4aHlTb2pRVk9MSGllWXk3QXN5d2ozK1RZcG1u?= =?utf-8?B?MEdTR0V5KzhoS1haYjBod3UwNFNqa0t6bGhpQWlESmphNk8wdStPb3NZV05w?= =?utf-8?B?dmJxRG1BVUlLNnlOV0p6M2pnK2I5REVORHJaLysxWjNTU09GYjc1eVpvZnhp?= =?utf-8?B?NHl6bDVjVFJkbEVjOXA4NGdrOU45MnNGUFhkUEpnQ200UnEwYW1HVWgyRWNW?= =?utf-8?B?K0RMUTlZRWJaUmNQYzBWN2NybGFlR0FRT3hEUlVFVmdtdWdzd1JjNFBoWlZV?= =?utf-8?B?aXZSclNHSmwyUDFaQnpaWXR0L1VTWHYvWWpWQ3FoNUxRRE4zWG02T1hucjly?= =?utf-8?B?SmxDeUJVek9sbHNDK29MRmVHazJsdFdiZDBVTUlybkpMMUpCaEJaM202UjFz?= =?utf-8?B?RWdtWS9YSURJcTNtMWxYMGdpWFliMU4wM0tsWHBMeTJndG04UlZRZjl0d0Nt?= =?utf-8?B?Yk1GVGRmZlJ6b2VHUlRLNHVBdzdqOTZ4ZU9GaE41TTA3R1FzQkt3aDRqMUFh?= =?utf-8?B?QzNtM2ZZYjdyRFgxMW1OcDd5dkxZSk5YaTBXTUFaMUxYOTNKeEFzaEY0ZWlk?= =?utf-8?B?L004azJxL3FUZHM1UW5JSmM3ZFBMYW1CTkduRzg3alFqVmJiQ0g4b1VDcFpE?= =?utf-8?B?bys4YzMyYjdvTUR5Mjh2ZW1MdzYvcDhUclhSWDcyRmgvUGRmQUo0ci85Rk1t?= =?utf-8?B?ZEt4b3czMUJnWDRxczNkRTBJSDFkQmMya0x1d3kyR1FKOVJidGJqSUV4WXhH?= =?utf-8?B?NVBCMDVMK3dySmlyWEZpZ1E1VkxVZ0pLdllacy9mUTBiYnNOcmJ0bzJLQVlN?= =?utf-8?B?ZGo1a2g3bWFTRitDcEVNWjcvYlFKZ0ZsNnpsYlFacXh3QktJUlIxNWUwMmlH?= =?utf-8?B?OTY2OURuMTg4dnU4bUx5RWtzN0hzM3lOb1VNc1ZwMk9lWDA3bFhmdHN1VE5R?= =?utf-8?B?Z0NpS3JoZHhBVEd1UnFFYW5sQ3RhSnl1Sk1iV080c0FtazJQYzl4RCtIMkYz?= =?utf-8?B?MWxPV1VUMElGVW5oT0xOQTNoZXNXelVtRUx6a0gvb1FmeGdmdU9QaFBCd1Jo?= =?utf-8?B?ZjhPSmViSmVteWNzalZZTGMzdnU5QUh5T3ljWWRISC9zTllQQjJGQWdLUFVa?= =?utf-8?B?SXdpK01OMUlXMHZmcWh1SHgxbEJmUUZuc2dZVlltRlpHZlFuS1VOdUNDUkpq?= =?utf-8?B?MDB0OXZJWmZHVmNyTVppdTE5MTlLR3dINHpLZUpVc095cU5xMnJCVE44K0dk?= =?utf-8?B?WG5MSHRuVStrckU0YlErQjU0eU84dXkxZWVtNzBQRVlyZ05SU3ZJbWNFcU5i?= =?utf-8?B?QWZXL3hPYkVkTXN2ZGRWaWs0KzhhV2JXTko0cGk0V2JtdlNaQmZJc2JWWkNV?= =?utf-8?B?LzZRRFIvck5RSDV5RFNZQTFobzFyRkxKUzh5V29hZ3FGVEgvK0Z6MTV2djIv?= =?utf-8?B?VkRjR01kOUZCaHNBMnp3U0tLcWVINGlud1BSZUJCMm5DYmFoa0JuYkxTSkJ3?= =?utf-8?B?VHltUjVaZklxcXNhSmlHb1N4VnBCMTVKcG9ERXJ0NFdwOHBYMjBscXZiRFdu?= =?utf-8?Q?4ESOMG1ABKa8irgaNVAp?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 37caf681-fde6-47ae-0152-08dbc3a094ad X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Oct 2023 23:37:46.2221 (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: PR3P250MB0049 Subject: Re: [FFmpeg-devel] [PATCH 1/4] avcodec/mpegvideo_dec: Check for existence of planes before accesses 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: > Fixes segfaults with -debug +nomc -flags +gray (presuming > a build with --enable-gray). > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/mpegvideo_dec.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/mpegvideo_dec.c b/libavcodec/mpegvideo_dec.c > index 7aa46a4e25..77d90104f6 100644 > --- a/libavcodec/mpegvideo_dec.c > +++ b/libavcodec/mpegvideo_dec.c > @@ -249,10 +249,12 @@ static void gray_frame(AVFrame *frame) > { > int h_chroma_shift, v_chroma_shift; > > - av_pix_fmt_get_chroma_sub_sample(frame->format, &h_chroma_shift, &v_chroma_shift); > - > for (int i = 0; i < frame->height; i++) > memset(frame->data[0] + frame->linesize[0] * i, 0x80, frame->width); > + > + if (!frame->data[1]) > + return; > + av_pix_fmt_get_chroma_sub_sample(frame->format, &h_chroma_shift, &v_chroma_shift); > for (int i = 0; i < AV_CEIL_RSHIFT(frame->height, v_chroma_shift); i++) { > memset(frame->data[1] + frame->linesize[1] * i, > 0x80, AV_CEIL_RSHIFT(frame->width, h_chroma_shift)); Will apply patches one, two and four of this patchset tonight 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".