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 2D16C449F3 for ; Thu, 29 Sep 2022 18:08:44 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 4A11B68BA6D; Thu, 29 Sep 2022 21:08:42 +0300 (EEST) Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-oln040092065073.outbound.protection.outlook.com [40.92.65.73]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D5E1668B9C2 for ; Thu, 29 Sep 2022 21:08:35 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=in88JHvwOeSO6lfwlPhNOCJovJPm7Nx41paXuR6QzFGfXJIYlZPBs0lXbzHD3dz7qO7g76hFrxkMtDbTKiFTEtaR6QOd4/eZQjTzemWyvuGbXa9xIcWc2oj7J1QbSA1UBo+Plc9OXq3kgZSiWlKFLlgJ7I2BHOeUeFELHoiCGeXiIyL33HsPyB6zreef8TNuleGEbDsAVFZNf8Wfra7/nBfem4yqWYQuCccT8zfA79ujUPmkzj74OmuDvEky5/3N0E73k6bd6LvXvqpPoB+bL3soziVe0AWTyuoPaO0VmJqU3jYr/nwsUsx3E5jetZeI+/IN9XYy1CEjnwzM9OL3EQ== 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=s/x1eB3j3vN5+B3XKnNTpC/LyoDhbAsK2VfcvQFLfxA=; b=Mwwu++vHyFL3rCbDOphgyjHWomqTIWoBRwbO3wOJdRhrEFAZuX9kHTq5Q++0m0KzAD9neJ6QaxFeSCAuLmuLOHrli806ublyzOugsNio3Sl/Rzzpb5B/Z8c3EEw3FJwzpuPtuywQ1j5Ck5fHIfH1KgXNDMe4McV6Es3bb+/GCCgmobNMPBjbr05B2e97pYGqI/J+SoGz7qgEcoFdLlQDuwh13/M8hx2c7T9IBGLNmUldSLqFgyIdjQ7ZWb9oaXXpXL/+fqS2R9g9hNBWJIGvvuGyuJI2Sa/J/Co6VdQd9mAC64t7tj2iCsgUoHQ79TRdq0rFw2HT9ZRrd5PreZTjHA== 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=s/x1eB3j3vN5+B3XKnNTpC/LyoDhbAsK2VfcvQFLfxA=; b=UFHPYXGbWPYIi9eZBoFbbnZf4md9fZOmhx+GCNQ/5HgL8Ag39CDsBpcoWvcH94yPXmf13awDUTAdb8NQS8qnBHsiggZNkgSY96Nt9mCJXwe64Pwem6hCAaCmbQDPyNES9ufOzAeoVWPDg8qLnETAOuDbZFVu5ytxhgMRiqyFt/2ckPWm2Xi2rUCg4woPCHLDeNmocmQb6wnPpTXHWSBtJWi7rwSDxwYzRzk76gHFvoMN4/xEJPh/iW+J6E9UMjFxFQZE5S6BWUt+jCvR7OXw/aQcO/Hf2aHcrpftGf4nn8nQY+iuR/LCP1xnDVCrbew2QravCLMli57zTe8HWiThPg== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by DU2P250MB0205.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:276::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5676.23; Thu, 29 Sep 2022 18:08:34 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::68bd:2fc7:ac52:38f8]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::68bd:2fc7:ac52:38f8%9]) with mapi id 15.20.5654.025; Thu, 29 Sep 2022 18:08:33 +0000 Message-ID: Date: Thu, 29 Sep 2022 20:08:38 +0200 To: ffmpeg-devel@ffmpeg.org References: <20220929180410.GE6583@pb2> Content-Language: en-US From: Andreas Rheinhardt In-Reply-To: <20220929180410.GE6583@pb2> X-TMN: [kwuhIbQEH3Mf8dtF7tfzN4DuNCD3waZRiqaX5tLO1zU=] X-ClientProxiedBy: ZR0P278CA0093.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:23::8) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <1455cac0-0be7-8659-49f2-58733c175c1f@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|DU2P250MB0205:EE_ X-MS-Office365-Filtering-Correlation-Id: 6000820b-b987-4fd5-4f2c-08daa2459f24 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: FjUwktZAXFMH9h9n7qsDxkbzoDRiu7zfwrFRwUoTFsWiUPbGf2Vqfcbj7DdBoU1QWn5lTiukc8vewsc4IXLEq7svb/twQsg8AVYQuPUPW6hHJYwfAAXweH82j3XcuuH69DsJWArVJ3TvzhRcFbrmWQSLxTqPop4M2XlZYIGMDU1vaRQczEL+c8OQvzy23VJSoq7iy2KtGLf+nDoWR+qZ11r9/6cJA/zMamwKgkTjZMZOwz7S/yrjkQUrkGC8QBtAFxuDx/g2bATdiI1zr4jYARHa43TqedUKkzwRYCLFIQIKVBGupvCCKyK34LLej2fqAw1UCKQJDKSMkyYTYCg1YgEep/TEAXb2GhN/w/xmABJSkESwdMvjbI9uXhaq7Vs4n/yF6C8mOiSPxTWlhCwm/CcU/xbUdREGvUOYiUEgX7OnFM8tmJLTEIW40uiVtTEH/8K5JgegSbq6fG53ba+MZbn2JLIh45QrJSnAcoRG1nsmDk8AZA2hgBNfD1iWgdlcPTrVWDSha0MubWYOeD6sYTvWu0RxDnwoUJEcTFgs7VW4mm+zXwYgEzFDrdbhNvZch+C0IPx0KGWo/xBGI3TWWkN0Gu9v9/YpIC/hAbE4KFW4JJd9w2ybninzRlmd5545ynh2+2gnaSMPbbBsRgq1EYA199jObcffo7kelOOxQDxD7B32lriTnJ1oA/R1miEj X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NllQZFUzNkNlSW9JUlNPZFhFbUxGOEpQWjIzZ2tFKzZlZytFTGZ6OUxSaFZI?= =?utf-8?B?cGVYeHZDM28xbXdUN21GNER1bllyM1NYc2RoVFBhWnlLVnMxNUZ6N3dSTHB6?= =?utf-8?B?eVV0OGRsWjNISG96Uk00RHBUNWpwdkEyOFBXOXlmRHBuN1E3RVlrQkxDYWNt?= =?utf-8?B?NWZIR0Rkalg5b282NE9lNWVBd0hoakV5d2UyNVZGYkV0TjIxdmpremRyTi92?= =?utf-8?B?bFpRVEtTNXB0bkdSOXU4RGp3dGpFMHgveDdud29FUUlkZmFYNkpTM0tCYkRi?= =?utf-8?B?QzRWUUZkRDB4TFhucEVCMTR4aUVSb1IxMWlkQUtqeXljamI2M0NTaTBkTUdF?= =?utf-8?B?TytwOGtZYkMwVzQrVFI2QUk5azhmVy91cVA5WjdFaUhjckQ4WGVQd2R3SUZj?= =?utf-8?B?OEk0cWZrc2xCa09qRDNsWUFBcTNvNGMwWENMckRHZ201Y1AyU1hrSGd4d3p3?= =?utf-8?B?bTBScDVIL1ZwbEs5WVl5bVpuZHJhc2JOVzRRd3lzbW9xTEpPRGtsYmVkaDE0?= =?utf-8?B?WlZ5S0xFV01PMGs1VHFXUnNVTnk4dFd2VzY3SXBSRTlPYjZjZFlPWG9GeUI1?= =?utf-8?B?RXR6UFlSRjUwekcvNnhEaDVBd3A2d0FCdU9mektEdnB6aXByT3NGNW9Kd2FT?= =?utf-8?B?SVFVZTlmdXNoVHlYTHorZ1h6b0dVSDZ3NDJiTjU1NkpVWlhtek1uUHZPekJU?= =?utf-8?B?bHR4OU5obk16bmNqOG96QS9PVys0YU11VG5NNjV0UGdJOGF0NkUrUmNqTWV1?= =?utf-8?B?emlEb0hGZkM4K3lhbS9OU2JMRnZqVDRwdDFlNDFUaURPcnFQTkxJWC8vak5C?= =?utf-8?B?eFJkaFIwWXAzUFNUUm81aFFyMUltbkxOSkxhK0xTaDlLU1RReEVOR3hIRDBv?= =?utf-8?B?TnJqMitidmo4L28rYjFyY3poRllVZkdzNVZBWGRKMTZGbXBoTlVWb2lkTWtq?= =?utf-8?B?UWV0amtWT3RGVGNQRDN2S1ZMYVVtRmlZdWt6YVZtNjdrbzZCYUtKcUltNFNs?= =?utf-8?B?OXpCK0FpQ0NDM2h6MFNSS1R0eFVvQXY2V0x3SDNTQk1HalpiWnRpT001WTJ1?= =?utf-8?B?akxwQ1A4b1VKWUUvek5HQUxSTFM2R1lvc0xseTBmUlA5eWMyWmxlMXh4VzN0?= =?utf-8?B?YU90MmRsNDNldDBtNmdaTVRmTk8xbW1YZmxWUmFHekVoOWxwVHlUNCtsZVNN?= =?utf-8?B?VG5IQ2dkMzE1aDJCWWovQmtKMzh5QWZ1anFCQXZJUlZlK0d1MHczTndsVVFm?= =?utf-8?B?MFFWcG5lQlJhOUlETjFZNVFhMGdGUHA3K0RrSVNxVmpkOUVUQ2NWMWNoRVZZ?= =?utf-8?B?b0dNSnF5cG55ekdMNHdDb0tEMENWakZ5MUY3L1psc2w1Q0xNTjQ1Q05mZmdz?= =?utf-8?B?N0xycVBDUjJmSVFObDFNRldWMEZWUnhubGYwQ3R3dXl1R3lwamxzM0hjU0Zh?= =?utf-8?B?R1QvSXVscy9CN3FTdjJjN3k0UENoYVBCVUNtTUpkMjljQm1yWEFBdUJJMFhH?= =?utf-8?B?azlJV2M1alV5bC8vai9FMlViczRvcDROZjNCbGhhYTloRGFEUFhrb0Vvd2p6?= =?utf-8?B?ZzUzNVpieHdkTWtIZ25YMTZYRS9zYW9XMHc5bmhTTW9kRG5aSXl6d0ZYNHEx?= =?utf-8?B?alRCOTBzUTBnbHdoSGQ0V0NHOWt2aTdWcXZuRW1uYVhrQ1pRQWl6TE1yaUtC?= =?utf-8?B?cWV1d1lIUjZyZ3VRRllaV0lBYWRCME9NUU1DVnQvM2l6cE1BL2dYeDZMdS92?= =?utf-8?Q?Afuuah2Jgjd1mihhfC1ah10GQMcpEO7hyhCxO+V?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6000820b-b987-4fd5-4f2c-08daa2459f24 X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Sep 2022 18:08:33.6496 (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: DU2P250MB0205 Subject: Re: [FFmpeg-devel] [PATCH 4/6] avcodec/jrevdct: Fix UB left shifts of negative numbers 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: Michael Niedermayer: > On Wed, Sep 28, 2022 at 08:58:16PM +0200, Andreas Rheinhardt wrote: >> Affected the rv20-1239 FATE test. >> >> Signed-off-by: Andreas Rheinhardt >> --- >> libavcodec/jrevdct.c | 38 +++++++++++++++++++------------------- >> 1 file changed, 19 insertions(+), 19 deletions(-) > > iam a bit surprised this was not spotted long ago but LGTM > > thx > Most of these changes are in ff_j_rev_dct4 which is only used if lowres == 1; I guess the fuzzer doesn't use lowres which explains why this has not been found before. - 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".