Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#186 faad2 crashes on large ID3v2 tags

None
closed-accepted
nobody
None
5
2015-02-11
2012-10-05
Mike Crowe
No

faad2 skips ID3v2 tags on ADTS AAC files. Unfortunately it does so using advance_buffer which causes fill_buffer to misbehave if the ID3v2 tag was larger than the buffer size.

The attached patch fixes advance_buffer to loop around only advancing by the size of the buffer which works around the problem.

Discussion

  • Mike Crowe
    Mike Crowe
    2012-10-05

    Here's a better patch with the asserts removed so it actually compiles.

     
  • Mike Crowe
    Mike Crowe
    2012-10-05

    Improved patch

     
  • Hello,

    is there any reason why this patch hasn't been accepted yet?

    • Fabian
     
  • Just committed it, can be closed.

     
    • status: open --> closed-accepted
    • Group: -->