As said on IRC, i missed the fact that Mpeg4GAProgramConfigElement is called only for LOAS. there's nothing for ADTS.<br>So the patch is wrong, code must be added to read channel config in case of adts.<br><br><br><div class="gmail_quote">
On Tue, Sep 11, 2012 at 4:26 PM, Rafaël Carré <span dir="ltr"><<a href="mailto:funman@videolan.org" target="_blank">funman@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
Le 11/09/2012 14:29, XilasZ a écrit :<br>
<div class="im">> A value of 0 for channel config in ADTS header lead to "Invalid ADTS<br>
> header", preventing sound playback on some streams.<br>
><br>
> But it is a valid value (cf <a href="http://wiki.multimedia.cx/index.php?title=ADTS" target="_blank">http://wiki.multimedia.cx/index.php?title=ADTS</a>).<br>
> 0 means that channel config is not specified in the header, but in another<br>
> structure, following the header.<br>
<br>
</div>Nice work!<br>
<br>
Does it fix a bug in trac?<br>
<br>
If yes please add "Close #xxxx" to the log so when committed the bug<br>
will be closed automatically.<br>
<div class="im"><br>
> Besides, in mpeg4audio.c, in case of 0, Mpeg4GAProgramConfigElement is<br>
> called to find the channel config from this structure.<br>
<br>
</div>That function still has "/* TODO compute channels count ? */" though,<br>
should we fix that TODO before committing that patch?<br>
<div class="im"><br>
> So the check on the header is wrong, this patch fix it.<br>
</div>_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</blockquote></div><br>