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 3EF3F4D127 for <ffmpegdev@gitmailbox.com>; Wed, 16 Apr 2025 05:37:09 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 8894F687D9F; Wed, 16 Apr 2025 08:37:04 +0300 (EEST) Received: from EUR03-VI1-obe.outbound.protection.outlook.com (mail-vi1eur03olkn2093.outbound.protection.outlook.com [40.92.57.93]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 730B1687C94 for <ffmpeg-devel@ffmpeg.org>; Wed, 16 Apr 2025 08:36:57 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=XxnXqlhJhQK6jiocvY1piTyBt3uPHt0YtKTOX36NaGnQnWh45ky0iGvzC8Gu2XCen4KTWOsBozzf5GO+X1QJRuiQ1CQ2zX+xrKkEFqAvVQlhzwbAFbrmXNLyrIVRC8xevcW2PsENtk4h+kzj1tZGB87Iw8jI4rIS6ghciC/AWv1juo9WK+W02Fdxt7UvRsnT1YKLVPdp/7zsW9E9zxK0VS8fRjBrC9iQocE1q839uOkx2Nuq2ZRBCrSeoR2mRfD9DYQGpVSkQK0vu0fO5eSDmh804r7f7mBH5qetidt0/nR0B+il25cphLEco5t8X7Ima6GhMljLZNwTxSedZH8DPw== 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=bFk3wSCCn5GYZpWIeg55ZVagkMtf4LChkbIYGw0rv10=; b=ISviq1ShgAVvH++zGIMSd9qCBFYrE6eX/J4GiZMtmVe8XiupZ/cjpf4Yr4LrBdcUqQFQqPfgm/+VNfC6jxXbN+EzSuTvvzbFH8IUDhZ3uc9pu/8mQjN9IVaV9hASjwmRxtZj3R1ZwqrYUUf4a62tsjIwo5P4mmxr7Yh4EFFM4V5G4HSBe7RH4W+T6iiWe17EI2ux66bXBVGVnWcG2R6Jhe8LNkOBCx+/E75nHpKq9wL+TTH4PzAZqSL87On1/on7QJsFA6uFIa89lHOFA1nYq/nimvNcpah6KJXnwF5WlZ+1PcDd6hXjeY4CLwpI8sAL9QTBM0M7vEwLZzNptoxFIw== 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=bFk3wSCCn5GYZpWIeg55ZVagkMtf4LChkbIYGw0rv10=; b=uk/TT+nsCwli70OD5ImGHsLPWiGTROW8GVdAsxSOCKBeNHrCU2Zp+yh/rci2Fg/e5TXSsls1RcyZvkNToItus3cwBtrMUDKviT3JJzotRx0p9pWl/kENevbO+eGJd0e2OTGyCOyWiy+spoL4n54GvJDA8Qsq9n5JbXL6119LHufHdtd+ZHLUfllRLeT77di7tjLjC8REYO+tY4H+yr5y8eHGYCYusQjt1ULm3YU6rOMU7OQedDBzhpHU9n1ImrlTzKNA6k95q1x/UKEkaYjkvOvZIgmdgYQrr9SwXESxNzzlCmdt93vP93RRjlfAvwB5sQSyr9dvjaxaKOZtfGr9Eg== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by AS8P250MB0792.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:571::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8655.14; Wed, 16 Apr 2025 05:36:55 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614%5]) with mapi id 15.20.8655.012; Wed, 16 Apr 2025 05:36:55 +0000 Message-ID: <GV1P250MB0737947C887B0E8F0123D5728FBD2@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM> Date: Wed, 16 Apr 2025 07:36:53 +0200 User-Agent: Mozilla Thunderbird To: ffmpeg-devel@ffmpeg.org References: <GV1P250MB0737FC8C3B89F1100A6324E38FB22@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM> <DM8P223MB0365FA00332594FD619AA750BAB22@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> <GV1P250MB07376B6342E54BFF4FC853378FB22@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM> <DM8P223MB0365A746840DC346FCBA543CBABD2@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> <GV1P250MB073782E37FF068C4070974E08FBD2@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM> <DM8P223MB036503000DE840BECEFCDFFFBABD2@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> Content-Language: en-US From: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> In-Reply-To: <DM8P223MB036503000DE840BECEFCDFFFBABD2@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> X-ClientProxiedBy: FR3P281CA0111.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:a3::14) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <cd1005bb-aa3e-4b3c-a394-26a27406f3bc@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|AS8P250MB0792:EE_ X-MS-Office365-Filtering-Correlation-Id: 039f0007-5f2d-4a80-0e6f-08dd7ca8b2a5 X-Microsoft-Antispam: BCL:0; ARA:14566002|6090799003|5072599009|8060799006|461199028|15080799006|19110799003|7092599003|3412199025|440099028; X-Microsoft-Antispam-Message-Info: =?utf-8?B?T2p2cTNaN1ZjZGd5ZE9SMzhsNGZaMFNVTElDdEtLclBvRTB0UENBYm5UZWw2?= =?utf-8?B?UGpJNjBQNGQrYitKUXlERzdBcGpsWGRVVDdOenAzbC9ZaHIvQXVPWVpFbGV5?= =?utf-8?B?Mng4TkhReGdWSjNSaCt2R1BpdWdMRUNSc0xuV0ovMmNXbGlsbXQwTld6enE4?= =?utf-8?B?VTEzcGJubHQ0ZHYyeStKWG5ZOWM1elZDWTRyUFY1dDBCVWpuc2d2dExWaldL?= =?utf-8?B?TUJUaW40NCtGbWo1aTdya3ZuRDJwSDJJNENkczVBQThNV2JmNmF1WXFSNkt0?= =?utf-8?B?MWo3NXErMU5IVGxlYlhZdWJlWGR4Y0FHSGVjZDlvTkloUDdlNFIrb3NhaElX?= =?utf-8?B?OEVyZzBjQXlweVJ2dXdpZ2U1VFRHMWZNTFI1SkoyRkozTzJyTlhZNFVmM2dJ?= =?utf-8?B?SEI3L0plS2ZKbUVsU3ZGN292QmxYOGcxcjJPQkFTTXFVaml1Vkt4M2FKQzBJ?= =?utf-8?B?TThUdEtYL1dXWm1Ra3poMzhRd1c5Mk1TS2VOMGlFTkwrNjVlVFBsdGJhUFBK?= =?utf-8?B?cVUyN0V0THRLTzhrMENvSzcyRk91NEZzdC9pYXNreGxmYzZqWlU5SFRFSG05?= =?utf-8?B?QmFkMElYVG9wdVdkNG1vV09jQ2ZJSlg3dTJaQlJ3dDlLakFmSEZXRmh2dDdu?= =?utf-8?B?TmVNOU1EWW4waWljcWd2TzA5bnh0R0pnNUx4bmtueFdsanRTWFNjbWowalF0?= =?utf-8?B?L1VaLzZNTEhhMFhyUTEvcnhQVzJxU211OXVodHd3K3c1OTIvUTltSS9XUXMz?= =?utf-8?B?dlhiV3AwOGI2ZWpaWlJXcndIdTQ4UkcrQzRuNFlkTXIvc1dnVTl0dDBQRktQ?= =?utf-8?B?c3haa0ZLRnBJaEJKRFNBQTFTRTZqWEtyOXAzSTRhUitmd1FtMHUxRk52SnhO?= =?utf-8?B?RzRHTEJxL3o4YUpCSjJuU3o3N0xZbGZ6UjlJbDMxelBDNjdiRWVGSmZ6aDd3?= =?utf-8?B?anJqNnRVbG5iNDl3TjhsM2Y5R3ZhYkpMTXBNZ2VScDlaVlJqcVR1ei8yeHJr?= =?utf-8?B?OVVORExtNWJRV2l0RWpsQndCSm1WMWRwMG5JT092YkEwa3BTM0R1bUxjZTUy?= =?utf-8?B?R3lCM3FHUDVoVld4SjRMS0VMSU5TN2pOY0NLc01Cd0VOdVoraWx6a3dIcXA0?= =?utf-8?B?NC9aQU9wM3NycFU3K3RJOFdEYWpDZThIZVVZWDVQalRHMTJZY2plRXppTDRh?= =?utf-8?B?OVR0bUZ0Yk9kMnphcmJkcUJHOU40M2lMbXY2MVExUUlCTGdWc2Q1bG1qdmtO?= =?utf-8?B?S2pmQ0pvZFIzVHFTOUhRNW1ocE5iSWpHaE1zMmNjOThWankvaVIzTlFORGlO?= =?utf-8?B?cHVxYWN3cmpHdTBZdzRQYTY4K2Qwb0U4d2tLVUcvMkMvOTBHZUoyNVYzaTBw?= =?utf-8?B?cVo4RktMcno0aHArOGJYbEpOZFIzRDJROFdVZGNSMGdyYksyT2RKV2xpSFo5?= =?utf-8?B?MUF6TXlLT3lpM042MUtXeWloRXlVaWQ1Y01DV2ovNHZocktuK3dYOTc0enNs?= =?utf-8?B?V3FLSDB1UkFIalJ0WEYyRSttNlg2TXprL1VIUnAwNkRrODdwSnBkK1JJYjBU?= =?utf-8?B?SWFMUT09?= X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?VGhHaWRyWDBVOE9IdUE2RTlXUnoydTJlNmZLZUswZkRiK0VvNGNwc1RidGwv?= =?utf-8?B?STlBaG5Rbzd3WThiMWx5QVM2MkQycmVid1NUZURZQ2pqaHcvZ253SVFPRWN3?= =?utf-8?B?TiswTVZ6SmFaK1VzaDRjdzU1UEU4dTNOb3hsdzhVeWJmRWk5cEcvUVg4elRY?= =?utf-8?B?b0xURWg4bzF2OWZVNzFiTVFCN0hFRGJKU3lnSXExMDBjUHZEZWV6RmUwOThD?= =?utf-8?B?NmhYT3NwRkRFQlNudzRMaGZlNXJTV2dIL2R3M1Q2dHF0RFR4SUgyQm9Sa2lR?= =?utf-8?B?WitMTFhRRTFmK3JWSjBSWjlRUnRPUWNoczAyZTRqeW1mV3hGTDVCNTNPZXFz?= =?utf-8?B?TFYrQ001Zyt6dFE4WHgrL1BNeWJxRCtsaktLMForMWVnVVNET3BIZEx5cUUv?= =?utf-8?B?OStmdGNQd2xLL0k0cFFLdG9jYithQTVUWklTWUx4Y1BPR1dhQjVSdUl1dkRP?= =?utf-8?B?NUpHQmE2ekhyRXFESDdyV29XY3k3RTE5MVAyc1NCUDYxclhMTkRldE9MR3FT?= =?utf-8?B?TDg4UzJaY2RvelBhd1IxMzRzMVRLR3Awb1pqR2I5aStPZzR6RFQrTXM4YVZW?= =?utf-8?B?RjNQSGVkZzAvSHRFTG9zcmJ5TUE1RGd4S3NRWS9mNFpkSVd6c1V4Z29wQWp6?= =?utf-8?B?NFFWb0dFYmRhTFZuZkRISjZIL2xZRkM4ay81Ui8zUDZkcFQvaGdSdkdDcVcz?= =?utf-8?B?UEpab29uZjZOL0ZVblNLQm82RnM1ak04ajRiblpTMFpCSTdpU2MvNGI5WUVj?= =?utf-8?B?Q3BmemJCa2xpV2x0OW9pTFRaZGZnU0V5ZWoyUTlnRlhHUFRtSUlOZVNDWGNJ?= =?utf-8?B?QklQOWo3QTUvTG1KUnVKOVBBaEUvU3lYdzh4alpjZ29jMW1jRWJBVjgxMitj?= =?utf-8?B?N1RtRXBrdlVlMzdLRkF4T1ROaUdQN0p4anltZkE1SUc3SW1ORVN6bG1yTmxi?= =?utf-8?B?cU5xRVFOZjVNYkU4ZHFhYVQwZ0NPMGx3N3l4MzN3R1JybWdqa29pYVExcXdy?= =?utf-8?B?czlCcEo5RUFUc3dhb2tiRTBWMnJEa2Z1YnFxdmdxYVord01YU0ZpZHBUc3dK?= =?utf-8?B?cTVXbG44d3B6c0lKU2h3RTlHWkF0Y1hjY1Zvc3NXeWdQWi8yT2dyQnpxVHla?= =?utf-8?B?R0JSVnhja05VeFh5ak5MRkk2Rnc0anc2cEk1R2hnV3VWQ01NUFhRNm9EeXNj?= =?utf-8?B?bURHYmo2d05NNm5QVEViYVFyZFBOcUk5V1hVZytscVFubXVXYnoyVlRpWEUx?= =?utf-8?B?OVJHMUx3TjFyN3A0ZkN1N1oyY1Z5TTdIQTE5MlVnMXJ4V3VNOVMyVWZyQWVV?= =?utf-8?B?Vk5mU0puNHJobG1xRWtyYkJ1UFZ4T0tEcUsrb3hvN00xWm5kSll0TUFTSGlP?= =?utf-8?B?SzdOQlRFd25nSDBaZnViUUJhSWZuTVhiM00xakQxTjNIaitjSGgycDZMMDdB?= =?utf-8?B?RStIT21XbytuSDZsZ1hPUzAyekMzaExxcTZNZW5EQWt5QnRnZGJRQXVmM2l6?= =?utf-8?B?c050bDZGanVPaFNTRE5MdW5XSGMvc2tJOTdEb2RudXFETVRBazZCN2c4V3k3?= =?utf-8?B?SWpwWDRuKytaL24rbzd3QUpVekNQdkxtMWQ3RWRwUE85cHhEa1VUUjNIdGUy?= =?utf-8?B?blhJcytxTTgzQXZoYlp0RjdWeFRuUVhpY1dVVkgvOU5EbVFSRWQ0aTlYOEdK?= =?utf-8?B?Q094VWZxQlljNnhwWWJoSXhhSjVZS3drVTRvdDMwSkxSZERhVFlLV1Fxbklt?= =?utf-8?Q?ziXhPX5Z1dJJv3Z/fP4h0jeVxH9E9//wBZWLMY7?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 039f0007-5f2d-4a80-0e6f-08dd7ca8b2a5 X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Apr 2025 05:36:55.6646 (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: AS8P250MB0792 Subject: Re: [FFmpeg-devel] [PATCH 01/12] fftools/textformat/avtextformat: Simplify avtext_print_rational() 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/GV1P250MB0737947C887B0E8F0123D5728FBD2@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM/> List-Archive: <https://master.gitmailbox.com/ffmpegdev/> List-Post: <mailto:ffmpegdev@gitmailbox.com> softworkz .: > > >> -----Original Message----- >> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of >> Andreas Rheinhardt >> Sent: Mittwoch, 16. April 2025 06:28 >> To: ffmpeg-devel@ffmpeg.org >> Subject: Re: [FFmpeg-devel] [PATCH 01/12] >> fftools/textformat/avtextformat: Simplify avtext_print_rational() >> >> softworkz .: >>> >>> >>>> -----Original Message----- >>>> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of >>>> Andreas Rheinhardt >>>> Sent: Dienstag, 15. April 2025 10:36 >>>> To: ffmpeg-devel@ffmpeg.org >>>> Subject: Re: [FFmpeg-devel] [PATCH 01/12] >>>> fftools/textformat/avtextformat: Simplify avtext_print_rational() >>>> >>>> softworkz .: >>>>> >>>>> >>>>>> -----Original Message----- >>>>>> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of >>>>>> Andreas Rheinhardt >>>>>> Sent: Dienstag, 15. April 2025 03:00 >>>>>> To: FFmpeg development discussions and patches <ffmpeg- >>>>>> devel@ffmpeg.org> >>>>>> Subject: [FFmpeg-devel] [PATCH 01/12] >>>> fftools/textformat/avtextformat: >>>>>> Simplify avtext_print_rational() >>>>>> >>>>>> Patches attached. >>>>>> >>>>>> - Andreas >>>>> >>>>> >>>>> Hi Andreas, >>>>> >>>>> thanks a lot for working through this. I'll go over it tomorrow. >>>>> >>>>> As to not waste your time, it's probably best when we get those >>>>> changes applied in a timely manner so that I can rebase the new >>>>> patchset on top of it. >>>>> >>>>> Since you're sending the patches as attachments: >>>>> How do you want me to reply with code context? Whole files or >>>>> just snippets? And quoted? >>>>> >>>> >>>> Snippets is better. So is quoted. >>>> >>>> - Andreas >>>> >>> >>> Hi Andreas, >>> >>> thanks again for the well-spotted improvements. Just two notes: >>> >>> >>> 0007-fftools-textformat-Use-av_default_item_name.patch >>> >>> In the new patchset, those macros are removed from the individual >>> files. There's now a single macro in tf_internal.h and I've applied >>> this change there. >> >> So you use a move/deduplicate commit to change something? Not good. > > No. The patchset has a deduplication commit. That's what I've submitted > to the ML already. > Now I made another commit locally (on top of that) which makes this change. > > >>> 0008-fftools-textformat-avtextformat-Fix-segfault-upon-al.patch >>> 0009-fftools-textformat-avtextformat-Fix-segfault-upon-al.patch >>> >>> Can this happen? >> >> Of course it can. All allocations can fail. That's why we check them. >> Have you been coding with the assumption that allocations never fail? >> (You can use av_max_alloc(1); to simulate allocation failures.) > > Allocations can fail, but statically initialized global const values? The pointers to said static objects are only set after having allocated the private context. So the issue can happen (not with current master) "if an AVTextFormatContext's private context could not be allocated" (as has been said in a commit message; similarly for AVTextWriterContext). (In case you are wondering why I did not set the pointers to writer/formatter before having allocated the private context: The uninit functions are only supposed to be called with the writer/formatter's private context already allocated (otherwise all of them would need to check for its existence), so this would not reduce the amount of checks.) - 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".