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 ESMTPS id AFD9B4DA91 for ; Fri, 28 Feb 2025 02:44:04 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 39FC568DD2B; Fri, 28 Feb 2025 04:44:00 +0200 (EET) Received: from NAM04-DM6-obe.outbound.protection.outlook.com (mail-dm6nam04olkn2021.outbound.protection.outlook.com [40.92.45.21]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 02B2F68DC38 for ; Fri, 28 Feb 2025 04:43:53 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=xiZx94Qki7FU64t/rtxdfhmbwL906eHTJhjtrZCD/W8a3BbDVGUXLleWWfWtF3+KozYs0UVpj9+4LxuaSircaw/59wU5cJqJ99Ix/Q3MLXSQJiTTLiBnIPMGijqlRKDq0rcSbw5uGUSH6WAK3wTTg49Y1zCRBUCHs2r041drlT/HqpGtsJPyxzzH3tKm5i7XJ8SEvhg0wThOoTHdbwikty8YuYxDNMdpnKf3Gn9P/JKBwQx7eqUz3FBbXzcNkVsaVqJZXP+Fl6Bdx8Wh0SIeBIwf+Sxiu9NDmyyn2NmeZJlhJoEH0Ts3e0X30FLA4gE3xom0beFDuHdxbsbltYVg5Q== 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=Fxdc/n4/XsxsVMqDZ+gvMo5ktEap95b+C8vaM5I6xvk=; b=CqyJAqE/ZEd+aBH/kxagsIJOFWBSLgS6E/BMmOQsTxC2Bb8zZVAOL/PLd7v2aQdiQV/KsZX5SYTsV2O5aHlqGi/or3VmT4DOKIq4Ee9MhD+nb2tjFjllyEgR9bLpEsbCKEV6NCn0xE6kIejhfNRWLw7/LKABi8h8Cd9jUD4Dj7o06Dm/lItphHJfCvI3ge39PyomsiWXQNPXUzkJiBp7YhdkQBWspRhSD8TtSYuXhyYJQJ+7+lLGSofWkbHJBYsuGei1YqBq50m+PEH8WjsbA875VKb47Dx3/CBEPPmMMesCdmeBr4YgwFKK502V/8nFQZgrYNSohxPlQD/dt9Xxxg== 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=Fxdc/n4/XsxsVMqDZ+gvMo5ktEap95b+C8vaM5I6xvk=; b=lTZ9xUegnvxyX2MSRcUjqvykhk12d2Km8UijFpDCA0pxm9jOmtM8YyGA60dNuRAEBdrSZKXxxbt/+7+sTvBkzW3s7jCEFqLiyOL+0L8ASUD+AyBJ3yWhrSoYVJgZDEv34MdQESzTJghTij0j1+S6kQ891MQpIhVSqEARXzOu0zx/3ssF4Gl1xb2AiEK6LQfDDxpj95pLj2DmUIw7ksDgfhrH2BoZF6fdeUNTScI90/C43Ap4yyvXjHN9gjOZWm7tEqwLlSnXsLvSP0oxfSLfqOhN/MCaFaa3qmZRs4uT588lbD+hBcXt16zAT3rO+HYtyvd1iH/VmdYcd5r/gL5OLQ== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by EA2P223MB1122.NAMP223.PROD.OUTLOOK.COM (2603:10b6:303:24e::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8489.18; Fri, 28 Feb 2025 02:43:50 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7%7]) with mapi id 15.20.8489.021; Fri, 28 Feb 2025 02:43:50 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and move writer functions and options Thread-Index: AQHbiSAsTL1DYjnhHEKR31MvWbpZXrNb7HSAgAABH5CAAARvcIAADNyAgAABTYA= Date: Fri, 28 Feb 2025 02:43:50 +0000 Message-ID: References: <89d538cab47c517b35ecaccb563e14334c9f9927.1740664900.git.ffmpegagent@gmail.com> <20250228012321.GP4991@pb2> <20250228022915.GQ4991@pb2> In-Reply-To: <20250228022915.GQ4991@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_|EA2P223MB1122:EE_ x-ms-office365-filtering-correlation-id: 7f8fcbd2-3fc8-4b00-433f-08dd57a1bb6d x-microsoft-antispam: BCL:0; ARA:14566002|8062599003|8060799006|19110799003|7092599003|461199028|15080799006|3412199025|440099028|41001999003|12091999003|102099032; x-microsoft-antispam-message-info: =?us-ascii?Q?7DIiojpr4Zt0vtlkBrBb7rSG2cop3choc/QEEe87yj/eJrZWxO2Iq0g+5lcf?= =?us-ascii?Q?Rsw60p2K8dvZN82Rb2HIqFYDowNCeOWg26lvGpKtLODKEV8Y4FEaqS/JJYsX?= =?us-ascii?Q?gxgOK5MHruM8StK+vB2t6AdkZGKObdOBd9QuN4SfyyXVpRcGkscahUBVGVtd?= =?us-ascii?Q?EuNrUD6ssl8KeVFLG8na4A/U4QYJ5kgFaqWxCpDPehxwF6QvrK2kbP4Wms+f?= =?us-ascii?Q?HEvBByt31RcGN1UQQzEXH/RSmjJhVIdMamfW5FlGhRwkBBMN0nYLM27016Vw?= =?us-ascii?Q?FlpQG0KOTvk0nsYVRJHdzSVwoHgU0Ic+ZrC7SSje0DGC9ChDMAAvanB+eqxp?= =?us-ascii?Q?gT9jvNw48tazrQgaH4UFTNYf5NCUav2eRMvGEFDPMXPavKZ3M6QXI6pj9Jte?= =?us-ascii?Q?vnKPG511lx25aLUPkCTHKmZwVWxLFarSFHJgLMjppY1ZVkqFnQp9wNoGYSnI?= =?us-ascii?Q?TbkgaombXZXTDGhWmgFV+21iGBy2DGyF5cCAEDlQ0K+g3IL38JM83ue6T5/i?= =?us-ascii?Q?9aErfxXKOb9Uc5lOiV8olV+9p3dBD/p+noZvLgY4tYTE5XXLAWYJ8Y5Kc/Xe?= =?us-ascii?Q?aVqK3Xt9ZnS9YNhMs9FYqI2inmPjPvzwfn4O06iTjr7qqq8ZDlQi7rkCpF5D?= =?us-ascii?Q?bHQWM0QiNYIPvI7OFDY761qRXWsoA1VrpWQrZxsR1xZJJ6LuSjTyivHoB94P?= =?us-ascii?Q?V50Cd7r/AUaVg77K811bJ/v2J6TVwe9kd1ZR1iPrJf2HElu/YY9GaekW5+qZ?= =?us-ascii?Q?SdqEZwHSo/wycaZowlal9rWuBD5hPsJKvAosdY9zut2sn+fnauhEQzWf9Y1h?= =?us-ascii?Q?7lPKDk6yN/9+ybgXhXasacXmbE6/Mi2tcp4ilTuXBPGQ80hEF/5wK02Zz7du?= =?us-ascii?Q?i90i/xcgpiUI5efZ+/o11ri5yM6BQ6xupznFS31LE9mFjRkL3sYdkeSAE/1X?= =?us-ascii?Q?TEjTX7jf4rq6gu3WzsEp6oS/0SSB++wHFit7A3Z+DZOwQShgVsk3/ng7pdo2?= =?us-ascii?Q?R4xWYuNgbx/mRiTgv2mg8vOYA9ww1Cm88rAk4s6IL9TluNNUjDWknE21eCMH?= =?us-ascii?Q?8eIOlGcsHKITKaqG3m3ONwIYd8FS9011X4SHmm7QViP23tj+ihkowFAtZalN?= =?us-ascii?Q?yQQ7mCO48vtYZmFg3P8ZrImIELuFQfqaPQ=3D=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?O5FE7AVHBxWEVebQIycGQBK3p1RupzSRrqWEA9vEqqlSblEEV0YEHtPN6qAI?= =?us-ascii?Q?/c7xpnmLzz0qi/YYUzjiUk5YTLZRDx/fF9yHr9n+OjhnVZIcB+TZ4xlpWzdk?= =?us-ascii?Q?e0Yj5tCrJ2ibjWUr5oSR8UHTlCoWcEva8Vsce5Dwol8UsCXDnXWCsetcbtwU?= =?us-ascii?Q?HdtuG7yIDiSTZj6+9M2PKLunRt2GlRL9StUYFQN+W4cRCqb6djU7e181CBtZ?= =?us-ascii?Q?9wM8AlfhwaTX83GOJia65gFeCD2ID376Ets1upDZek+fKlmseTJi7In5wmlf?= =?us-ascii?Q?6q7NcWJhoglerahisRv3ptNxR7fUhxvDFIdSPXLrBcwhTnK11DnfHdCC1e50?= =?us-ascii?Q?HP3BIYk5wHURsoIDtkcm8ad30tEPci7fFcJj9RIQUWjv10SGPXz99gh2O+5f?= =?us-ascii?Q?gPuzBpQC0DkTXL2WMGAnP3KPy6RS0PXGYDYDMDoC5VaY1SUxyAxS0vszQ40Z?= =?us-ascii?Q?5lVjYssbu3ajTdTSbnjf2uxTck7bvhD4A7JqE8ooYNmmVkk+zXQuznLdft4Q?= =?us-ascii?Q?4WckxtZoWqLy7Dt+pgIv1nwUwDMOxewAOR7A5yB0+djMa2EVIW2WBNoNueOy?= =?us-ascii?Q?MRKztTJ/Fr44KndH9Cd/W8ViZBTU7GNp4rbLIWndgUnCksEbi++WiZBRaGTo?= =?us-ascii?Q?/+n2r1ZdIp0JuiHwIZLGetixb0zT0x6f97jV5jiB1oWot3r63RuybISTfqaU?= =?us-ascii?Q?bkrr+OM/xzpfOdDNW38WW2W2ECReQYpO1NVRGRuy9X1oXJRGeBjRzNz9yCea?= =?us-ascii?Q?Mwn7GJ5WhlxXdud7Q31fv9yFy+tARJiFJIaFlEWI2WmdIFvmldHbcapeSShY?= =?us-ascii?Q?sxPHM0jz4YnUwDQrGJsu1Q9rJjOtnmSgkN3cW0k1pq5ybIUkXZEggs0LuVKN?= =?us-ascii?Q?n2YzXXAkgyjtI31nJWBB/oIeo/u+a4hSvN7eSv5pKvmKLc93L/FH1feJi7CM?= =?us-ascii?Q?4gGJfD3bUCN1aGKHYowOmMuHpsMXGlIaBYbXNw4f3vqtx+JxUDbjnwv0SKLk?= =?us-ascii?Q?RV8sIdwqiLW3pL9gyJcsGjw79XUzh4UXwp/EgGne5kbEc+MdXseRaddz2aSU?= =?us-ascii?Q?JbIf6XojgtkSMwm5FKUlk/1qoC5qZy1ROnnVLTvJmGFHnOhYzLjzXlJp6bVh?= =?us-ascii?Q?/IiCwOdmAtkthTHiH/+XDaW4l58sufPzQK7wgL+reA51vtd5uuCNCVJBpDrj?= =?us-ascii?Q?GXm7Oswh7Ow8+MYEcRRcb2uA3gakaj+W/7vS3/mz6TIuaSp/NlpdwztEQv8?= =?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: 7f8fcbd2-3fc8-4b00-433f-08dd57a1bb6d X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Feb 2025 02:43:50.2265 (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: EA2P223MB1122 Subject: Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and move writer functions and options 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 > Michael Niedermayer > Sent: Freitag, 28. Februar 2025 03:29 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename and > move writer functions and options > > On Fri, Feb 28, 2025 at 01:47:05AM +0000, Soft Works wrote: > > > > > > > -----Original Message----- > > > From: ffmpeg-devel On Behalf Of > Soft > > > Works > > > Sent: Freitag, 28. Februar 2025 02:31 > > > To: FFmpeg development discussions and patches devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: Rename > and > > > move writer functions and options > > > > > > > > > > > > > -----Original Message----- > > > > From: ffmpeg-devel On Behalf Of > > > > Michael Niedermayer > > > > Sent: Freitag, 28. Februar 2025 02:23 > > > > To: FFmpeg development discussions and patches > > devel@ffmpeg.org> > > > > Subject: Re: [FFmpeg-devel] [PATCH 2/8] ffprobe/avtextformat: > Rename > > > and > > > > move writer functions and options > > > > > > > > On Thu, Feb 27, 2025 at 02:01:34PM +0000, softworkz wrote: > > > > > From: softworkz > > > > > > > > > > Signed-off-by: softworkz > > > > > --- > > > > > fftools/ffprobe.c | 910 ++++++--------------- > ---- > > > -- > > > > - > > > > > libavutil/Makefile | 1 + > > > > > libavutil/avtextformat.h | 50 ++ > > > > > libavutil/textformat/avtextformat.c | 620 +++++++++++++++++++ > > > > > 4 files changed, 848 insertions(+), 733 deletions(-) > > > > > create mode 100644 libavutil/textformat/avtextformat.c > > > > > > > > this and patch 1 crash > > > > > > > > ==1574970== Invalid read of size 1 > > > > ==1574970== at 0x483EF46: strlen (in /usr/lib/x86_64-linux- > > > > gnu/valgrind/vgpreload_memcheck-amd64-linux.so) > > > > ==1574970== by 0x7683D14: __vfprintf_internal (vfprintf- > > > > internal.c:1688) > > > > ==1574970== by 0x7696F99: __vsnprintf_internal > (vsnprintf.c:114) > > > > ==1574970== by 0x1239AA1: av_vbprintf (in ffmpeg/ffprobe_g) > > > > ==1574970== by 0x1256473: av_log_default_callback (in > > > > ffmpeg/ffprobe_g) > > > > ==1574970== by 0x1256162: av_log (in ffmpeg/ffprobe_g) > > > > ==1574970== by 0x125CB4B: av_opt_set_defaults2 (in > > > ffmpeg/ffprobe_g) > > > > ==1574970== by 0x127611C: avtext_context_open (in > ffmpeg/ffprobe_g) > > > > ==1574970== by 0x2F9672: main (in ffmpeg/ffprobe_g) > > > > ==1574970== Address 0xffffffff is not stack'd, malloc'd or > (recently) > > > > free'd > > > > ==1574970== > > > > ==1574970== > > > > ==1574970== Process terminating with default action of signal 11 > > > > (SIGSEGV) > > > > ==1574970== Access not within mapped region at address 0xFFFFFFFF > > > > ==1574970== at 0x483EF46: strlen (in /usr/lib/x86_64-linux- > > > > gnu/valgrind/vgpreload_memcheck-amd64-linux.so) > > > > ==1574970== by 0x7683D14: __vfprintf_internal (vfprintf- > > > > internal.c:1688) > > > > ==1574970== by 0x7696F99: __vsnprintf_internal > (vsnprintf.c:114) > > > > ==1574970== by 0x1239AA1: av_vbprintf (in ffmpeg/ffprobe_g) > > > > ==1574970== by 0x1256473: av_log_default_callback (in > > > > ffmpeg/ffprobe_g) > > > > ==1574970== by 0x1256162: av_log (in ffmpeg/ffprobe_g) > > > > ==1574970== by 0x125CB4B: av_opt_set_defaults2 (in > > > ffmpeg/ffprobe_g) > > > > ==1574970== by 0x127611C: avtext_context_open (in > ffmpeg/ffprobe_g) > > > > ==1574970== by 0x2F9672: main (in ffmpeg/ffprobe_g) > > > > ==1574970== If you believe this happened as a result of a stack > > > > ==1574970== overflow in your program's main thread (unlikely but > > > > ==1574970== possible), you can try to increase the size of the > > > > ==1574970== main thread stack using the --main-stacksize= flag. > > > > ==1574970== The main thread stack size used in this run was > 8388608. > > > > > > > > testcase: ./ffprobe_g -v 99 ~/tickets/3679/wgetmjpegdump > > > > > > > > tell me if this file is not in the ticket and ill send it to you > > > > > > Hi Michael, > > > > > > only patch 8/8 is expected to work. Since this is a RFC for the > moment, > > > I spared the time for making sure that every single commit builds. > > > (please see cover letter notes in 0/8) > > > > > > Once the direction is clear, I'll prepare a patchset with all > commits > > > working of course. > > ok > > > > > > > > Thanks > > > > Did you try 8/8 - does it pass for you? It's fine locally on x64 > Linux, on AzDevops Linux x64 and on Patchwork andriy/make_fate_x86. > > But what bothers me a bit is that it failed on the Chinese CPU system. > > i found the issue in 1+2 as a result of bisecting, so no the whole set > crashed What was the error when running all including 8/8? The same? > > > > testcase: ./ffprobe_g -v 99 ~/tickets/3679/wgetmjpegdump > > > > > > > > tell me if this file is not in the ticket and ill send it to you Yes please. It doesn't seem to be there (only some pgrep and ncap files). Thank you _______________________________________________ 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".