[FFmpeg-trac] #183(avformat:new): ASF/WMV partial packet change causes infinite loop

[FFmpeg]

#183: ASF/WMV partial packet change causes infinite loop
-------------------------+---------------------
 Reporter:  tracey_pooh  |        Type:  defect
   Status:  new          |    Priority:  normal
Component:  avformat     |     Version:  git
 Keywords:  ASF demuxer  |  Blocked By:
 Blocking:               |  Reproduced:  0
 Analyzed:  0            |
-------------------------+---------------------
 hi all,

 I'm new to posting bugs with FFmpeg, but have been using ffmpeg heavily
 for years, even customizing and patching it.  At some point, I'd like to
 try to work with folks to see about adding some patches.

 OK, back to the "bug" -- granted this is a likely *heavily* poorly (and
 cutoff) video, but it's representative at times of the random stuff we get
 in our 500,000+ videos are Internet Archive / archive.org 8-)

 The patch here:

 http://git.videolan.org/?p=ffmpeg.git;a=blobdiff;f=libavformat/asfdec.c;h=36876fa1fd053a27a7218908dc2881ebbd298bd6;hp=843b10951127246ddbee59c811ee46311e682f01;hb=bf09a0198121467b5a79ba03363b0d1e4dbc3e58;hpb=d09b5a869fea0de5c2f43c547f8ccba8f4740293

 causes ffmpeg to spin and write (seemingly) without end -- I have to kill
 it.

 A one-line "revert" for part of that patch will make it stop spinning and
 just kind of cut off the audio like it used to,
 but it'd be nice if the original diff provider / moderator to the longer
 change (above) could take a look and see what
 they think might be best:

        if (ret != asf->packet_frag_size) {
           av_log(s, AV_LOG_ERROR, "xxxx scrambled\n"); return ret >= 0 ?
 AVERROR_EOF : ret;

 (that is, inserting that one line "fixes my problem" but likely the
 greater patch had other intentions to it).

 Example: run
 ffmpeg   -i holymatrimony.wmv  -vn    audio.wav

-- 
Ticket URL: <https://ffmpeg.org/trac/ffmpeg/ticket/183>
FFmpeg <http://ffmpeg.org>
FFmpeg issue tracker