[MPlayer-dev-eng] [PATCH] Duration of VBR-encoded MP3s was wrong

Rich Felker dalias at aerifal.cx
Sun Mar 25 20:40:04 CEST 2007


On Sun, Mar 25, 2007 at 11:34:39AM +0200, Nicolas Bonifas wrote:
> Hello,
> This patch adds support for XING and VBRI headers (in 
> libmpdemux/demux_audio.c), and corrects the bitrate given by mp3lib (in 
> libmpcodecs/ad_mp3lib.c), which does not take VBR into account.
> I tested it against many files from different encoders.
> Hope this helps,
> Nicolas

The idea is welcome, but someone should check this and make sure it
doesn't introduce bugs (infinite loop? etc.) or vulnerabilities with
invalid files before committing it.

Rich



More information about the MPlayer-dev-eng mailing list