# HG changeset patch # User reimar # Date 1278805328 0 # Node ID 63edc8683095ab47be648617bb303240f1d4f213 # Parent e630da0f5861a206886e9f44345706d4eb0d498a Check url_fseek results in gxf demuxer. diff -r e630da0f5861 -r 63edc8683095 gxf.c --- a/gxf.c Sat Jul 10 22:12:30 2010 +0000 +++ b/gxf.c Sat Jul 10 23:42:08 2010 +0000 @@ -397,9 +397,11 @@ 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 @@ 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_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 @@ 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 @@ 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;