[FFmpeg-cvslog] avformat/dashenc: Remove muxer overhead from Bandwidth field in DASH manifest

Karthick Jeyapal git at videolan.org
Tue Sep 11 15:31:36 EEST 2018


ffmpeg | branch: master | Karthick Jeyapal <kjeyapal at akamai.com> | Tue Sep  4 14:03:48 2018 +0530| [44ee858b6df0f3f350abd56bd18340c1d8d94ee8] | committer: Karthick Jeyapal

avformat/dashenc: Remove muxer overhead from Bandwidth field in DASH manifest

Fixes bug id #7386
Muxer overhead calculations was intented for HLS playlist as Apple's mediastreamvalidator tests were failing.
But applying the same fix for DASH manifest proved counterproductive, as Bandwidth can be used for segment name templates.

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

 libavformat/dashenc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c
index d8b4230850..87e31e25fc 100644
--- a/libavformat/dashenc.c
+++ b/libavformat/dashenc.c
@@ -611,7 +611,7 @@ static int write_adaptation_set(AVFormatContext *s, AVIOContext *out, int as_ind
 
         if (os->bit_rate > 0)
             snprintf(bandwidth_str, sizeof(bandwidth_str), " bandwidth=\"%d\"",
-                     os->bit_rate + os->muxer_overhead);
+                     os->bit_rate);
 
         if (as->media_type == AVMEDIA_TYPE_VIDEO) {
             AVStream *st = s->streams[i];



More information about the ffmpeg-cvslog mailing list