[FFmpeg-cvslog] avconv: fix parsing bitstream filters

Anton Khirnov git at videolan.org
Sun Jun 26 20:23:48 CEST 2016


ffmpeg | branch: release/3.1 | Anton Khirnov <anton at khirnov.net> | Tue May 24 16:49:19 2016 +0200| [fe7b21c8f148493c6fbceb7f887a77531dd1ae0e] | committer: Anton Khirnov

avconv: fix parsing bitstream filters

The current code modifies the user-supplied string, which is shared for
the whole output file. So a bitstream filter specification applied to
multiple streams would not work correctly.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=fe7b21c8f148493c6fbceb7f887a77531dd1ae0e
---

 avconv_opt.c |   18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/avconv_opt.c b/avconv_opt.c
index c02e479..b760a18 100644
--- a/avconv_opt.c
+++ b/avconv_opt.c
@@ -941,7 +941,8 @@ static OutputStream *new_output_stream(OptionsContext *o, AVFormatContext *oc, e
     OutputStream *ost;
     AVStream *st = avformat_new_stream(oc, NULL);
     int idx      = oc->nb_streams - 1, ret = 0;
-    char *bsf = NULL, *next, *codec_tag = NULL;
+    const char *bsfs = NULL;
+    char *next, *codec_tag = NULL;
     double qscale = -1;
 
     if (!st) {
@@ -1007,18 +1008,21 @@ static OutputStream *new_output_stream(OptionsContext *o, AVFormatContext *oc, e
     ost->max_frames = INT64_MAX;
     MATCH_PER_STREAM_OPT(max_frames, i64, ost->max_frames, oc, st);
 
-    MATCH_PER_STREAM_OPT(bitstream_filters, str, bsf, oc, st);
-    while (bsf) {
+    MATCH_PER_STREAM_OPT(bitstream_filters, str, bsfs, oc, st);
+    while (bsfs && *bsfs) {
         const AVBitStreamFilter *filter;
+        char *bsf;
 
-        if (next = strchr(bsf, ','))
-            *next++ = 0;
+        bsf = av_get_token(&bsfs, ",");
+        if (!bsf)
+            exit_program(1);
 
         filter = av_bsf_get_by_name(bsf);
         if (!filter) {
             av_log(NULL, AV_LOG_FATAL, "Unknown bitstream filter %s\n", bsf);
             exit_program(1);
         }
+        av_freep(&bsf);
 
         ost->bitstream_filters = av_realloc_array(ost->bitstream_filters,
                                                   ost->nb_bitstream_filters + 1,
@@ -1027,8 +1031,8 @@ static OutputStream *new_output_stream(OptionsContext *o, AVFormatContext *oc, e
             exit_program(1);
 
         ost->bitstream_filters[ost->nb_bitstream_filters++] = filter;
-
-        bsf = next;
+        if (*bsfs)
+            bsfs++;
     }
 
     MATCH_PER_STREAM_OPT(codec_tags, str, codec_tag, oc, st);



More information about the ffmpeg-cvslog mailing list