Hey Alex, Thanks for the reply. I'm glad that there is no functional issue associated with the warning. It turns out that ffmpeg version of the libavcodec library silences this warning already (Thanks to Carl Eugen for bringing this to my notice). Following is a commit that masks the log. So maybe we need to do something like this in libav.
commit 1ac606bae60e46522189b678b58a7fb8376046ac Author: Reimar Döffinger <[email protected]> Date: Mon Apr 9 21:19:30 2012 +0200 aacsbr: silence message for SBR extension "padding". Some files contain a few additional, all-0 bits. Check for that case and don't print incorrect "not supported" message. Fixes trac issue #836. Signed-off-by: Reimar Döffinger <[email protected]> diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c index 79a0ca2..ea2dcbb 100644 --- a/libavcodec/aacsbr.c +++ b/libavcodec/aacsbr.c @@ -933,7 +933,9 @@ static void read_sbr_extension(AACContext *ac, SpectralBandReplication *sbr, } break; default: - av_log_missing_feature(ac->avctx, "Reserved SBR extensions are", 1); + // some files contain 0-padding + if (bs_extension_id || *num_bits_left > 16 || show_bits(gb, *num_bits_left)) + av_log_missing_feature(ac->avctx, "Reserved SBR extensions are", 1); skip_bits_long(gb, *num_bits_left); // bs_fill_bits *num_bits_left = 0; break; Let me know if you'd like me to upload a patch. Cheers.. Akshay On 22 January 2015 at 12:01, Alex Converse <[email protected]> wrote: > So I capped the stream. Extension ID here is zero which is reserved for > future use. Decoders are supposed to skip extensions that they don't > understand yet, which is what we do. We spam the log to try to get samples > of new extensions being used in the wild. It looks like once upon a time > DRM used a PS draft with that extension ID. > > In this case the unknown extension appears to be padding, which is > unfortunate because if a real extension is ever added here [which seems a > bit unlikely at this point] it will conflict. There are a few places in the > bit stream to add [optional] padding or application specific data and this > isn't one of them. > > It's also possible that the PS decoder isn't reading as many bits as > required but not detecting an error. > > It's also possible that they are unintentionally padding due to an encoder > bug that isn't getting caught because it's writing extra data that decoders > ignore. > > Regardless of this warning the decoder processes all the data it has read > so far including SBR and PS so you aren't getting a quality drop here. We > could silence this warning, or do we have an av_log_once() yet? > > Cheers, > Alex > > > On Wed, Jan 21, 2015 at 11:35 AM, Akshay Garg <[email protected]> wrote: > > > My bad, sorry for posting an mp3 stream. > > > > Please use the following url > > > > http://edge.espn.cdn.abacast.net/espn-network-48 > > > > The issue starts once we get an advertisement in the stream. Usually > within > > 5-10 minutes of the playback start. > > > > I use GST_DEBUG=4 to check the logs. > > > > Thanks > > Akshay > > > > On 21 January 2015 at 11:01, Luca Barbato <[email protected]> wrote: > > > > > On 21/01/15 19:37, Akshay Garg wrote: > > > > Hi All, > > > > > > > > While trying to play the this ESPN Radio Station on Tune in: > > > > > > > > http://edge.espn.cdn.abacast.net/espn-networkmp3-48 > > > > > > > > using the following Gstreamer pipeline > > > > > > > > gst-launch-0.1 soupthttpsrc > > > > location=http://edge.espn.cdn.abacast.net/espn-networkmp3-48 ! > > > > decodebin ! pulsesink > > > > > > > > I keep getting the following warnings: > > > > > > > > "Reserved SBR extensions is not implemented. Update your Libav > version > > > > to the newest one from Git. If the problem still occurs, it means > that > > > > your file has a feature which has not been implemented." > > > > > > > > and eventually the playback stops. I have the latest version of libav > > > > (11.2) installed on my machine. Even the gstreamer (gst-libav) > version > > > > is latest i.e. 1.4.5. > > > > > > > > Looking at the libav aacsbr code it seems like the only extension > > > > that's currently supported is SBR+Parametric Stereo. > > > > > > Right now it is sending mp3 so I can't check what's wrong =/ > > > > > > lu > > > > > > > > > > > > _______________________________________________ > > > libav-devel mailing list > > > [email protected] > > > https://lists.libav.org/mailman/listinfo/libav-devel > > > > > _______________________________________________ > > libav-devel mailing list > > [email protected] > > https://lists.libav.org/mailman/listinfo/libav-devel > > > _______________________________________________ > libav-devel mailing list > [email protected] > https://lists.libav.org/mailman/listinfo/libav-devel > _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
