From: lance.lmwang@gmail.com To: ffmpeg-devel@ffmpeg.org Cc: Limin Wang <lance.lmwang@gmail.com> Subject: [FFmpeg-devel] [PATCH 1/2] avformat/movenc: initialize pts/dts/duration of timecode packet Date: Wed, 2 Mar 2022 21:58:31 +0800 Message-ID: <1646229512-12103-1-git-send-email-lance.lmwang@gmail.com> (raw) From: Limin Wang <lance.lmwang@gmail.com> Fix below error message when timecode packet is written. "Application provided duration: -9223372036854775808 / timestamp: -9223372036854775808 is out of range for mov/mp4 format" try to reproduce by: ffmpeg -y -f lavfi -i color -metadata "timecode=00:00:00:00" -t 1 test.mov Note although error message is printed, the timecode packet will be written anyway. So the patch 2/2 will try to change the log level to warning. The first two test case of fate-lavf-ismv have timecode setting, so the crc of ref data is different. Fixes ticket #9488 Signed-off-by: Limin Wang <lance.lmwang@gmail.com> --- libavformat/movenc.c | 2 ++ tests/ref/lavf/ismv | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 4c86891..74b94cd 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -6383,6 +6383,8 @@ static int mov_create_timecode_track(AVFormatContext *s, int index, int src_inde pkt->data = data; pkt->stream_index = index; pkt->flags = AV_PKT_FLAG_KEY; + pkt->pts = pkt->dts = av_rescale_q(tc.start, av_inv_q(rate), (AVRational){1,mov->movie_timescale}); + pkt->duration = av_rescale_q(1, av_inv_q(rate), (AVRational){1,mov->movie_timescale}); pkt->size = 4; AV_WB32(pkt->data, tc.start); ret = ff_mov_write_packet(s, pkt); diff --git a/tests/ref/lavf/ismv b/tests/ref/lavf/ismv index ac7f72b..723b432 100644 --- a/tests/ref/lavf/ismv +++ b/tests/ref/lavf/ismv @@ -1,7 +1,7 @@ -48fb8d7a5d19bd60f3a49ccf4b7d6593 *tests/data/lavf/lavf.ismv +7a24b73c096ec0f13f0f7a2d9101c4c1 *tests/data/lavf/lavf.ismv 313169 tests/data/lavf/lavf.ismv tests/data/lavf/lavf.ismv CRC=0x9d9a638a -d19cd8e310a2e94fe0a0d11c5dc29217 *tests/data/lavf/lavf.ismv +79646383fd099d45ad0d0c2791c601dd *tests/data/lavf/lavf.ismv 322075 tests/data/lavf/lavf.ismv tests/data/lavf/lavf.ismv CRC=0xe8130120 3b6023766845b51b075aed474c00f73c *tests/data/lavf/lavf.ismv -- 1.8.3.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".
next reply other threads:[~2022-03-02 13:58 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-03-02 13:58 lance.lmwang [this message] 2022-03-02 13:58 ` [FFmpeg-devel] [PATCH 2/2] avformat/movenc: use warning log level and small adjustment for the log lance.lmwang 2022-03-03 1:55 ` [FFmpeg-devel] [PATCH 1/2] avformat/movenc: initialize pts/dts/duration of timecode packet Andreas Rheinhardt 2022-03-03 4:57 ` lance.lmwang 2022-03-11 13:58 ` lance.lmwang 2022-03-11 14:04 ` Andreas Rheinhardt 2022-03-11 15:58 ` lance.lmwang 2022-03-11 16:16 ` Andreas Rheinhardt 2022-03-12 0:07 ` lance.lmwang 2022-03-22 12:37 ` lance.lmwang 2022-03-22 13:28 ` Andreas Rheinhardt 2022-03-22 14:32 ` lance.lmwang 2022-03-14 15:36 ` [FFmpeg-devel] [PATCH v2 1/2] avformat/movenc: initialize pts/dts " lance.lmwang 2022-03-14 15:36 ` [FFmpeg-devel] [PATCH v2 2/2] avformat/movenc: use warning log level and small adjustment for the log lance.lmwang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1646229512-12103-1-git-send-email-lance.lmwang@gmail.com \ --to=lance.lmwang@gmail.com \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git