From b5bc74240237a35215be6b236a5a67a4143fc62d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Reimar=20D=C3=B6ffinger?= Date: Sat, 10 Jul 2010 23:42:08 +0000 Subject: [PATCH] Check url_fseek results in gxf demuxer. Originally committed as revision 24179 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavformat/gxf.c | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/libavformat/gxf.c b/libavformat/gxf.c index c524db9dc2..af2ee9dea8 100644 --- a/libavformat/gxf.c +++ b/libavformat/gxf.c @@ -397,9 +397,11 @@ start: if (tmp != 1) goto start; last_pos = url_ftell(pb); - url_fseek(pb, -5, SEEK_CUR); + if (url_fseek(pb, -5, SEEK_CUR) < 0) + goto out; if (!parse_packet_header(pb, &type, &len) || type != PKT_MEDIA) { - url_fseek(pb, last_pos, SEEK_SET); + if (url_fseek(pb, last_pos, SEEK_SET) < 0) + goto out; goto start; } get_byte(pb); @@ -407,8 +409,8 @@ start: cur_timestamp = get_be32(pb); last_found_pos = url_ftell(pb) - 16 - 6; if ((track >= 0 && track != cur_track) || (timestamp >= 0 && timestamp > cur_timestamp)) { - url_fseek(pb, last_pos, SEEK_SET); - goto start; + if (url_fseek(pb, last_pos, SEEK_SET) >= 0) + goto start; } out: if (last_found_pos) @@ -477,6 +479,7 @@ static int gxf_packet(AVFormatContext *s, AVPacket *pkt) { } static int gxf_seek(AVFormatContext *s, int stream_index, int64_t timestamp, int flags) { + int res = 0; uint64_t pos; uint64_t maxlen = 100 * 1024 * 1024; AVStream *st = s->streams[0]; @@ -492,7 +495,9 @@ static int gxf_seek(AVFormatContext *s, int stream_index, int64_t timestamp, int if (idx < st->nb_index_entries - 2) maxlen = st->index_entries[idx + 2].pos - pos; maxlen = FFMAX(maxlen, 200 * 1024); - url_fseek(s->pb, pos, SEEK_SET); + res = url_fseek(s->pb, pos, SEEK_SET); + if (res < 0) + return res; found = gxf_resync_media(s, maxlen, -1, timestamp); if (FFABS(found - timestamp) > 4) return -1; @@ -503,7 +508,8 @@ static int64_t gxf_read_timestamp(AVFormatContext *s, int stream_index, int64_t *pos, int64_t pos_limit) { ByteIOContext *pb = s->pb; int64_t res; - url_fseek(pb, *pos, SEEK_SET); + if (url_fseek(pb, *pos, SEEK_SET) < 0) + return AV_NOPTS_VALUE; res = gxf_resync_media(s, pos_limit - *pos, -1, -1); *pos = url_ftell(pb); return res;