[FFmpeg-devel] [PATCH] lavf/mp3: Properly check return values of seeks and reads while reading the header

wm4 nfxjfg at googlemail.com
Fri Feb 26 18:52:45 CET 2016


On Fri, 26 Feb 2016 17:32:36 +0000
Derek Buitenhuis <derek.buitenhuis at gmail.com> wrote:

> On 2/26/2016 5:30 PM, wm4 wrote:
> > The ffio_ensure_seekback() checks are probably unnecessary.  
> 
> I can remove them if you prefer. Should I?

I'd vote yet, but that's just my personal opinion.

> > Seems a bit verbose, but ok.  
> 
> I added the av_log stuff because otherwise you just get "Invalid Argument"
> from the ffmpeg cli, with no other explanation.

OK then.


More information about the ffmpeg-devel mailing list