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 369FA49823 for ; Mon, 19 Feb 2024 23:23:30 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 46E3F68D443; Tue, 20 Feb 2024 01:23:29 +0200 (EET) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05olkn2018.outbound.protection.outlook.com [40.92.91.18]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id E53C868D376 for ; Tue, 20 Feb 2024 01:23:27 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Gi0E0SlPpQmMojNIVnovF3CyPbnG0AqgAzzM1JgXA9O0MKZpXBx9fxr9wkWS1a4TVlxmm9TGItiQ4/vS9Ik1czoqDH88Jb61bCtV3CT1/5Jl4SjPYBo+VB0AvdSqFpHYoGJZxYPKJgE2xyk8S8cz5spvAfC8cJDEoijWx2WowMsNzE7YmVrHHXBiCAlgw5R0Drskiu5NE5yarzWfjkAQmchn1+CfKN5apg6bqYqtyhJWwC2L04QPEAUMs/C6WFECSikj15o95uC0HOMLUpAH/Ur8EX67LFdozGNAzyAtvjAOeMHoQRURls7rA0xzCClU/oOLWSKVCsuie0SNuOROuw== 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=3OAK2AdXdJZlR6XHNUOhnZDtZeDxDh0X/pF3rg/WEUk=; b=EwZzOlfejwnrZ8inBvHBQLd2Yn82vKO54LokXhVEaKYA/8lPGnNjxBkZSgTWeK0O4j2AjFB1Z84Z2BmC2bgioIGadU8JdSE8MJoHt8+/cQx3UBaz4Vws/0uJIahXGK8sMlNzVJXYasAQgoKtneUhTHyoBInGLxRK/tRaRQ4RRoBB9AwsanzdTV79V/ptyIEWRXX2FL7y0WnXDjQkVxZVCy4BgpollpDtSKOwpgKzlIkGMp+nINkFqhk+etYo5yzYEHGXIRklWcTQUtUPdYS6FNNtE3iswBbUyaPLSPeWZtcC7iS5+X+5aSzhXSDVlwjCdJjK39PuxaIr4VfiEzrUxg== 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=3OAK2AdXdJZlR6XHNUOhnZDtZeDxDh0X/pF3rg/WEUk=; b=Rcjfks9QTbjD/9jL7N4o+cBacoYebMiQtxsWL4MWN2yFHIJ13eD7vla3ucm7IlTCfnN6nGMtmtM5HO2lGVqReGUt5kvd5atM7XzaMHDLFVCCZN1PPYCzKnSes1wJJU8QnrP9ized1cHP9UvxqDYB38fTk1YPU13SVm0WU0Ucpwa6meJBSt2RMV9pISx3SUCR87QVdgwuHpqKxGvlcUNuT0a99xnZkFTZUpbcBSGm3l7y68iJqHVoxp3DMcsWODkFXd8UdKDQHRXgpLjd6AgpMgnBJIpez6jrsD9QLqg3UzSneWEzY1AUyNh0CgCyUfNXMgydBFd7bInJ0482PzlY9g== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by AS8P250MB0203.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:37a::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7292.38; Mon, 19 Feb 2024 23:23:26 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::65aa:deb0:a18e:d48d]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::65aa:deb0:a18e:d48d%5]) with mapi id 15.20.7292.033; Mon, 19 Feb 2024 23:23:26 +0000 Message-ID: Date: Tue, 20 Feb 2024 00:25:23 +0100 User-Agent: Mozilla Thunderbird Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: From: Andreas Rheinhardt In-Reply-To: X-TMN: [TIcjdQN8YK41e13gi1wvBStuk7hb4ECPllGlM6l2Xu8=] X-ClientProxiedBy: ZR2P278CA0045.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:47::18) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8P250MB0744:EE_|AS8P250MB0203:EE_ X-MS-Office365-Filtering-Correlation-Id: 183c6e48-65dd-4d63-f757-08dc31a1c609 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: AAAZ9EBGfbXD6Yb93T4t9bdviph4pzKCAdHM40iVxQp6ngLM9o80otOZXunsVr/Xi8ntp0tq4n+qNJp5wDd76IXSmwocBGmuf0rEFeL3FTRNcPamzOVGAZ1gRlwcjFzgFcIOyOHPUZyyO1d88ZXhsh363UBWKAG80KawwJBG7+th3pPF8wqJJqU9zOk+T7L/FImsnn1DBwO3p0YHJfF2GYE+B5FNOmmvrdGl2p1FDeuJ0PhnwODDpXcy7ShNQCN5oEifEETLZ1gJXojjVav7UbEjpX8vxBOH357EIUvL9HmfJJhAEDrc+KoFQkHno6EFovOyZtOPUecmdi7k69z6LNkumk6709TNgVGoup6MkSY5vuMV/m9E5nuCSRzjP5hJVY6FDj8GXKApTqWjhB4Gg9jVeFbucgoXIwkZxBY9MECw85MeXegE2x5GAZrGy6bowXWv5ra4/0Iyxf3WhRM1DbOJdFEvNDBQhZFieivpIT2XdaK6MpEJx7Zuk8lfqpPTaWZR7IMJNrRghKp2P/8My79XJYe/FF/JMXLfAkPcevjhx61C27SQjciQtq89JHjy X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ekpOdGhrQ3NCSXRPZ05xWWRmYjYvVyswVi90R3ppbnhHUzZMTmlJL0drNnQy?= =?utf-8?B?Vkp0Q1N3amJKNjJBUTBsSUg4SHlXYXFaZGhsb0ZhYnR6UFZzb0JBQytKK2ta?= =?utf-8?B?UFU2S25NUDI2RXRYcitydlBXbll2dk42UzQrWjFjTTh4a0srOGFKQ0k0SXc1?= =?utf-8?B?UTJ2T2E4bTRSaVZhUndOTStwaWFFbDlSNjdDS0VOOVg0S29KYVk1cW5rYWY3?= =?utf-8?B?Q3AveEJlTlExbFNwU2lqWk5YQ1IyVlFsR0w1OVJ3cmdMOUxGTVMrb292WnBB?= =?utf-8?B?OUcrQ3RJZERLL0J6ZW0xTkl4L2lORTVYRkVnK0Q3WEdYSlpWaCtSaUhQYjRz?= =?utf-8?B?Si9lRzZiMU9OREp0Z0h6eVdkTmFkVzFTbUJvQWtheXlBQjBucG9XYzdnVUhR?= =?utf-8?B?ajlvUmRqT1JHbzh5U3VhTkhWOFJqWVZ4RXRRNG1tdmRyaWU2bWJwNDZMYUsx?= =?utf-8?B?Z0ZBOFFFMkRPQk1DaEpwL01QQ1pPaDBYSDYzL3NLMnVhQkxCZ0FwYi84RC9B?= =?utf-8?B?NGFqTnYxdjJacTA3Rm5UK3pvM0VGZjVyM0xEMnIzbThRUE0vRjltT25aQ2t0?= =?utf-8?B?L1UyeFVha0VadEVVWXpzMUN2TTBYdlF4UjdmMnY5Y1FFblJ2NFJSMEhWeVdM?= =?utf-8?B?Zm5waXF6blQrVWVsU1JNK3NaUytpTzh4ZlJ6SFV3RFowelREYU8xcTFMWmVP?= =?utf-8?B?aHg4OSt5aHpSeHpCVEJwc0ZJZUZqalp3UkdNNlRRd0VBenZ1aWx5NWtjckFZ?= =?utf-8?B?cEoxWnFIc0N6R0drWCtDTXpRUXFjZjByMmFXMU5HeVF6UnJOK05GaUtPTzlK?= =?utf-8?B?MEFFdmVFcVQ2WUNhOWlId1ZMcnJCbWIvdDJEdEFOQTdvZFowcjJzaGpDWWtj?= =?utf-8?B?SkNidmoySnZGdXZpNTF2bzhsV1hCVEN0M3pURHdPNUlKemdxb0ZCYW1NWmhK?= =?utf-8?B?WFBDVzFPVFJUTUNrU0VwbDRTWmF4S1hPOGJkRVFEb2lpY29NTjBjSHJVNXp1?= =?utf-8?B?QXlvR05CNnk2cE15ZVZ0NXp0UDN4ZkQ2YWJ5OUU3aFE2QmRUSWFNUEtzQjd5?= =?utf-8?B?RlUxVkZ3MEEraGJhNEJHcHk3RVYvbklTdUlSTndsbzdwNUhzVFVybS81QkJF?= =?utf-8?B?OVFOQm5kS2JIdHZ0dHdVU3FzUmcyMjE3S21IQzRSczRRWVZsQXB3TUJXL004?= =?utf-8?B?R1FzUEhpdTdDNWZLb2R6RlV5YndRMDhVZHNVeXJoM3I2UTJGcGovVExEa3R0?= =?utf-8?B?dHVDUUlGUXQ5WkpWK3RZTThGeUlma3hrczdSS1dlZ0RGVjJDbmZzNHVEN0RL?= =?utf-8?B?Zjd1ZmJYemNoSGExcVlRWDk4cmxBaFdab3NIZTRTT2E5UFgxOU1NQWhEOWg2?= =?utf-8?B?akdLNVdCU0lOOWZJd1FMVUxaKzhCbk16STE0dm9DWW5GNTJPZmh2dFRWN2lF?= =?utf-8?B?YWFWK0Vpa1hiVWZTWmRpUHViNG84dnRiVGxsZ2V2OEJuZFFKU2dEN2ZWYXlR?= =?utf-8?B?SFJyb0pUc3ZsbEloSTFJYlo2emJMUlJGUFhvY0RrVkExRDMrUnpzRDIvbXlz?= =?utf-8?B?K2huTGtOM0crQlZMVGxwQlEzNGxnZzVLempPNmx5VytmUnZzV2Nma0loejZN?= =?utf-8?B?S2IvWW1XbjBPREJPUnJBYlVIWGtLS2xPT3NZMnBZTWpJWUx5TXBlYnEyazRO?= =?utf-8?B?YUtoK3dnREp3LzNoaG1jd0JhUnJkak1RUHdaTG1NbFlnZzlkQmEwVHl3PT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 183c6e48-65dd-4d63-f757-08dc31a1c609 X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Feb 2024 23:23:26.3148 (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: AS8P250MB0203 Subject: Re: [FFmpeg-devel] [PATCH 1/7] avcodec/assenc: Use size_t for length of string 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: > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/assenc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/assenc.c b/libavcodec/assenc.c > index e548b9fd3b..e734522729 100644 > --- a/libavcodec/assenc.c > +++ b/libavcodec/assenc.c > @@ -45,7 +45,7 @@ static int ass_encode_frame(AVCodecContext *avctx, > unsigned char *buf, int bufsize, > const AVSubtitle *sub) > { > - int len; > + size_t len; > > if (sub->num_rects != 1) { > av_log(avctx, AV_LOG_ERROR, "Only one rect per AVSubtitle is supported in ASS.\n"); > @@ -59,7 +59,7 @@ static int ass_encode_frame(AVCodecContext *avctx, > > len = av_strlcpy(buf, sub->rects[0]->ass, bufsize); > > - if (len > bufsize - 1) { > + if (len >= bufsize) { > av_log(avctx, AV_LOG_ERROR, "Buffer too small for ASS event.\n"); > return AVERROR_BUFFER_TOO_SMALL; > } 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".