[FFmpeg-devel] [PATCH 2/3] lavf: replace all uses of url_fskip with avio_seek

Måns Rullgård mans
Mon Feb 28 15:13:17 CET 2011


Anton Khirnov <anton at khirnov.net> writes:

> @@ -304,7 +304,7 @@ static int flv_read_packet(AVFormatContext *s, AVPacket *pkt)
>      int64_t dts, pts = AV_NOPTS_VALUE;
>      AVStream *st = NULL;
>  
> - for(;;url_fskip(s->pb, 4)){ /* pkt size is repeated at end. skip it */
> + for(;;avio_seek(s->pb, 4, SEEK_CUR) < 0){ /* pkt size is repeated at end. skip it */
>      pos = url_ftell(s->pb);
>      type = avio_r8(s->pb);
>      size = avio_rb24(s->pb);

Sorry, I misread that when skimming the grep output.  That doesn't check
the return value at all.  Feel free to rewrite it in a sane way instead.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list