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 BB62742DA3 for ; Wed, 8 Jun 2022 06:39:33 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id A202968B609; Wed, 8 Jun 2022 09:39:30 +0300 (EEST) Received: from NAM12-DM6-obe.outbound.protection.outlook.com (mail-dm6nam12olkn2062.outbound.protection.outlook.com [40.92.22.62]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 46DCB68B333 for ; Wed, 8 Jun 2022 09:39:24 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FYkAco9/Ze85wTiSX1KYPF7FefdoY3imZwBV2+Ld+UfISLxFQ9/8rxrq3ZZZmQZFywmQ7rjVzu/TTVrWCFJ6T8F9eixNPWXqx1/mKftdwsznI+yURSYAdKtUwVlc1uIy2DxJQQI/iCUVk18TQ22ly6NiHNy4ubuC56FP4O+gWCmi17YazAHXSIGlrCs3T8BwdwGgphEugYyureoe7qjKSvlRbqDz7M2BHzqD7v14Lby+Hsb4+a9ptzZ/rylGj/GeKKrPUg1Slm0PWNd+asi/ij+yzRhLdDcFGOxeSseQ2qG4RBvVTzaDhNDSpifpn4hiCo4HAiE4EmLsHq8K6EcAwg== 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=1vrVccnM24WnSwB2k/nAEUj6ayG8AZ/Nj/cDROEhZ5I=; b=Nq43PDrkjDi/i0l5xvXTo2Z3B6yOKmK3+dW11YUrKHhcZM0cuUIC3hKtFX6HntttQ0tqhSgahJz82pSYXKwFSAECGy5jK5yn/fxI/sjEfvPsBg+T6sH1+M4mUf7d/S8dhaZ+fgICjfcyChehBZ8yHp0jkSrWPIvt91xNsWRm5uHyPSaRSNyhgUhZrTAH75kNlwH/KJr2FBohbbr7FOyJvshSzXOs0X3IvGQgu2RBkEw8YKqrGWn5FhKgY2FeX3KbKsKp8/mQJ/+gnymNsWDhQQ4oN1VuPU1/41Yo+q2+fhvnPNUTBN3LlThiip65hSxrnSmyhOUIu6RxDiud12SgJw== 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=1vrVccnM24WnSwB2k/nAEUj6ayG8AZ/Nj/cDROEhZ5I=; b=pb3hFPTChzAgXL+TQQn4X5WzC1nBAQO9AFm4NvSb3MXUBPdaDpKFYwhjIodcqgJHuJXn3X1BeWdvrbcYmDELK6zD8bNJWCUMFcNt7GuNj8ysCR8tbpNLsMsrtczoFJitEnmFr4fXVlUfxVzudJgxVv72KrcojqhF6qoxa4aOXHCZ+RpMDRmPhYxCdTznezxXRLlOv//FZQQEbbnLAaOJtc1/ZABEWSpdgV71ncUzlNQEAeDPdLmw0+jCmFe+stgwxU5E8jcocGTOmEmDMZtgOiEj0Gfs4QRmklh5p9q4qLpKkgEuDsMpu5XXzI2bwosDXsgg1+R1Jajd6A2QNFzwrw== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by DM4P223MB0472.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:ac::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5314.19; Wed, 8 Jun 2022 06:39:21 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::f144:21f9:cdb1:5c8a]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::f144:21f9:cdb1:5c8a%7]) with mapi id 15.20.5314.019; Wed, 8 Jun 2022 06:39:21 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH] lavf/matroskadec: stop mapping text/plain attachments to AV_CODEC_ID_TEXT Thread-Index: AQHYemX/APXFdmsBaU2+kzHdFR1g661FChGFgAAAclA= Date: Wed, 8 Jun 2022 06:39:21 +0000 Message-ID: References: <20220607115839.18515-1-anton@khirnov.net> =?utf-8?q?=3CDM8P223M?= =?utf-8?q?B03650454F09DE4D7C396CFB4BAA49=40DM8P223MB0365=2ENAMP223=2EPROD?= =?utf-8?q?=2EOUTLOOK=2ECOM=3E_=3CDM8P223MB0365397786EAF497BA5DC577BAA49=40D?= =?utf-8?q?M8P223MB0365=2ENAMP223=2EPROD=2EOUTLOOK=2ECOM=3E?= <165466902879.5088.14253593912707356675@lain.khirnov.net> In-Reply-To: <165466902879.5088.14253593912707356675@lain.khirnov.net> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [eyfXTzwtzY5l1mq+nHLSteQ0M0qmRomO] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 90f98ca8-787d-4e7f-48e0-08da49199eae x-ms-traffictypediagnostic: DM4P223MB0472:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 2gwvYctzXj7mPrF5pJL56aDpj/iKLUHX/hZp1hSSbHf8fa/VnyIyT4prrPT7olnpgY9uoy4jnfX1tTsL2aWQt9u4ttpQlJZwS4rz0UO/WfP7Qrn3/le9iZshrnO/Zpyjvh8T7Jqi4QvNheS6qm8kZNolnzByE0R64x5LLavKyzrmHBlUBAIGARLVWNhZot++AuKpKHVPY/2ZP+rsGinmQNHkxQWXckS9woNMscufewgRh9uIC6mIxt4+8ZFxPVzAt2jwgshTqbKrGgTHKrh/kU3BNNdjlb9lRtZq5bY1jAb8pM8CXWjc11E9L4w7P5Zle3ExXwaLTfNdYLVietDzuZLR3UTl6Cu6WP+aUMeHRxrTvH653QwKd8R3KLfQtw7AW8kzToJqbSEhLP1L6oimdRQuvCGf4TOSolixHOGvFDQTf0Z4wxp0yk9ueQYeowKgbgNNHPRoFxUMVp2U1zcSo76Z6EbsxhklbqFsCY6kjGxqLF2CxmTpD2YJ/phAk51vI1HJ8WWQW6ZMoz4+18A5JroC3+0NnT7PfHgI52rXuskKaWjgcs6yBwRmrHtCaN6sBWVVZbuTVfRHCu/Il2yj7g== x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?eWx4N3AvTlNSZm1ocXBDZkJrTUhJZkNtekZPYWFUYnRoaFhDQ2gvM2NPSEIw?= =?utf-8?B?R0N4WS9ZbEsvM1pndXRzeGxhR0hJZS84WEtKdVRReXBzcnpQYXZDWFdyLzla?= =?utf-8?B?QlRpWWRiTmFUajRKbEQ0NUFYaG9rcmhOYS91bCtYd2tQcjFZRlV0UmlkSDY1?= =?utf-8?B?UEdnYkR0RDRsR1N4bisxNjdaNzR1RlNMeFdSaEtrREFqTHVsUzhyYmpQTWlD?= =?utf-8?B?alE2eWdtQlNTWFhScE44QnozeGxPNjNkVXROazM5empsa1J2Wm1XbDBqZTkz?= =?utf-8?B?aXFPUnNXSUhDSWF4amhIQ0oxbWFFNEtPQUtTL0VtNm1sMUJXYi9aZ2dyNDlB?= =?utf-8?B?dkQ0M05uMXJOcE0zOFBrbzhhNEViL3hWT0huYzFkN0VXSXlxcHIyYk4xeUJ3?= =?utf-8?B?NDFjK3RkUTBqQUZ2eGxKMHkvR1JKbjNvanJZMFJhN1JmLzVoaWZqc2htMXNa?= =?utf-8?B?WFo5ckhrbEFITG1oMkhjOHhxcVdaQUNFbDdsaVpVMmpGR2RVMkxidTViM1ZC?= =?utf-8?B?bUt0RGJmSGxjYzYwZk1RcVJXVEVOa2xwbmV6Nklia1BUWW4rTnNTZlhMa0xl?= =?utf-8?B?a1RkQjBkRWsxM3BFampzQWZsMXBINFM5U3FaTkdLM1FuQ0xqNUh6Y21aWjN6?= =?utf-8?B?b29ZcERuaWdQZEZaRnRPRFVydzFKaWZWRkZCbzd1bnVCN012YUR3TGRDYWVo?= =?utf-8?B?ZDFOZUFGUGRnRzd1SG94b1BMRWtaMWgyUStyR1J6NTJ2Ri82elI4VXhES0VG?= =?utf-8?B?bk5GMzBKdU9XanRWWmV5aCtlUnVHS1hUYlR4d3pTL0t2bVk3NU9keTBWWHhW?= =?utf-8?B?SmtxYjRlL2FvS0F1QlkyTDZITkFyUG9kQmNZSXpMWEVLM3Q3ZVA2TmZuYXdG?= =?utf-8?B?ZjVyOWc1dlRzenFnNG5aQjVPN1NLZ1BrbVUzRC9sLy82SFFQMnR3bjhJcGFR?= =?utf-8?B?QmNoMW5rYjM0MGZKSWllMDluYVBLdWFRQkNMQ0Z3bzYwNk9aRGcyNXl3MmVR?= =?utf-8?B?VExEYzhtM1NZSy9DSWRwSDV2WVZqTVlUb3d6YU54U2RwSlc5THdxYzAyR0tL?= =?utf-8?B?bXJ0dTUySzVBSmxtd2pMTVhFeENOM1YxR1U0NkRlRmhlek9IRXpMSFNMbkQ2?= =?utf-8?B?YThOVzlzWnp0YVBuWHNJbkgrS1B6bHdGZWJEMUd6dm9GaEhVeVJmamVTQ2l1?= =?utf-8?B?QnJPTmFVcXB1SmRYSVlPckE4TEtnVEVaUG1nelpmcGMwaDJDQmtiQUtQNEtU?= =?utf-8?B?Q3piUFRFQTV0U1lMdDdOd1NORmVRUTMrVGpIbStaSlE1ZTRMa0dCcUIzSXNk?= =?utf-8?B?QU4xV3h6WCtGNFU2UWxsZnNqQmg2ZTM2UWhITkFqTStjMDNlZDExOGNsTXZZ?= =?utf-8?B?ZSszeElSWkkxR0wvMTJwK0dia21VZDRmeW02blZkaFRCTjIxV1ozUmx0dkow?= =?utf-8?B?eDZGQ2FPR2Nveld4TXNJTzdReEpGeVpZV3RoK2l2YXhGNkVUcExxejhnMjhG?= =?utf-8?B?NkM1ai9EWFlJTVJGczZFWTIxNFV0OGxHWGZXbTBLWUhDbWdabk1rK2lmWDRF?= =?utf-8?B?YXlzVm5WVm10OEdqcDRaUVpxdzM3d3ZaYlFvKzBiQWNFdWdQUzAzeWNYcnNx?= =?utf-8?B?RW9rQ1IyRmtRT0x1UmpSUk9CRTNtb3AwUGpWR0o5N1RYWm9Id20xQy9rWHpG?= =?utf-8?B?elljcXp4Vkp1ZUthRlFITW9ycHp0MzZFa3NtZWtEMXE4RW5oZkR0cWRyckh4?= =?utf-8?B?dzM2bGFwVWlaT0hSYkZZZlhZL0tEUXpEWkExaGFmYkVZenIzVU5DdlpxbU9R?= =?utf-8?B?T2NVdkU5eUcrOG8vR2R2bW5hNFlJblVPMk5sRHRaT1RBMndqTmk2NUN6dHRS?= =?utf-8?B?Q2dSVGo5bFpNUEkzVXByNklXTlF4VFBYczdaT1JUSW1UZlR1cEpRUEhZblI3?= =?utf-8?Q?jaJvJkE4yhA=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-1ff67.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: 90f98ca8-787d-4e7f-48e0-08da49199eae X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Jun 2022 06:39:21.1265 (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: DM4P223MB0472 Subject: Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: stop mapping text/plain attachments to AV_CODEC_ID_TEXT 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: > -----Original Message----- > From: ffmpeg-devel On Behalf Of Anton > Khirnov > Sent: Wednesday, June 8, 2022 8:17 AM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: stop mapping text/plain > attachments to AV_CODEC_ID_TEXT > > Quoting Soft Works (2022-06-08 05:45:38) > > You might allow me the question whether we can be sure that > > this is the only case which is subject to the regression? > > > > Besides from what I reported above (and you might probably come up > > with a new codec ID for discriminating text subs vs. text > > attachments), this surely fixes the specific case I reported, > > but I wonder whether other cases could exist? > > (it's meant to be a normal question - I just don't know) > > We can never be truly sure of anything except our own existence. > > As far as I can tell, only two bits of code in lavf can export > AVMEDIA_TYPE_ATTACHMENT: apetag and matroskadec. apetag does not set > codec id at all, while matroskadec will now only export codec ids that > do not (and most likely will not) have decoders: fonts and generic > binary data. > > So after this patch, to the best of my knowledge, there should never be > a case where an AVMEDIA_TYPE_ATTACHMENT stream has a decodable codec id. > Then again this does not exclude all possible cases of a mismatch > between a stream's codec type and id. OK, that sounds fine. > Overall I'd say this just strengthens the case for my original lavc > commit, since it is clearly helpful in exposing bugs in other code. As said already, I never doubted the validity of your patch, it was about the effect and unacknowledged responsibilities. What do you want to do with the text attachment "none" caption? Maybe a separate "cummy" codec id? > > Here's another thought that might be worth consideration: > > What turned this from a minor into a major issue (from my pov), > > is that it is causing ffprobe to fail and exit with error > > and incomplete output. > > What I'm wondering about in this context, is whether it > > even has to be like that? > > > > I mean, an unknown codec doesn't cause ffprobe to error-exit, > > does it need to do so when avcodec_open2() returns error? > > > > I would find that behavior ok and consistent when the same > > would happen when running ffmpeg (ffprobe fails <=> ffmpeg fails). > > But ffmpeg doesn't fail (unless you use the stream), so does > > ffprobe even need to fail in these cases? > > I suppose it can make sense to log an error and continue when opening > the codec fails. This could be useful also for probing genuinely broken > streams where e.g. extradata parsing fails. > > There could also be an option like ffmpeg's -xerror that would make > ffprobe exit on failure. Sounds good to me, but I'm not sure whether everybody would be ok doing it exactly like this, as somebody might argue they would rely on ffprobe failing in such cases. I can submit a patch for that - unless no objections or better ideas would appear.. 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".