[FFmpeg-trac] #5047(avformat:open): Regression in mp3 audio stream handling

[FFmpeg]

#5047: Regression in mp3 audio stream handling
-------------------------------------+-------------------------------------
             Reporter:  fritsch      |                    Owner:
                 Type:  defect       |                   Status:  open
             Priority:  important    |                Component:  avformat
              Version:  git-master   |               Resolution:
             Keywords:  mov mp3      |               Blocked By:
  regression                         |  Reproduced by developer:  1
             Blocking:               |
Analyzed by developer:  0            |
-------------------------------------+-------------------------------------

Comment (by fritsch):

 fritsch at t440s:~/Desktop/FFmpeg$ git bisect bad
 6ec688e1bc76dd93151cbca1c340162ae4b10d77 is the first bad commit
 commit 6ec688e1bc76dd93151cbca1c340162ae4b10d77
 Author: nu774 <honeycomb77 at gmail.com>
 Date:   Fri May 9 23:05:42 2014 +0900

     mp3: enable packed main_data decoding in MP4

     14496-3 suggests packing main_data of MP3 that is usually scattered
     into multiple frames due to bit reservoir.

     However, after packing main_data into a access unit, bitrate index
     in the MPEG audio frame header doesn't match with actual frame size.

     In order to accept this, this patch removes unnecessary frame size
     checking on mp3 decoder.

     Also, mov demuxer was changed to use MP3 parser only on special cases
     (QT MOV with specific sample description) to avoid re-packetizing.

     Signed-off-by: Luca Barbato <lu_zero at gentoo.org>

 :040000 040000 857f53268fa5c5a994fe27c3b3beb91cae3e5320
 43810c8e18abbccb618dd9057837de4d7a8e1fe3 M      libavcodec
 :040000 040000 2f6dfb3868f684ca94a575da5ef89176d1fd20e8
 37eefdc905b277f7efd727a4529d8276840b970d M      libavformat
 :040000 040000 eac8ef6d85c3818ebec2d8c573a99eaa9c234ce4
 f9f33d9633a9d44873f7842b2a1eadd84d1714e4 M      tests

--
Ticket URL: <https://trac.ffmpeg.org/ticket/5047#comment:4>
FFmpeg <https://ffmpeg.org>
FFmpeg issue tracker