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 A569F436C6 for ; Tue, 21 Jun 2022 21:46:03 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id D422368AF3A; Wed, 22 Jun 2022 00:46:00 +0300 (EEST) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05olkn2049.outbound.protection.outlook.com [40.92.91.49]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id B617468AF3A for ; Wed, 22 Jun 2022 00:45:53 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZbsoexUHBFvxekmM27vBXXdz7KOmpIP0uIeQQwQ3baOSxvjAaOXUc7QWyi4hrBDn1gFTT1/qL2ovU96XrChlDK48ijKaGtmusjgA3UFUvJ+Q8ORdC6Cso5ir6CB49VovRT0/kPvvow208Mzc8JaNvGERIqoVFhtoSRHH1RUGK6kcrZjV/4D8SIStX5e6zWGUoQW/iNl2axVS9yqqrkkHgQ6bSdpqJfHkMTtQv0n4P08A5ERRveaCqu3IERvd7R4u0IkSXhbFrgW8xdeXelaAaDHoBwjUQ41whoDpcaSFQNqT6VknmbRMtFJyN0uKnqcUXEdJw5mszBU64tmYLfS31Q== 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=5n9oJdYcwmDqiEi7L7EmcrDKNkUAZKjhzO0wu4V9CRM=; b=eB7eZ6MQ3rxUJQDn9HF4TCjYv6qQC1/qUrio42UVPjthrf7UGMsfhMHiZVKZerY5akIzoY7sygAm/qTUbsyfaIUu9IZ0CO2wUQyViX861ZLzmMFG1PC/d5nSBv8klrucdWheWjAOxnaycPQPxyWiZGvI3a3NkRYHco5IXdaBScoqsnipx4eZoFMZMVC33Ep5zMY6G45wf7zAoTxZ6z3rZYkFQWA5SKMBk1lCPvqZ77SlTVtXjv1ikRxu4gGo2ubIHzvx/Iltyu4wksjA/u01tTV4/gtExQfSU2rlNjmt4R5r3GHxAU9N/Sz7o4qoxoNbAOszH2rl5Hye8cE2gR747g== 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=5n9oJdYcwmDqiEi7L7EmcrDKNkUAZKjhzO0wu4V9CRM=; b=bzNevxZ3PKsEQ8bAUC643yjLdYocdB/a+ZZg2lnWBt7Wd7U+Hxj+X5crAge10zJJSEOcIhnGXLfCNKSWKUHuwZXhkfTCbkaiQ2LZODxzn/2EyJDgaT6PKoFyOjl0+hS+qLuwL5TCnDsV6C2YpQiJQnr4j4bYahZjlZdz+r96YxrflM4+ChlKtyIKH/ueMkACzWZH4RVZIXmZDh///6i9+3xNwA1Bm6DEfwDY6YQutPRnMlXi0Mg4kfvxes0+FJaG+aEHl9UFrf5b/Lg/Mnb1QvGlNzyztSYWjV3U/QE+YBCpjq4F2r5Axtv+mWFbCxuacVHSeNqCwVRzc/yX03FrwQ== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by AS8PR01MB7781.eurprd01.prod.exchangelabs.com (2603:10a6:20b:2a0::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5353.16; Tue, 21 Jun 2022 21:45:52 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c%10]) with mapi id 15.20.5353.022; Tue, 21 Jun 2022 21:45:46 +0000 Message-ID: Date: Tue, 21 Jun 2022 23:45:43 +0200 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <20220621152248.GH396728@pb2> From: Andreas Rheinhardt In-Reply-To: <20220621152248.GH396728@pb2> X-TMN: [sgNJh+iT0FFcQIOJTJPPuBH9I3AvOhAZ] X-ClientProxiedBy: ZR0P278CA0026.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:1c::13) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: b2ebf44b-f7a2-408a-8029-08da53cf657c X-MS-TrafficTypeDiagnostic: AS8PR01MB7781:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: RNSRkUbI9PWaW7qG4CuwzfSkHIL6e78cSjhRiiCqJvxWr+4GbGJF7gmjN3ovgasSEFpthflJK4OtpwLg4OVj1fAaPx/WXkjj0L+Qr2sD9iBS42m82demL/vViTn3jZm/0HSqJiUAbpyFgTzYK3oN+turTLfFKAED33iLimUR8BcgE8XtPNn51Se0AcfpKhL9NnvLf7WqTcx53ONn1Z/9ge9uEI3F8bkQjlrCBAfOhu/n45gKruMaHiC7tONpwDJYZs/beCuUv5PSkF89uws0Imiux73dJSefGmkfS32J9fMXowiIbTUWN4emtxJsw9YwhQN1IOOcagEGNkF79DAoETVemXmTWfjZF/vCERKb4L10J4rbLsJ5jS4FVKCZ/XKOCIaTMFCJXXqIH4OyJ3YHSsqSN7gtX2teSC8u4DlwcmMAYiDvMd4LFbBai/HvSqg+U+YqVSJHgng4IcdWvcCb/XqMNSF/KZBMctefAC7PrC4WqzdcSMD7vGa3kc62vDEnzTaYgevZhU+h1q1iWiyokvai3JO3416rrqnFbPv5SKQBiiHe0QRzVoFBdg1jrtP1XuDTueQpDYzkpqELSuGxaA== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RFhjbnluOGlYY1hkRFVyZE8xNCsvcjYrWmY1Q1dwNjNhSm5tQTB4bFZObUEx?= =?utf-8?B?bjE1R1kvTlJlaFFOeDRTZlJMS3NrbGZwQUhKOWcyaktLMWFwQW1ZS05sZ3BK?= =?utf-8?B?NlhaZGVsWk1qN0ZqNjl5b2ErZ2VKRnk0dy9FVUUrTkxWeWsvemI0dG9FN3lo?= =?utf-8?B?RlZNV2E3bXdpQ1ZSNDE4WDF3NVFkenJlc3lSRWN0Z0xqa2sxZVMweTZJNDVr?= =?utf-8?B?d3JLNVI1QktLdzNwTU9qNndvTjJNTzI1aHVNb3RPbCt6TkNyNEZCZ2FsdW9V?= =?utf-8?B?OWtmZ3F2UWREVGdIaTU4MG5oZFpVRWVVZnpNRHdhZFJabFZjYjVQL0NFUG0r?= =?utf-8?B?OUFSZVBUZGFGNmJ6NGxOeENtaFdwUTZEOGZpNllsc1ExSDJHTnU2NlBWWGE1?= =?utf-8?B?QUtOekY3bmRLRmFSR2RHZVByV1RqOGN3czdScHFqdUFoNlQ0TFlWUmt0VEhy?= =?utf-8?B?Nk9xYUs5bWF6VWtIWndJcEFCbHRzS282cXh5TURZNUVzWEVmdmMrZEpjL3Yx?= =?utf-8?B?UkZvc2M1Q25HR2dCRXptTVdJNGFSazE2YnRDemhVN0pSUEdGOFhxRGtQL1h6?= =?utf-8?B?OUY0QlFkd09SUzJrR3JrVFB4OElBajVOVkk0MWlQR1lyWTRIQW5NdCtOZVcv?= =?utf-8?B?aHgxN0ZIcG9iMzRHTlBzdm9IcWxsMDNLdDhseU9lbzlocDJDeU9UQitsOTFN?= =?utf-8?B?U0EybW1QN3JuUkJwYjZYUS9EUDNkWW50OTRMZ1FtUFZTQzRJaitaSnlTcTBa?= =?utf-8?B?clNCM3hLSGJDOWpFZmxZUmZCWGx1QzFpTUg2UnJISThIeE9vanFqU2ZZYVdp?= =?utf-8?B?djhHaDFOci9zUjI0OWpMendHUmJxMFBkY1gyNnR6VVFVMWQ4QlR2Sk1WN0hu?= =?utf-8?B?OFpOeVd2SnJjcUV5aFd3YW1MREF2OGE4M2lhNGJXQWxtTThLcXF0ODlxVU1H?= =?utf-8?B?b2psMk5EdWM4VGh5bmVubW5HZmQ4ek9Eall1UWFPV3pMMXVlOVFYNkpKZGpJ?= =?utf-8?B?OWZibUNWQTRJTUFMZFU2Q01TWExzZXlvZm9HMFI5cHcrbkVNeThrVXoybHla?= =?utf-8?B?ZGd3NFpSeGZ5VExBMUN5WmZadlBpa2h3cnFnbkFnUytEZ1JoWld3TUpnOHdy?= =?utf-8?B?TjdDclV5UGJEdlljOVNaekI0RkxsV2kyM01QYllRL05ZNWZMLytXSlU4VnQv?= =?utf-8?B?LzNwcjJQS1gyR2g0M3U4YytDWTYzRGN4Rmhjb1BFcUpOY3RhUXpqRWZYWjlx?= =?utf-8?B?bHFpaGg4RGlUTVp1M2kxemNFYW8rY2h4MnY0c0tNTXhtZWlXQ1JaRzBUYUIy?= =?utf-8?B?ZWw5VGdVQ1N4ZXJ0bVBjN2hZbFpEanZuU0plZUg2VkNTT2VCWDlDbGdLTXV2?= =?utf-8?B?cm5leVRHSUk5NUZoOFFSbnhXVm5nRmV0UFVVMWduM0w3QVRDR3FSRWh4ZW5x?= =?utf-8?B?N09NMDNOdEc3V2JBTC9tdXdYWVBpUzlSNXRaR0tJSjgwdEUvMmpnOXdHdUZQ?= =?utf-8?B?R0VRRlk0TmdRRWwzb0JtR2tubmZpZXlKRjB3MGhYTzRKdksvSndJZzNLOXV0?= =?utf-8?B?dWcxaHBCYlhiNXg1bUJ6T09nZG8ySUZLMGFzSDJoSHdmRGVIZmdLZ3ZUeVFI?= =?utf-8?B?d29vNUNPR20wYzMvT1NJUVJha25kQUlvWHhuYVB0UjNCUUFPZzNkUllJbW1H?= =?utf-8?B?TWZ4c1pDb2NwMEFmMERGaGhHU3B5K2ZGS3QwQm1CVkVhS1RGWk1aVkZpN2VX?= =?utf-8?B?WmZvVEdYbWlreXRUVFgzTXQ3bzRZdkVPb2dnL1NEaU54UjNNbFArcTdlYnVE?= =?utf-8?B?SXhvSWtpL1VNOFF2bWtWMEVXVUtZemFzR0xzMXJYb0w2a2lKdldGM3RWL2JR?= =?utf-8?B?VlZsWGU4aFFjUmRTWXBXTGJFb3Q4RmtYSWpTK25WQXU4dFdKSEExUGxxTyt6?= =?utf-8?B?aG9qMVRiaHhFNWl3bFd5Z0hxSG0yampsbmdKSXRuQ2hvaUtGYlc1L2JXUmtU?= =?utf-8?B?VDFkUnlJdVpBPT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: b2ebf44b-f7a2-408a-8029-08da53cf657c X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Jun 2022 21:45:46.3901 (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: AS8PR01MB7781 Subject: Re: [FFmpeg-devel] [PATCH 3/6] avformat/matroskaenc: Split updating CodecPrivate from writing it 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 Tue, Jun 21, 2022 at 04:34:15AM +0200, Andreas Rheinhardt wrote: >> Up until now, updating extradata was very ad-hoc: The amount of >> space reserved for extradata was not recorded when writing the >> header; instead the AAC code simply presumed that it was enough. >> This commit changes this by recording how much space is available. >> >> This brings with it that the code for writing of and reserving space >> for the CodecPrivate and for updating it diverges. They are therefore >> split; this allows to put other common tasks like seeking to >> right offset as well as writing padding (in case the new extradata did >> not fill the whole reserved space) to this common function. >> >> The code for filling up the reserved space is smarter than the code >> it replaces; therefore it is no longer necessary to reserve more >> than necessary just to be sure that one can add an EBML Void element >> (whose minimum size is two) lateron. This is the reason for the change >> to the aac-autobsf-adtstoasc test. >> >> Signed-off-by: Andreas Rheinhardt >> --- >> libavformat/matroskaenc.c | 120 ++++++++++++++++++--------- >> tests/ref/fate/aac-autobsf-adtstoasc | 4 +- >> 2 files changed, 84 insertions(+), 40 deletions(-) > > triggers assert in fate > > ffmpeg/ffmpeg -nostdin -i fate-suite//audiomatch/tones_afconvert_16000_mono_aac_lc.m4a -f segment -segment_time 1 -map 0 -flags +bitexact -codec copy -segment_format_options live=1 -segment_list ffmpeg/tests/data/adts-to-mkv.m3u8 -y ffmpeg/tests/data/adts-to-mkv-%03d.mkv > > Metadata: > major_brand : M4A > minor_version : 0 > compatible_brands: M4A mp42isom > creation_time : 2024-12-23T09:22:20.000000Z > iTunSMPB : 00000000 00000840 000002C0 0000000000007D00 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 > Duration: 00:00:02.18, start: 0.132000, bitrate: 35 kb/s > Stream #0:0[0x1](eng): Audio: aac (LC) (mp4a / 0x6134706D), 16000 Hz, mono, fltp, 20 kb/s (default) > Metadata: > creation_time : 2024-12-23T09:22:20.000000Z > vendor_id : [0][0][0][0] > [segment @ 0x562c56aa3f40] Opening 'ffmpeg/tests/data/adts-to-mkv-000.mkv' for writing > Assertion codecpriv_size < max_payload_size failed at libavformat/matroskaenc.c:1239 > Aborted (core dumped) > Thanks for this. An earlier version stored the complete amount of bytes reserved in the header including the bytes reserved for header and length fields, whereas this version only stores the bytes necessary for the payload and leaves the details of how much bytes to actually write to mkv_put_codecprivate(). When I changed this, I forgot to correctly update the assert (it is now of course entirely normal for codecpriv_size and max_payload_size to coincide, so it should be a <= instead of <) and it seems I also only ran the final version with an assert-level 0 build only; earlier versions were checked with --assert-level=2. How embarrassing. Anyway, thanks for testing. - 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".