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 EDD0E4A10D for ; Thu, 21 Mar 2024 12:34:28 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C32C368CA66; Thu, 21 Mar 2024 14:34:25 +0200 (EET) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05olkn2060.outbound.protection.outlook.com [40.92.91.60]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 2A03268CB56 for ; Thu, 21 Mar 2024 14:34:18 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=U+avrLFfeidjjZ+SBmfTHzGmc4+v3VZIilGA/592LkEASvcmfsLLKqN5mgaHZPvs8G9PvGpb5tvBFIVRqlfRbvttgjYzOKMN1GJJC5k1MeKBh2nXWFfZlvFi2mMiUJB0yvoxIsFjqjp6LR5qaJA/3ubnXL9FN0T2D775swLSZV36d/51OHoh8t4lmWN0Px4ITeXhHUZOQhJqj7hAfxV+ij6NZPAhE0WXWzMk4g5S6F7m3cm5Hpw3z8e+8EKDNje6ouVEKkQ95lUOW5ca0bDIQf1o8Q1cvdl5eznRiHjJduovQOGkUNqK6CL6kNVJ7eW/QTkbzXSIc1d75h2i4hQAfA== 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=QQKBqvI0Vatl3jDUArvN9p2E5W1xhmrfrU4hhFfDLYo=; b=W4scCICjQHcs09530BxF8kGkddYe9xWHlnTbDWQ70bvKyQ1znZqGRfxBLAdv2LGdencCGZAIyt/zo7rjEC8PY8er6fC87pJtv4CR/Y1r918i091Wznu1+dqoD9k5U9qJMdZsfipMG+dMczaiOnj5MbMvTlkWfKkQtO2SF1sc6+yi1NaEAejO9QD3YivojT1+tjjkbFnK8YrG1ziIuNPg0WtB9NXIi86PtInW/Zkx/lzGqplc10aX3wjStscgvXdBh72pWL/y1bBMx3UZgrFhc4FlUvbCfFsroZYi7mjbpZCgdqr3yaCIXg82e57ZW1yHtlsP0XK2WPNojyH6j6MV+w== 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=QQKBqvI0Vatl3jDUArvN9p2E5W1xhmrfrU4hhFfDLYo=; b=IwBJWpZZCr3GBjMgUoGHXGSTfGkE34NVFMnc4tfMClvxJ+KDj3qnyAAxSnU82kuiKg0VixfX3Scoi4j2bFSCut4xGkzit4C0oAS5uGgCmXQTZJWJ5o1nRy5yY2y7xCSvQvThYWUezeGvi4ddjjxLYltp8mnpTKYCMCR3NUJJB9sD7MJgJUqsqQHGosf2jj5qxRj704LtJSl1iwlRuZVnyoJcxkj6tYjpGYh7HPIZvXKm7paf/sRL3rlQq2iG2EnRccmX2VrzU1tUzPxSSlMx6Fgybv73lNS/cvJQm/94Pacw/kzyOcWJvPGkhDRU9v07oUvZxfC1Yd7TXocC8jSZOQ== Received: from DU0P250MB0747.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:3cb::16) by AM8P250MB0075.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:36e::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7386.31; Thu, 21 Mar 2024 12:34:14 +0000 Received: from DU0P250MB0747.EURP250.PROD.OUTLOOK.COM ([fe80::8230:ffad:27e5:20bf]) by DU0P250MB0747.EURP250.PROD.OUTLOOK.COM ([fe80::8230:ffad:27e5:20bf%3]) with mapi id 15.20.7362.035; Thu, 21 Mar 2024 12:34:14 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Thu, 21 Mar 2024 13:34:05 +0100 Message-ID: X-Mailer: git-send-email 2.40.1 X-TMN: [3N3lZB9Zv7q0uKmKXYsQt5MmEozVcZlcvoouXeJCn+s=] X-ClientProxiedBy: FR5P281CA0037.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:f3::17) To DU0P250MB0747.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:3cb::16) X-Microsoft-Original-Message-ID: <20240321123406.499087-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DU0P250MB0747:EE_|AM8P250MB0075:EE_ X-MS-Office365-Filtering-Correlation-Id: f2c82d66-9519-4d93-979a-08dc49a3377e X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: qPFe+VbmOL6kf3pP8F3DBsph+r7RXKdTQmQVT6WWzIFC7TH4ZSUpEmRuWmscDPrvhRLYjh1kRcp6bmwmRJ15NkXyc2MzKOhUzw+fBhE/xN6VVugufKTva8ogis3aLkWwBSfsssB/CQaKXWb1PLaiipZ/RFG2Oy8j5WJPC4U4+cm0Uh0sarCHH+43n4k/siUXDPqmd1KYtWCJxZ5t4ucKeFsqAOQtgLNz0KxAMSgEVsin3a7nbPUnlK/vQ4adEpJLe6LsVZzJQ71kSFdSdZL5ntveaeT8QUOrjMg8OSAT38GrL5TSXbpv81eWk2AU2lweWkvXBp2QPG/jUnTKHpiSRM7uEVCXcfVeiQnMte/yGPFbZ4WKL4e8ODH8ie7WVLiWy+j4LluyNfR0iyj0FVKwlXjvOnUUJxOdft9OcL7CcZz/lFMsFY63T78LK/Iv/sogAh1NdkQbEKToLolIvLp2lGXtmU9h7cLGBTBLocnAadks5L2Aj6BJYzPqBkfyMu3YTCoVp8ZlYoryhS6YNJViFlMUYIYUQdytTk8DB7Gak4JGqcrJCJAxSra4tJB9457YFWTJ2RAcQ07YGtW2mKIFt1n9OIUU9lE2ppOhsKTPpEMSf0VZmKijHBiqCNu3UP0Q X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?6vxkl36Un72mencloNW9AY6wGn8jjWk5xDWoL2mwPGKcnrJe3N1yv8jLfpOQ?= =?us-ascii?Q?ymcSvrEpppeOJMsc7+RUrIpguw8ZcJQ9+BN+0QHc1GCPUQLFZTVsNskJl4bM?= =?us-ascii?Q?i+Kgb2IqajGUwQChQAx/sdSj8jZsqb+CUAeBl1hB9jmekLEYClwiQMW33+o/?= =?us-ascii?Q?Tg58bI99fTqDabFokko/bhncSVbmSAwASzC9Ch8lB4tHEo1tmlVBVfYXQ243?= =?us-ascii?Q?TOE1LL94nY6SWPTG3HlOa/YQLNtUaPrAWmkSMW+ENEB+vpMMhZd6yo4iWDV5?= =?us-ascii?Q?oTAb/Q0KCt2gVjKyFd0lN2PvhnNu00lO2HudSHI8ZAGucIDKCeK2nhHxpC1w?= =?us-ascii?Q?J17242Kxh+fjPc86hKh24uDIpYg0BfKN17N4ZtHkvjRoXn48WvWCqS9TlNfN?= =?us-ascii?Q?bzVTV92FjdForHhdPn7CuQTWUvX4agp9zKAZxjQWj6RmoG8wJJoXhDnOTKXu?= =?us-ascii?Q?K/DZfYg98eFWkaQJ3zxJqYjwU/RlTmM+jvrNU9seAcKqnNGsIpVNBQplvTBt?= =?us-ascii?Q?L6iIO3Y4nE5jmEzOBRzUBx5FxrBy3rBWgHYMYj23ewYqquQrYsIr0lvXPixm?= =?us-ascii?Q?QssXF91BirLOlTRuaLHt1zbeSC7LQt2xqibTmCDIZh4D0lwVpJDYpD2iYQSV?= =?us-ascii?Q?4uXiI87E5eb4GnLvZO50SJ2WDlaX6PMB4aiXphOsR5J+PSmCUcFglxk3NY5S?= =?us-ascii?Q?9fvXKVJpkzUvk66opP/Mh9wUvhDqrNZzzqWdwPIogsHyz65rV1aCNiaOMI7Z?= =?us-ascii?Q?ftcrqcI1q4iiNlxXhll22i1uq+dGefsD53fQws5Gb/l881cDAH6EmNfhaxwB?= =?us-ascii?Q?dIuCYeWhGANHWFcvp5oHlSDgm9HkZMtl1ADZFweKnwpWnpQPhnENu4jsY0nW?= =?us-ascii?Q?bUJvFkXkFcZmcOVYNCZi6hjsNxo89ak3lcRCtdj3MU0JuxaxJ8WIneb9t9Vv?= =?us-ascii?Q?BzU0cKYOhzg6euSr/zsGVAAlynfPg92k+tlbayZ0njuXd9wYXRN8VoP+v4Pd?= =?us-ascii?Q?d4IHCvQ3QFSOFneA4I9kcYu5gD+oXdxSmBDSGy7OpPpqAcgzpzH3jELXV5kz?= =?us-ascii?Q?wynTQPkmrb9XvfnBE7hVVhL7j/7MkqXF/R+JP2tTYoKNZgxDR/MeLBVp/b7I?= =?us-ascii?Q?+BkMBdZcFJkQDGOLQc8G4aT0MjUvg171iQAlWkI+EO+E5Nlqo/HxEBeT5ugH?= =?us-ascii?Q?Oli3D1MiHme0WmpIyNUQMyDqKmmGgzoaDyPQF9vlkBf6ycUXQsWu0+DYkRmI?= =?us-ascii?Q?jYXc1SUqaucyNBmMSkjOgvUoDMq+DaKkTsxXEn4TPA=3D=3D?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: f2c82d66-9519-4d93-979a-08dc49a3377e X-MS-Exchange-CrossTenant-AuthSource: DU0P250MB0747.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Mar 2024 12:34:14.1205 (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: AM8P250MB0075 Subject: [FFmpeg-devel] [PATCH 1/2] avformat/demux: Restore pkt->stream_index assert check 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 Cc: Andreas Rheinhardt 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: It has been moved after "st = s->streams[pkt->stream_index]" in b140b8332c617b0eef4f872f3ef90c469e99920f. Deduplicate ff_read_packet() and ff_buffer_packet() while fixing this. This also fixes shadowing in ff_read_packet(). Signed-off-by: Andreas Rheinhardt --- libavformat/demux.c | 75 +++++++++++++++++++-------------------------- 1 file changed, 31 insertions(+), 44 deletions(-) diff --git a/libavformat/demux.c b/libavformat/demux.c index 4c50eb5568..4345ed4c8c 100644 --- a/libavformat/demux.c +++ b/libavformat/demux.c @@ -532,9 +532,6 @@ static void update_timestamps(AVFormatContext *s, AVStream *st, AVPacket *pkt) { FFStream *const sti = ffstream(st); - av_assert0(pkt->stream_index < (unsigned)s->nb_streams && - "Invalid stream index.\n"); - if (update_wrap_reference(s, st, pkt->stream_index, pkt) && sti->pts_wrap_behavior == AV_PTS_WRAP_SUB_OFFSET) { // correct first time stamps to negative values if (!is_relative(sti->first_dts)) @@ -555,13 +552,24 @@ static void update_timestamps(AVFormatContext *s, AVStream *st, AVPacket *pkt) pkt->dts = pkt->pts = av_rescale_q(av_gettime(), AV_TIME_BASE_Q, st->time_base); } -int ff_buffer_packet(AVFormatContext *s, AVPacket *pkt) +/** + * Handle a new packet and either return it directly if possible and + * allow_passthrough is true or queue the packet (or drop the packet + * if corrupt). + * + * @return < 0 on error, 0 if the packet was passed through, + * 1 if it was queued or dropped + */ +static int handle_new_packet(AVFormatContext *s, AVPacket *pkt, int allow_passthrough) { FFFormatContext *const si = ffformatcontext(s); - AVStream *st = s->streams[pkt->stream_index]; - FFStream *sti = ffstream(st); + AVStream *st; + FFStream *sti; int err; + av_assert0(pkt->stream_index < (unsigned)s->nb_streams && + "Invalid stream index.\n"); + if (pkt->flags & AV_PKT_FLAG_CORRUPT) { av_log(s, AV_LOG_WARNING, "Packet corrupt (stream = %d, dts = %s)", @@ -569,13 +577,19 @@ int ff_buffer_packet(AVFormatContext *s, AVPacket *pkt) if (s->flags & AVFMT_FLAG_DISCARD_CORRUPT) { av_log(s, AV_LOG_WARNING, ", dropping it.\n"); av_packet_unref(pkt); - return 0; + return 1; } av_log(s, AV_LOG_WARNING, ".\n"); } + st = s->streams[pkt->stream_index]; + sti = ffstream(st); + update_timestamps(s, st, pkt); + if (sti->request_probe <= 0 && allow_passthrough && !si->raw_packet_buffer.head) + return 0; + err = avpriv_packet_list_put(&si->raw_packet_buffer, pkt, NULL, 0); if (err < 0) { av_packet_unref(pkt); @@ -585,14 +599,18 @@ int ff_buffer_packet(AVFormatContext *s, AVPacket *pkt) pkt = &si->raw_packet_buffer.tail->pkt; si->raw_packet_buffer_size += pkt->size; - if (sti->request_probe <= 0) - return 0; - err = probe_codec(s, st, pkt); if (err < 0) return err; - return 0; + return 1; +} + +int ff_buffer_packet(AVFormatContext *s, AVPacket *pkt) +{ + int err = handle_new_packet(s, pkt, 0); + + return err < 0 ? err : 0; } int ff_read_packet(AVFormatContext *s, AVPacket *pkt) @@ -612,9 +630,6 @@ FF_ENABLE_DEPRECATION_WARNINGS for (;;) { PacketListEntry *pktl = si->raw_packet_buffer.head; - AVStream *st; - FFStream *sti; - const AVPacket *pkt1; if (pktl) { AVStream *const st = s->streams[pktl->pkt.stream_index]; @@ -656,36 +671,8 @@ FF_ENABLE_DEPRECATION_WARNINGS return err; } - if (pkt->flags & AV_PKT_FLAG_CORRUPT) { - av_log(s, AV_LOG_WARNING, - "Packet corrupt (stream = %d, dts = %s)", - pkt->stream_index, av_ts2str(pkt->dts)); - if (s->flags & AVFMT_FLAG_DISCARD_CORRUPT) { - av_log(s, AV_LOG_WARNING, ", dropping it.\n"); - av_packet_unref(pkt); - continue; - } - av_log(s, AV_LOG_WARNING, ".\n"); - } - - st = s->streams[pkt->stream_index]; - sti = ffstream(st); - - update_timestamps(s, st, pkt); - - if (!pktl && sti->request_probe <= 0) - return 0; - - err = avpriv_packet_list_put(&si->raw_packet_buffer, - pkt, NULL, 0); - if (err < 0) { - av_packet_unref(pkt); - return err; - } - pkt1 = &si->raw_packet_buffer.tail->pkt; - si->raw_packet_buffer_size += pkt1->size; - - if ((err = probe_codec(s, st, pkt1)) < 0) + err = handle_new_packet(s, pkt, 1); + if (err <= 0) /* Error or passthrough */ return err; } } -- 2.40.1 _______________________________________________ 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".