From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ffmpeg-devel-bounces@ffmpeg.org>
Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100])
	by master.gitmailbox.com (Postfix) with ESMTPS id 04F7D4D0C5
	for <ffmpegdev@gitmailbox.com>; Wed, 16 Apr 2025 02:19:01 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id BE424687D7C;
	Wed, 16 Apr 2025 05:18:56 +0300 (EEST)
Received: from NAM12-BN8-obe.outbound.protection.outlook.com
 (mail-bn8nam12olkn2090.outbound.protection.outlook.com [40.92.21.90])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 29D43687CEA
 for <ffmpeg-devel@ffmpeg.org>; Wed, 16 Apr 2025 05:18:49 +0300 (EEST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none;
 b=LcjKHoo0LkpcHUcPPVlWveB2Nml/9lDBlK/c7L0u/EJn3SaMCy7CxQek0L+5A/egCOl7K8pwXvquCQNzU8NtgIAp7w53D2rWJvHzd+Nk/oOwNw9CGC5XBI11muuvRkmFArEw6WCfOSgNXbqHhQNjQl6t+UfKWj/jaPBuQp/0QP8uxh4ZT/FbbNCY5zfl+rE6ToO4hd1u8kg49sn3wl3o6laxV050Es8HsxgSBeuzDG81FEeBsFEfXXPX0BxCSl5s9tltJxS4ibSMTH9xDJBGZtSkGK5ayTBZeicPPWCXE1AYedxl2OKeetp/yLSBE+BcxezWiMOxVhm8yibgZQsxWw==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; 
 s=arcselector10001;
 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=zZtiFnKT5WrjITj307c1yn+geRJVxlUgSFZf+I3L9RI=;
 b=vz8fJW2MVa94gL/7LT5y25bSWB3FGvnZpj+pSb1dRJO03M/IBx2FxjQpVUway+NPi29FXt6myZft1tOFj7uZnN9mwnjg6EPODWgM8SifBrMwVH/vTed4/WtxE1XG/2Ub8FA8kxqGzOPmrbVh+HrccVOD6QQqiscBoJSiwxaY4njEdn0682I+cXtye3P+Vq23DWHxFDlxhZATQltSsiRBQrXMf74pas5550vWLaVy8CkCxLqQGmoUCgIuer0820l/qrdG5ZAdnGhryP23xNmtRC8T8wACGhixOx6C+5+5+y/gp8M5FidSo/XmeSNt3d7a4aWoxN6qwz4CZNDkwU9LxQ==
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=hotmail.com;
 s=selector1;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
 bh=zZtiFnKT5WrjITj307c1yn+geRJVxlUgSFZf+I3L9RI=;
 b=jUXXNADOjBjNCWCbQNHo3SbmN1DWHy5u8h0kjF5C2pNc7uNwaskVoEI90W2mNx3Tdb2RTG5xc7PsjFZSmdeRP35k6lvN0nEfjM2mHaASSnw15rJRu686wsEqolWezRdAOPDS2Wd7AZXGUQAI0SgxEx3xb80Ma+Wxzsqy038w31Av30yBt0kc4sCO4JN7tSTw+hWPcHUobcxhoUUaEaxV3NAcr5w0WVPbc9KGQqFedco6JAVU4IHQnMNacpU4LtZBYRVDHExzijLNF7IXY51TjmJXV2X36zo/iSlnoshPWIDkzIigUT0cuPL7JbiE1U9RxJVA/TPLD8A6dyw4MHs6Wg==
Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by
 IA2P223MB1200.NAMP223.PROD.OUTLOOK.COM (2603:10b6:208:4b2::12) with Microsoft
 SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id
 15.20.8632.35; Wed, 16 Apr 2025 02:18:45 +0000
Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM
 ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM
 ([fe80::bf09:8e9:b07f:98a7%4]) with mapi id 15.20.8632.030; Wed, 16 Apr 2025
 02:18:45 +0000
From: "softworkz ." <softworkz-at-hotmail.com@ffmpeg.org>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Thread-Topic: [FFmpeg-devel] [PATCH 2/2] avformat/id3v2: Check that
 decode_str() did advance
Thread-Index: AQHbqzD6b8QhS9IB3U2xSLilkCUpvrOfQsxwgASOQgCAAAZwAIABQhQAgAAFo+CAADvZgIAAAb7QgAAop4CAAACN0IAAAsaAgAAJGiA=
Date: Wed, 16 Apr 2025 02:18:45 +0000
Message-ID: <DM8P223MB03657754E3AD87730AB74AEFBABD2@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
References: <20250411222719.2779176-2-michael@niedermayer.cc>
 <DM8P223MB0365A90C10FBC9EF6512F9E1BAB12@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
 <20250414231950.GN4991@pb2>
 <DM8P223MB0365928500AB53069869DC99BAB32@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
 <20250415185538.GR4991@pb2>
 <DM8P223MB0365E10D38EF9174A32557DFBAB22@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
 <20250415225001.GW4991@pb2>
 <DM8P223MB0365EFE83DE448577271D852BAB22@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
 <20250416012145.GE4991@pb2>
 <DM8P223MB0365836071D915E428907879BABD2@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
 <20250416013339.GF4991@pb2>
In-Reply-To: <20250416013339.GF4991@pb2>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: DM8P223MB0365:EE_|IA2P223MB1200:EE_
x-ms-office365-filtering-correlation-id: 977f0f2c-9203-413a-b42a-08dd7c8d03f4
x-microsoft-antispam: BCL:0;
 ARA:14566002|8062599003|8060799006|19110799003|461199028|15080799006|7092599003|3412199025|440099028|12091999003|102099032|56899033;
x-microsoft-antispam-message-info: =?us-ascii?Q?Xb6kvzHd+UUjAGNWAjkl4GT3xj+54pdfBjK3+QMP68A6yv2WoGy9Cdr1UrqA?=
 =?us-ascii?Q?kScZkhrlRVlAnM/EMO1gw6ZZFfjIXeW1w3Wi4umwXCphFNKZj5G6QiLABnsU?=
 =?us-ascii?Q?/5+sqTj0B7shMDijv8lY18a4ljxMew/GAZ3Acyy2SZTteZRyEx7700xj58HL?=
 =?us-ascii?Q?Knc533Hx5ctsmlsaK9DPQEkJCEamNmE1Lj0dHWNGsmJl77jNTpR7IZPTNYkr?=
 =?us-ascii?Q?eVnjn8RymXuo4q2YVzABqXcp9odWbb4xkxwoEMyLVn9l2x5zA7A4l+1nB/5m?=
 =?us-ascii?Q?XCRk83W4eMAnwziw+GgUrKAtjPnvr95tRmIfRr5TuGiNAeNvlh/0PQtoqov5?=
 =?us-ascii?Q?uUj4gmVIPk+r89MfZMpBuAGlAvXjkIR25UtuZtdkHxpuE51UkdRuXnkPiNfk?=
 =?us-ascii?Q?RhdkyzQrd3FzdwYHgie4sz0oAmZX8mKKNtVPTUf02qntQuQThIDpdIzlX2U6?=
 =?us-ascii?Q?tTgKUauWDv+py/jg6Qf3Lm9+98bnmUcnYgmKKBr2FjwofRoGT9bsauEvT2co?=
 =?us-ascii?Q?uMDraU6NDf6ObXBlGqDlYy2D3puTqvISz2Q6JthoxtRbSDzaQ4eJ5uDR7s3N?=
 =?us-ascii?Q?Gi6C2ZZexJ7WYrxjz5kF2y+/5ImlIE+9jVZBp/pWTjWTVt4IZ74by9Q89SAA?=
 =?us-ascii?Q?204ETqwxZW6wJBlxjMypNMp6J6iwgKHAEUWQSgyJI8pCaNktR5YdnlS+0lJb?=
 =?us-ascii?Q?JBqPnYWA4ihv3KpNtpOxME2SjPx0rOL/3tzF7so1vx8Oz1B+mkjJvo1PqGsa?=
 =?us-ascii?Q?FOKf273Nd4a/wFhpBak/O7UtL963pKv+s/SzDS6FvUiivVJ2XT1H6F0igHPt?=
 =?us-ascii?Q?yI/x1rUfQK6SUSf/6AfXfhARMcBs7022UkRPQieNlwbHUxfjqdyB8Vsg66H8?=
 =?us-ascii?Q?02FZNg30UQJcq9sMKVZuV13CWqV4qqLNfYj5q/K0IyxA9y2UnIrkHTsc1hsp?=
 =?us-ascii?Q?1xazi6bmF3f9Eq4T8MGAH/4ZzurbN19RhDY6G8c2QPVhngQm/0H1y38+RagC?=
 =?us-ascii?Q?O6XiLEuw4qyLRQituVflQfbjD3pL6yaycQDtQWcjiGpoKocoWTbML8W5FK1I?=
 =?us-ascii?Q?TNhDThTY2CfwJU9Mu8nxSfkvDQ0wm2Dwjh7gljyPzrRCREI6pgeZlZUJZ30h?=
 =?us-ascii?Q?ftElYty3BJ79zqJlJTLebrGvM9ha+eCiyQ=3D=3D?=
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?sW8z8oo5byX3GFDFtyi5CiuFbDr/EAbLEwL65csYw5j4UHeFNRSSgjAeXF5K?=
 =?us-ascii?Q?XGWiwOcnl/skWR0DKN2QEK1pQ2w/u5tuqJsz022F89KlKk7cX60TxFhhq63T?=
 =?us-ascii?Q?fUGIjaFFTldOgnZsY+uKy3HsGb3ziBEC9Ri7gT7ifd7RfqGyITNJzE/LGIon?=
 =?us-ascii?Q?ycg2lpS53KsjSLU0e6Yoi9XynX1FUivSBXoEPB6VErAr+PkyMCJZ3jAFo568?=
 =?us-ascii?Q?sWjGxmHVtd/R8QP2diQ40dCaePBIZ2CjQEtWFr8yUoBKd67Jq6KEIlvYghDU?=
 =?us-ascii?Q?rURzpp3KfCnxqLvB3BzKqkpWbKggJsIX95LTPwCP4iSGxhrIIwr1whyk/9yl?=
 =?us-ascii?Q?n6NqXcNJaRN6zo/7yn3n7DyrwFEQagDjFgTN5PoIV2+n1Hr/xHAIRZbzWR3p?=
 =?us-ascii?Q?9A/UmjjvIyDmbGUc/Ff3rNJfAe45Xc0jaX00iowRZkUi0uN7S1MYuQ5JCUcM?=
 =?us-ascii?Q?YkF7T0waVs8dNLJOBV461b924ioi04kOX/r+3JZTs8RYoRjWu06P4BwFj3d1?=
 =?us-ascii?Q?madQILn2UYYQnI1haV0IrIsMvMN2mE2lomHxlyCEi12SJpCpt8ov29IdzMMw?=
 =?us-ascii?Q?ENIgCZHaXm2kI7MohCTAH9y1DPCfSXbMz7J2DqVpLhHKwLCRE/1DKnrI/AYC?=
 =?us-ascii?Q?82BjeC+stvF/p/0uEqHMIp7uefT06+0IUGDTzco0d+ZGb61V0oSguAQRdcnu?=
 =?us-ascii?Q?d1IVnUbu+RERljt+oy7ZwGkQTmtDkb7pwOWh0Fa3MnjugCpYYe6PNa2eJ/jJ?=
 =?us-ascii?Q?w4PHWSvtIK/iK8N0z10sme9Hmy5TkblYXhC9iw8sYnyBUqfAFpA8ItpaWybP?=
 =?us-ascii?Q?vpXuspEvqJLOKNQqaKqVv7La68sqUdMQxRNy3SUv0JqTZot1UorWTdSAJ+4P?=
 =?us-ascii?Q?Y1XQQsZmmcdbIXKsd1HIWSIdkfyA+CdwcUiYgKp8/nH0H9cFb1waeiJDmKuK?=
 =?us-ascii?Q?D4NcVMLFYRSsspVB9LC502eS81gTy5oc+Nt1rEpAkLaWqPLCDutDEemd3sGq?=
 =?us-ascii?Q?fEGdqrDxt2/oxtFA9q0J2QTu29nnTLVtVYmvu8DzFREFFDgAyht9+HQDxjnp?=
 =?us-ascii?Q?EBzmXFPcn0XGKoortFlK1HkAwAHu14LQiPRr2HWDPFDPxlDM3H7VtZXO2CYJ?=
 =?us-ascii?Q?ei7F/uW9t+S7kXEmBzNvqfnL2qslThQU88CwvVKQbszxuq4wIHW7nurymVc9?=
 =?us-ascii?Q?eGrebX2oTZXkN8hpcuSVIWKrdeUMYcftubqDHT1CX7zdUiKvyMMckFaf2s8?=
 =?us-ascii?Q?=3D?=
MIME-Version: 1.0
X-OriginatorOrg: sct-15-20-7719-20-msonline-outlook-92255.templateTenant
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-CrossTenant-Network-Message-Id: 977f0f2c-9203-413a-b42a-08dd7c8d03f4
X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Apr 2025 02:18:45.4622 (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: IA2P223MB1200
Subject: Re: [FFmpeg-devel] [PATCH 2/2] avformat/id3v2: Check that
 decode_str() did advance
X-BeenThere: ffmpeg-devel@ffmpeg.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org>
List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe>
List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel>
List-Post: <mailto:ffmpeg-devel@ffmpeg.org>
List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help>
List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe>
Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: ffmpeg-devel-bounces@ffmpeg.org
Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org>
Archived-At: <https://master.gitmailbox.com/ffmpegdev/DM8P223MB03657754E3AD87730AB74AEFBABD2@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>



> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> Michael Niedermayer
> Sent: Mittwoch, 16. April 2025 03:34
> To: FFmpeg development discussions and patches <ffmpeg-
> devel@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 2/2] avformat/id3v2: Check that
> decode_str() did advance
> 
> On Wed, Apr 16, 2025 at 01:29:02AM +0000, softworkz . wrote:
> [...]
> > > > This will cause deserialization errors for many people in the
> world
> > > > who are processing FFprobe data.
> > >
> > > As said, ffprobe should not produce troublesome output
> >
> > As I said, it cannot be remedied on the FFprobe side without making
> a
> 
> If you want ffprobe to combine multiple author tags with ";", you
> certainly can do that in ffprobe

That's not what I want to do, and if at all, it would be up to do
for the one who wants to introduce multi-key metadata.

This has a far wider scope and cannot be introduced in a drive-by
way via a bugfix patch.

I have already shown how the additional values get lost when transcoding
and that was just about ID3 tag writing. 

What about other formats (muxers)?

Can they deal with multi-key metadata?

Does it create regressions? 
(yet untested by FATE)


What I want to say is that this is a big change that needs discussion,
planning and evaluation of the consequences in all affected areas.
I can't be introduced by a bugfix which causes regressions and others
having to clean up the mess afterwards.

Thanks
sw










_______________________________________________
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".