From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 37B4A4D161 for ; Mon, 10 Nov 2025 22:45:30 +0000 (UTC) Authentication-Results: ffbox; dkim=fail (body hash mismatch (got b'Iwzsv6QXbzG/8PulVAGtup0EwBb52x3AzkxwJy+uJpg=', expected b'g5/7XXnvtgyV0SPfAxfdSVpfY8uZec5Aq8tBPaIQFwE=')) header.d=ffmpeg.org header.i=@ffmpeg.org header.a=rsa-sha256 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ffmpeg.org; i=@ffmpeg.org; q=dns/txt; s=mail; t=1762814694; h=mime-version : to : date : message-id : reply-to : subject : list-id : list-archive : list-archive : list-help : list-owner : list-post : list-subscribe : list-unsubscribe : from : cc : content-type : content-transfer-encoding : from; bh=Iwzsv6QXbzG/8PulVAGtup0EwBb52x3AzkxwJy+uJpg=; b=Kx5VTQbJMHHadM9WjHP4auRcupdr0ppYWiu1SXd9idfQ3B4FdIRLQpeZShrNhoBVdlB3c 5la9ChAoANeFT47ata9tpJ/3HP8lqzAF5b0lE3N0c+0sNk1G0hlioKsr/2+cJx+avz6GNTT DQMtTWoBBWxrE6JvJ6AGEGpXjTt1RrZl80UohhjXXPwJvca07vIvdK2XIJ++msE5K/Da/qP wskmMNT3MPlBRkpIse5WrW7i8pDXEA94soTHvY2WmQkQk0qekZ+EMgv8JFGC4TWR7l7Epwp 3vYH9DsTgjwXjz3zlxSehc8gIF5Lyh+5omz1b89ZeZHKawK6UWpXn4bB64bg== Received: from [172.19.0.2] (unknown [172.19.0.2]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 513A168FC3C; Tue, 11 Nov 2025 00:44:54 +0200 (EET) ARC-Seal: i=1; cv=none; a=rsa-sha256; d=ffmpeg.org; s=arc; t=1762814669; b=eus4C7uaV5oeggkkLDWUbKl4POrO/GbwTcLqrJuQn6RceETgpvS0UqhSIAU/IVIDh10Dq sJuTK+p7zbekfhmL4b8Vs2umOAIPOKhuyQVkKJpVnhvFzODwq7f2TkdV7452VDxrD5NWxbA zjWr289sZc27lYcQ8sVB0ikiyHReYNcCvdUPgZYJnFoNIvZPImw7ANB4+ez66h28R+c8UT+ 07Sgl/+yfEBFjjJj0bupZx6C/Zul643w2vBNAZb9YF79H1HoB+kuXPkxeHifltjvKENJtVR zOcwxglPmJjjFawbuD7jxRFRqFSgQxTD4rI3IzHbhqGNvXuNbfGEIpML8hNg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=ffmpeg.org; s=arc; t=1762814669; h=from : sender : reply-to : subject : date : message-id : to : cc : mime-version : content-type : content-transfer-encoding : content-id : content-description : resent-date : resent-from : resent-sender : resent-to : resent-cc : resent-message-id : in-reply-to : references : list-id : list-help : list-unsubscribe : list-subscribe : list-post : list-owner : list-archive; bh=JgHvMA+KbVN5mP/vgxt/E2B6SC6WNWPVBgy55fz1hJU=; b=gZuqET0HnalHFcBAyA3UjD2JTDTiC2u/EK3aV0YoMRCuCxUO/83GCnurMMVJp3wWAo0ro 1hKmd2F2FgqPKFRfTKXpkBC+Ht7T/+Zw7zKyIsbkCgFNQnjKTV8BxtX/QLks/FqLJwuNtWy bnjfN5aY1DRh4qYnXDUfx91CW+/sceiCn5i/JrPAgmdi7kWU+WHPyS0lk2nJ3d2Ljb4b40V 7Of82CUuvRP56IpRBBP9Cupw299gzyU2YuEwsAgo/u6SHQcZPr8JnIyjqMCwc0mSnL+9Qdc rMyub9HTj1tNBDPmyHy3YhHvxZMSvmy/AFkqDWUidrrOUlQL/Cj+SunlJ1Pg== ARC-Authentication-Results: i=1; ffmpeg.org; dkim=pass header.d=ffmpeg.org header.i=@ffmpeg.org; arc=none; dmarc=pass header.from=ffmpeg.org policy.dmarc=quarantine Authentication-Results: ffmpeg.org; dkim=pass header.d=ffmpeg.org header.i=@ffmpeg.org; arc=none (Message is not ARC signed); dmarc=pass (Used From Domain Record) header.from=ffmpeg.org policy.dmarc=quarantine DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ffmpeg.org; i=@ffmpeg.org; q=dns/txt; s=mail; t=1762814658; h=content-type : mime-version : content-transfer-encoding : from : to : reply-to : subject : date : from; bh=g5/7XXnvtgyV0SPfAxfdSVpfY8uZec5Aq8tBPaIQFwE=; b=RxJdhtnW7oZSLsLY9mMKJLfiCVL96e48LOBpCNNJbCfIozWbl8eNKlxORVdBoPOapp7/N dgIDwD1itMAEDKRttIhr632EW3enJI+h4GtSwWQmczFMkO02v0pTjli9p2NMdNT2cl6AuLm PT5ulIWN+YEjim12Ay2cJgAvW80Ukymicp1c8qz/NsbVmm/ovVzV79YJTtSjZdi0/g1R6ct glhzBlA6qKJwr8PBID/qVFKd3dmCXHb4fl7H4RcBSlXVrV2lL/TnKaYJ+3GA2bWGonjubzK rwltM8crROtAEROMBtI83hbEJTWH4FKBHMC3faROTf21ymajNiN8BcYu05wA== Received: from 188d6d40ca7a (code.ffmpeg.org [188.245.149.3]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 9669C68EDAB for ; Tue, 11 Nov 2025 00:44:18 +0200 (EET) MIME-Version: 1.0 To: ffmpeg-devel@ffmpeg.org Date: Mon, 10 Nov 2025 22:44:18 -0000 Message-ID: <176281465887.25.6979679616954961386@2cb04c0e5124> Message-ID-Hash: W3542QXLWXEHAXCXVSEHR5KCYGUPLZB7 X-Message-ID-Hash: W3542QXLWXEHAXCXVSEHR5KCYGUPLZB7 X-MailFrom: code@ffmpeg.org X-Mailman-Rule-Hits: nonmember-moderation X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; header-match-ffmpeg-devel.ffmpeg.org-0; header-match-ffmpeg-devel.ffmpeg.org-1; header-match-ffmpeg-devel.ffmpeg.org-2; header-match-ffmpeg-devel.ffmpeg.org-3; emergency; member-moderation X-Mailman-Version: 3.3.10 Precedence: list Reply-To: FFmpeg development discussions and patches Subject: [FFmpeg-devel] [PATCH] Re-initialize stream on new metadata. (PR #20890) List-Id: FFmpeg development discussions and patches Archived-At: Archived-At: List-Archive: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: From: toots via ffmpeg-devel Cc: toots Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Archived-At: List-Archive: List-Post: PR #20890 opened by toots URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20890 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20890.patch >>From 7993a24cf4062fc4ed943ec5f3e66b0bbb325741 Mon Sep 17 00:00:00 2001 From: Romain Beauxis Date: Fri, 14 Mar 2025 00:27:04 -0500 Subject: [PATCH] Re-initialize stream on new metadata. --- libavformat/oggenc.c | 62 ++++++++++++++++++++++++++++++++++----- tests/fate-run.sh | 9 ++++++ tests/fate/ogg-flac.mak | 5 ++++ tests/fate/ogg-opus.mak | 5 ++++ tests/fate/ogg-vorbis.mak | 5 ++++ 5 files changed, 79 insertions(+), 7 deletions(-) diff --git a/libavformat/oggenc.c b/libavformat/oggenc.c index e1bb7dd972..943fc71c12 100644 --- a/libavformat/oggenc.c +++ b/libavformat/oggenc.c @@ -40,6 +40,9 @@ #define MAX_PAGE_SIZE 65025 +static int ogg_write_trailer(AVFormatContext *s); +static void ogg_deinit(AVFormatContext *s); + typedef struct OGGPage { int64_t start_granule; int64_t granule; @@ -634,13 +637,57 @@ static int ogg_write_header(AVFormatContext *s) return 0; } +static int ogg_check_new_metadata(AVFormatContext *s, AVPacket *pkt) +{ + int ret = 0; + size_t size; + AVStream *st = s->streams[pkt->stream_index]; + const uint8_t *side_metadata = av_packet_get_side_data(pkt, AV_PKT_DATA_STRINGS_METADATA, &size); + + if (!side_metadata) + return 0; + + if (s->nb_streams > 1) { + av_log(s, AV_LOG_WARNING, "Multiple streams present: cannot insert new metadata!\n"); + return 0; + } + + if (st->codecpar->codec_id != AV_CODEC_ID_VORBIS && + st->codecpar->codec_id != AV_CODEC_ID_FLAC && + st->codecpar->codec_id != AV_CODEC_ID_OPUS) { + av_log(s, AV_LOG_WARNING, "Inserting in-base metadata is only supported for vorbis, flac and opus streams!\n"); + return 0; + } + + ret = ogg_write_trailer(s); + if (ret < 0) + return ret; + + ogg_deinit(s); + + av_dict_free(&st->metadata); + ret = av_packet_unpack_dictionary(side_metadata, size, &st->metadata); + if (ret < 0) + return ret; + + ret = ogg_init(s); + if (ret < 0) + return ret; + + return ogg_write_header(s); +} + static int ogg_write_packet_internal(AVFormatContext *s, AVPacket *pkt) { AVStream *st = s->streams[pkt->stream_index]; - OGGStreamContext *oggstream = st->priv_data; + OGGStreamContext *oggstream; int ret; int64_t granule; + ogg_check_new_metadata(s, pkt); + + oggstream = st->priv_data; + if (st->codecpar->codec_id == AV_CODEC_ID_THEORA) { int64_t pts = oggstream->vrev < 1 ? pkt->pts : pkt->pts + pkt->duration; int pframe_count; @@ -720,7 +767,7 @@ static int ogg_write_trailer(AVFormatContext *s) return 0; } -static void ogg_free(AVFormatContext *s) +static void ogg_deinit(AVFormatContext *s) { OGGContext *ogg = s->priv_data; OGGPageList *p = ogg->page_list; @@ -738,6 +785,7 @@ static void ogg_free(AVFormatContext *s) av_freep(&oggstream->header[0]); } av_freep(&oggstream->header[1]); + av_freep(&st->priv_data); } while (p) { @@ -772,7 +820,7 @@ const FFOutputFormat ff_ogg_muxer = { .write_header = ogg_write_header, .write_packet = ogg_write_packet, .write_trailer = ogg_write_trailer, - .deinit = ogg_free, + .deinit = ogg_deinit, .p.flags = AVFMT_TS_NEGATIVE | AVFMT_TS_NONSTRICT, .p.priv_class = &ogg_muxer_class, .flags_internal = FF_OFMT_FLAG_ALLOW_FLUSH, @@ -791,7 +839,7 @@ const FFOutputFormat ff_oga_muxer = { .write_header = ogg_write_header, .write_packet = ogg_write_packet, .write_trailer = ogg_write_trailer, - .deinit = ogg_free, + .deinit = ogg_deinit, .p.flags = AVFMT_TS_NEGATIVE, .p.priv_class = &ogg_muxer_class, .flags_internal = FF_OFMT_FLAG_ALLOW_FLUSH, @@ -813,7 +861,7 @@ const FFOutputFormat ff_ogv_muxer = { .write_header = ogg_write_header, .write_packet = ogg_write_packet, .write_trailer = ogg_write_trailer, - .deinit = ogg_free, + .deinit = ogg_deinit, .p.flags = AVFMT_TS_NEGATIVE | AVFMT_TS_NONSTRICT, .p.priv_class = &ogg_muxer_class, .flags_internal = FF_OFMT_FLAG_ALLOW_FLUSH, @@ -832,7 +880,7 @@ const FFOutputFormat ff_spx_muxer = { .write_header = ogg_write_header, .write_packet = ogg_write_packet, .write_trailer = ogg_write_trailer, - .deinit = ogg_free, + .deinit = ogg_deinit, .p.flags = AVFMT_TS_NEGATIVE, .p.priv_class = &ogg_muxer_class, .flags_internal = FF_OFMT_FLAG_ALLOW_FLUSH, @@ -851,7 +899,7 @@ const FFOutputFormat ff_opus_muxer = { .write_header = ogg_write_header, .write_packet = ogg_write_packet, .write_trailer = ogg_write_trailer, - .deinit = ogg_free, + .deinit = ogg_deinit, .p.flags = AVFMT_TS_NEGATIVE, .p.priv_class = &ogg_muxer_class, .flags_internal = FF_OFMT_FLAG_ALLOW_FLUSH, diff --git a/tests/fate-run.sh b/tests/fate-run.sh index 6d1fe1185c..7e1c97c839 100755 --- a/tests/fate-run.sh +++ b/tests/fate-run.sh @@ -94,6 +94,15 @@ runecho(){ $target_exec $target_path/"$@" >&3 } +run_with_temp(){ + tmpfile=`mktemp` + trap 'rm -rf "$tmpfile"' EXIT + create_tmp=$1 + run $create_tmp $tmpfile + process_tmp=$2 + run $process_tmp $tmpfile +} + probefmt(){ run ffprobe${PROGSUF}${EXECSUF} -bitexact -threads $threads -show_entries format=format_name -print_format default=nw=1:nk=1 "$@" } diff --git a/tests/fate/ogg-flac.mak b/tests/fate/ogg-flac.mak index 07bdb232cb..516ebadf52 100644 --- a/tests/fate/ogg-flac.mak +++ b/tests/fate/ogg-flac.mak @@ -2,6 +2,11 @@ FATE_OGG_FLAC += fate-ogg-flac-chained-meta fate-ogg-flac-chained-meta: REF = $(SRC_PATH)/tests/ref/fate/ogg-flac-chained-meta.txt fate-ogg-flac-chained-meta: CMD = run $(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF) $(TARGET_SAMPLES)/ogg-flac/chained-meta.ogg +FATE_OGG_FLAC += fate-ogg-flac-copy-chained-meta +fate-ogg-flac-copy-chained-meta: $(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF) $(FFMPEG) +fate-ogg-flac-copy-chained-meta: REF = $(SRC_PATH)/tests/ref/fate/ogg-flac-chained-meta.txt +fate-ogg-flac-copy-chained-meta: CMD = run_with_temp "$(FFMPEG) -nostdin -hide_banner -loglevel quiet -i $(TARGET_SAMPLES)/ogg-flac/chained-meta.ogg -c copy -f ogg -y" "$(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF)" + FATE_OGG_FLAC-$(call DEMDEC, OGG, FLAC, FLAC_PARSER) += $(FATE_OGG_FLAC) FATE_SAMPLES_DUMP_STREAM_META += $(FATE_OGG_FLAC-yes) diff --git a/tests/fate/ogg-opus.mak b/tests/fate/ogg-opus.mak index 54b6fbabde..b8faabef4e 100644 --- a/tests/fate/ogg-opus.mak +++ b/tests/fate/ogg-opus.mak @@ -2,6 +2,11 @@ FATE_OGG_OPUS += fate-ogg-opus-chained-meta fate-ogg-opus-chained-meta: REF = $(SRC_PATH)/tests/ref/fate/ogg-opus-chained-meta.txt fate-ogg-opus-chained-meta: CMD = run $(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF) $(TARGET_SAMPLES)/ogg-opus/chained-meta.ogg +FATE_OGG_OPUS += fate-ogg-opus-copy-chained-meta +fate-ogg-opus-copy-chained-meta: $(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF) $(FFMPEG) +fate-ogg-opus-copy-chained-meta: REF = $(SRC_PATH)/tests/ref/fate/ogg-opus-chained-meta.txt +fate-ogg-opus-copy-chained-meta: CMD = run_with_temp "$(FFMPEG) -nostdin -hide_banner -loglevel quiet -i $(TARGET_SAMPLES)/ogg-opus/chained-meta.ogg -c copy -f ogg -y" "$(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF)" + FATE_OGG_OPUS-$(call DEMDEC, OGG, OPUS) += $(FATE_OGG_OPUS) FATE_SAMPLES_DUMP_STREAM_META += $(FATE_OGG_OPUS-yes) diff --git a/tests/fate/ogg-vorbis.mak b/tests/fate/ogg-vorbis.mak index 74805d591e..131c1362a8 100644 --- a/tests/fate/ogg-vorbis.mak +++ b/tests/fate/ogg-vorbis.mak @@ -2,6 +2,11 @@ FATE_OGG_VORBIS += fate-ogg-vorbis-chained-meta fate-ogg-vorbis-chained-meta: REF = $(SRC_PATH)/tests/ref/fate/ogg-vorbis-chained-meta.txt fate-ogg-vorbis-chained-meta: CMD = run $(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF) $(TARGET_SAMPLES)/ogg-vorbis/chained-meta.ogg +FATE_OGG_VORBIS += fate-ogg-vorbis-copy-chained-meta +fate-ogg-vorbis-copy-chained-meta: $(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF) $(FFMPEG) +fate-ogg-vorbis-copy-chained-meta: REF = $(SRC_PATH)/tests/ref/fate/ogg-vorbis-chained-meta.txt +fate-ogg-vorbis-copy-chained-meta: CMD = run_with_temp "$(FFMPEG) -nostdin -hide_banner -loglevel quiet -i $(TARGET_SAMPLES)/ogg-vorbis/chained-meta.ogg -c copy -f ogg -y" "$(APITESTSDIR)/api-dump-stream-meta-test$(EXESUF)" + FATE_OGG_VORBIS-$(call DEMDEC, OGG, VORBIS) += $(FATE_OGG_VORBIS) FATE_SAMPLES_DUMP_STREAM_META += $(FATE_OGG_VORBIS-yes) -- 2.49.1 _______________________________________________ ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org