[FFmpeg-trac] #7188(avformat:new): webm_chunk muxer is broken since commit 18ac64235939c4c5c7656546a9545f68339affbe

FFmpeg trac at avcodec.org
Thu May 3 16:38:15 EEST 2018


#7188: webm_chunk muxer is broken since commit
18ac64235939c4c5c7656546a9545f68339affbe
----------------------------------+--------------------------------------
             Reporter:  octop     |                     Type:  defect
               Status:  new       |                 Priority:  important
            Component:  avformat  |                  Version:  git-master
             Keywords:            |               Blocked By:
             Blocking:            |  Reproduced by developer:  0
Analyzed by developer:  0         |
----------------------------------+--------------------------------------
 The commit 18ac64235939c4c5c7656546a9545f68339affbe introduced a
 regression, making the webm_chunk muxer cease to function properly.

 {{{
 libavformat/webm_chunk.c
 @@ -127,12 +127,12 @@ static int webm_chunk_write_header(AVFormatContext
 *s)
      if (ret < 0)
          return ret;
      oc = wc->avf;
 -    ret = get_chunk_filename(s, 1, oc->filename);
 +    ret = get_chunk_filename(s, 1, oc->url);
      if (ret < 0)
          return ret;
      if (wc->http_method)
          av_dict_set(&options, "method", wc->http_method, 0);
 -    ret = s->io_open(s, &oc->pb, oc->filename, AVIO_FLAG_WRITE,
 &options);
 +    ret = s->io_open(s, &oc->pb, oc->url, AVIO_FLAG_WRITE, &options);
      av_dict_free(&options);
      if (ret < 0)
          return ret;
 }}}
 In my own testing, I discovered that {{{oc->url}}} in that code evaluates
 to null, making the muxer return prematurely.

 To reproduce:
 {{{ffmpeg -re -f lavfi -i sine -map 0 -strict -2 -c:a opus -f webm_chunk
 -header webm_live_sine.hdr webm_live_sine_%d.chk}}}

--
Ticket URL: <https://trac.ffmpeg.org/ticket/7188>
FFmpeg <https://ffmpeg.org>
FFmpeg issue tracker


More information about the FFmpeg-trac mailing list