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 D4A99428D6 for ; Mon, 3 Oct 2022 22:05:22 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 60BF868BBF5; Tue, 4 Oct 2022 01:05:19 +0300 (EEST) Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2013.outbound.protection.outlook.com [40.92.90.13]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6D9D268BBB2 for ; Tue, 4 Oct 2022 01:05:13 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ai5KhfqR0oqfI6T5Ux83RKmN1ujTsTNPJcAPJMVcPIA2MoEJSjCGyjUZG2DtbHZJfDGoTUkBHSbNYV0yyUQKlKbFMNdC3d9OogC0b4mYKlo2zQgspaEH8+bTnSNhu6dPKZK38MdLqRSbtGfJGrE4QTDJ2oATTc6X/ZzLVE3fQKDz7dIdjKUPPZHHyMK25I5jISFikpUAtXfJOfJ6cfbrLy88ZPKsTzUDSDBI1+vQpqPdVU8C+ElT8AuRBrvRROFRmtr+vnUbtAt6oBeALeEUIGFwWHyG9ahavKaP+ub5VaAGUj8gzCIbDbKfB+riHUdsYRvfSstzpnrh2ChMwKe2Vg== 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=RlYPROAHyVxi1v8XvhdC4vHD4S7FacBMft0g2SgGyaU=; b=jLoLHT+06l+mG6bTU5XordMkp/++936Qyk59jdWvbVRDs0wmROnQoMsGZuaKESE2BDLJ7ftelNP4Ke276HJ+J/eOxnQEF8ZwbKiTnge6Qj8xypmEXOzSa3tGOtQQWSB/sULGUddpjX1aArdSBJT6OSXvSl06jaboxDnPl/kjOi1NhmSaf6U9JqEM2OR421jPgqXdNkan9bgfM9uJ/5qPcW4nYhpaVxiiNudzVzi9jEayr9vXpzJOZK73iFuW4FtcuyDWKd/0F7UaNcflOCPtrBdTpUnXi2wj/3unzBwWZC8tdCerG1Q9UX+UNFH6pi/ingO19LR7RzsIYacVPiwWmg== 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=RlYPROAHyVxi1v8XvhdC4vHD4S7FacBMft0g2SgGyaU=; b=cBN/McvsOU6XqSO/pQmZaTM29KKdKDlF6bSiXRpEy5EJxUEujPyHCnEWfjC1yVZg3dlUm5BNBAoOEEzFV+ei5yHKNwQbTZYMYyhLvrnyaGf+grDRITPdYmjwKwDUMStFm4J5Eu53f9Q4VhLVJVeOqIB/5VsyyKLlzWmoFoV25mXHrGsxf0rtZStah+B6O6JQrFYIk7XiKcNTSCa8LFW4u8OEOFNAvc+mcsnLiiDOaIHftCB5noXdsVxnEQvi63ZETMaAePchGTxIPDYO85CDnEhtN52VB1eD1j54PCL3ZVeJ+KZ5AslmJfO4lAC9J7qihtpD0TrFAO10H0paGe3qWA== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by DU2P250MB0238.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:277::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5676.31; Mon, 3 Oct 2022 22:05:12 +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.5676.023; Mon, 3 Oct 2022 22:05:11 +0000 Message-ID: Date: Tue, 4 Oct 2022 00:05:17 +0200 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: From: Andreas Rheinhardt In-Reply-To: X-TMN: [4BiV+HtBxsEPUfakamdGf9LqCTK5Iw/3Ik6QKj27Y2M=] X-ClientProxiedBy: ZR0P278CA0195.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:44::11) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <743ce10e-454b-0676-9eb8-39cf279cef4e@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|DU2P250MB0238:EE_ X-MS-Office365-Filtering-Correlation-Id: e53e8189-a02f-4036-b118-08daa58b5763 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: /uEtezhjjL7fn2/goDMRykgNzVGbC5a766uMdmaOibeq+ZzMU5kEfk6m73GXTKnfqMuFuCXUJ3Ta1MAksNLehlatnHp+SbtNT7sVMZaktOezxKrD5u/IYdam323zK8g7GiqReoboYOdPp5m5YEPc/7lkZT1ekv6LuduCgEzrtyl1CvWiSjJb4QYBzvBcSP0Hfr05Luy86rYwuWV9L9ycDp56zX5oYqn0Pz3xhnAeYEOFYpzCuAWs+xllhbC1HyPn5gkYcjD7klJ6z/6MYFSfGYwWDpHzf7os9e/OSMs1vpMum/mhGEf8XC9zJULy0zHJTVJaT3fAAG5ET2CNlBPYIxF9YUqlQ/HizkewaswAnF/BW0eLaN4hS2P0zRAQ48r+Yc3baHQ9wupXLirkNcpMs+RflREtx3UsLnOhkriIlyw/TA1LyTwLAx40/0ppW1O0/uONMF34MBYhZkmEUa6qFGP7Nq9l9ntsDhqGIxD9Dh+HlJj+p1poddEwfrYRlTl+eQHETgooTeBZsDJmiQEGp3Z2SG/q59w9fH3h9nhAN3KXzJTyML9NLkKB5W+Luc1Uq86b+FSZHBfmICYODdNfQ6ygXD2A9tUNFlhjQD4casGyHVmt65V7iWeAZQ/cIDS5Urc1GLrsvqq4PvPLyenjH0HVqHqzkzkkJowjF1JVCwFqbyGBf5fhLqQIia1F1mKJ X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UG41d21Odjd4M081TW9kdFhVeHgvbnJNVGZuQmFPSEc0dFZjQnkrQ3J0SndM?= =?utf-8?B?ZFRaVDRmell1RC9QQ0pidHVMTGowUFNVanlxalpqREZPcWFHZ290ZXBTOCtE?= =?utf-8?B?ZHNxSlk5d2plbGVVQW1GZ1FaK2pOVWtXb3VKcEwxWEZWeEFGM2NKT3AxWFYz?= =?utf-8?B?NjJPV3NEYjd4YzNCelE2Q2hVK0N1VmlhQU4vL096MUM3OUxDbHVuSW4xNGc5?= =?utf-8?B?NlhtNkQ4MWZBb1VXQWdyVHk2ZTRvaFJhSkJTNzNBQ1plZ3EvM2tsZkVJeGhN?= =?utf-8?B?QnROdUtDRUcwNVcycHpFQXNQdmZGRFJycTFNeEFnbEVXbFVWd3BlL1BsKzky?= =?utf-8?B?YmxUQzdkandkN0hNNEZZcVZQWGlqbnRVS1V4MUt4R2xtN09MSmFyUW1TYjJi?= =?utf-8?B?QXF3M0tzNE8yNGJoNGZjNnN4MndjQkQyeS9UK2JQK3d6dEJkbGxndHBtTU5B?= =?utf-8?B?ZWhQYWJmME1JRTBRWVJmOXgvQ1V2TTFldE9mTWp2KzBMbUNIa3dGUTZLWWxO?= =?utf-8?B?T2tlaW5EcWI5NTNzTFIxWnFUeGRYVjk4UXJHVjh3VmtvOFFIR0pTVFcyV01S?= =?utf-8?B?cDVkbDVrODd2VnR2eXBWZDR3bnJ0aTRBV2hPQ1hlQXV6YUZEVmtkbWZjLzdh?= =?utf-8?B?RE1wLzVjUmErS2l6WVZOZy9zdldyaDRVd25CbGZnelJUTGQ1elZLeHp5djVo?= =?utf-8?B?T3ZpMERreVc2VnBYNGQzOVhVb0hDQWJ4aHdWOWd6S0Q4cmhDSGYxSVJ3ZjEr?= =?utf-8?B?Mnp5UmNWd1EwSzl4MmR4SnR5b1FQemhLVlRxUVhVeU4yVVdSUlNPNFJ2akk3?= =?utf-8?B?THNrVWpQbDdKVXcycXdLSEJtL2xFcm0rUHVEZzlzVURSRGVteC85THpYYXF2?= =?utf-8?B?YVZsSkh4dzIzY291T2lMRHNISGl2R1pjSW8yZkZBVDJwaDVJQ281M0wxcFI4?= =?utf-8?B?aUh6WkNIYzdSZXYxWUVveDFCeUVmODRSUFc0Vk1uOUJMNTN2UC9GOG5yczA5?= =?utf-8?B?ZTJ6RVJ6S3pPNFhpeUlsS2xXVFFvZ1orNWFsa2l0cWg1YkUyQjgzUVdlUUw4?= =?utf-8?B?ZzhxQ3V5VnB3Slh2dXFLelY2bXNCelArR3lPSklod2FzS1RmZlJZTndBTVps?= =?utf-8?B?b2hwOFFIMTFEd2F2OXUyQ1piVDkxdGRUdDE1eVFITW5sZEp4T2l4VDFZNlhP?= =?utf-8?B?MDRmUkpxMEd1UE1aTTJFWEdNWjJWVm93dmU2bmVHMmZMakFOL2FWYlptbjd2?= =?utf-8?B?cXlJSVJJUDJ2Q09Ua3dhSTB2SnhlVHpWTVIzVzJqVGZwblpEYXVJemxWSEdU?= =?utf-8?B?M0ZTdFg4L1ZJUjVSNUNVeFp0dzNiMTU0Ry84dDk0VnEwQ3kxOTlXYXArTkxn?= =?utf-8?B?MW5SRlNIK1N6MHh3azFtU0p6S0FzRVhMU1BCNVNPTnFuR1pSbTVVYkZNdkVF?= =?utf-8?B?QWNYM081eEgrUU50WWpuL09nVGUzckU1VkM0QUFQSHJTSUw0K2MyQkhEUlYx?= =?utf-8?B?Q3FDSWpNNUR2dmpDYmYweGFMdEZPbDNsZnM5ZkVWdHRzNXJxcldmOW1mZ0hi?= =?utf-8?B?T2NZWUtEdE42YmhXeUZtWVMzN2xvN2NaYmZBUzhub1N1d2kwckRQYVl1MlRh?= =?utf-8?B?Wk5WYXFEYnRVTUlERGhWcFBvMGFKOXdsVStwSzJSV0dMZHVyMmVnSkg0SGY2?= =?utf-8?B?emZGaEo4MC8wNWowc01MWTB5Zlh4c2g1a3Z2VjZLem5ubnlJSGw2UGMxVmJ5?= =?utf-8?Q?YbKun2Pp0WONn0dJssx0+y1pkPcAbnXBBN0/R6T?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: e53e8189-a02f-4036-b118-08daa58b5763 X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Oct 2022 22:05:11.3409 (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: DU2P250MB0238 Subject: Re: [FFmpeg-devel] [PATCH 1/7] avcodec/sunrast: Use ptrdiff_t for stride 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 negative linesizes; in particular, > this affected the sunraster-(1|8|24)bit-(raw|rle) and > sunraster-8bit_gray-raw FATE tests. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/sunrast.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/sunrast.c b/libavcodec/sunrast.c > index 77feef06e1..45b29e4d72 100644 > --- a/libavcodec/sunrast.c > +++ b/libavcodec/sunrast.c > @@ -31,7 +31,8 @@ static int sunrast_decode_frame(AVCodecContext *avctx, AVFrame *p, > { > const uint8_t *buf = avpkt->data; > const uint8_t *buf_end = avpkt->data + avpkt->size; > - unsigned int w, h, depth, type, maptype, maplength, stride, x, y, len, alen; > + unsigned int w, h, depth, type, maptype, maplength, x, y, len, alen; > + ptrdiff_t stride; > uint8_t *ptr, *ptr2 = NULL; > const uint8_t *bufstart = buf; > int ret; > @@ -141,7 +142,7 @@ static int sunrast_decode_frame(AVCodecContext *avctx, AVFrame *p, > > if (type == RT_BYTE_ENCODED) { > int value, run; > - uint8_t *end = ptr + h * stride; > + uint8_t *end = ptr + (ptrdiff_t)h * stride; > > x = 0; > while (ptr != end && buf < buf_end) { Will apply 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".