]> git.sesse.net Git - ffmpeg/commitdiff
avformat/hlsenc: fix default AES key file url with variant streams
authorBela Bodecs <bodecsb@vivanet.hu>
Sun, 19 Jan 2020 22:01:32 +0000 (23:01 +0100)
committerSteven Liu <lq@chinaffmpeg.org>
Mon, 20 Jan 2020 06:34:30 +0000 (14:34 +0800)
Currently when hls_enc is active and there are multiple variant stream
outputs, default key file url construction does not work, because it is
based on the FormatContext' url field. But in case of multiple variant
streams, it contains the variant m3u8 output playlist url that contains
the %v placeholder. So the result key file url will hold the %v
placeholder causing run time error message about "could not write the
key file".
This patch correct this behaviour, and use the master playlist url for
constructing the output key file url when master playlist is vailable.

Signed-off-by: Bela Bodecs <bodecsb@vivanet.hu>
libavformat/hlsenc.c

index 13f34197eddad3158fdf93679f92cb19981f80d2..2b3d3742d99e065d55e6c64b84f2de4f41f707be 100644 (file)
@@ -642,13 +642,14 @@ static int do_encrypt(AVFormatContext *s, VariantStream *vs)
     int len;
     AVIOContext *pb;
     uint8_t key[KEYSIZE];
+    char * key_basename_source = (hls->master_m3u8_url) ? hls->master_m3u8_url : s->url;
 
-    len = strlen(s->url) + 4 + 1;
+    len = strlen(key_basename_source) + 4 + 1;
     hls->key_basename = av_mallocz(len);
     if (!hls->key_basename)
         return AVERROR(ENOMEM);
 
-    av_strlcpy(hls->key_basename, s->url, len);
+    av_strlcpy(hls->key_basename, key_basename_source, len);
     av_strlcat(hls->key_basename, ".key", len);
 
     if (hls->key_url) {