]> git.sesse.net Git - ffmpeg/commitdiff
movenc: Use first H264/HEVC frame as extradata, if it is missing
authorMartin Storsjö <martin@martin.st>
Fri, 8 May 2020 11:21:32 +0000 (14:21 +0300)
committerMartin Storsjö <martin@martin.st>
Wed, 20 May 2020 19:55:00 +0000 (22:55 +0300)
Sticking a full frame in the extradata works, as the code for writing
the avcC/hvcC extracts the relevant parameter set NAL units - provided
that they actually exist in the frame.

Some encoders don't provide split out extradata directly on init (or
at all). In particular, the MediaFoundation encoder wrapper doesn't
always (depending on the actual encoder device) - this is the case for
Qualcomm's HEVC encoder on SD835, and also on some QSV H264 encoders).

This only works for cases where the moov hasn't already been written
(e.g. when not writing fragmented mp4 with empty_moov, unless using
the delay_moov option).

Signed-off-by: Martin Storsjö <martin@martin.st>
libavformat/movenc.c

index 32e81092687439c8b91e918bb614654a5c6670d8..e6b3ea7ef14a4751d7824a53488237decd2d504f 100644 (file)
@@ -5583,6 +5583,8 @@ int ff_mov_write_packet(AVFormatContext *s, AVPacket *pkt)
     }
 
     if ((par->codec_id == AV_CODEC_ID_DNXHD ||
+         par->codec_id == AV_CODEC_ID_H264 ||
+         par->codec_id == AV_CODEC_ID_HEVC ||
          par->codec_id == AV_CODEC_ID_TRUEHD ||
          par->codec_id == AV_CODEC_ID_AC3) && !trk->vos_len) {
         /* copy frame to create needed atoms */