changeset 5418:58fb1f12d94f libavformat

Calls to url_fseek should have their return value checked in av_seek_frame_binary, just as they do in av_seek_frame_generic. Otherwise, function may return success even though url_fseek reported failure. Patch by Sean Soria, first.last at gmail
author daniel
date Sun, 06 Dec 2009 00:03:53 +0000
parents 9d7de5529047
children 006c95a7a3d2
files utils.c
diffstat 1 files changed, 3 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/utils.c	Fri Dec 04 16:52:42 2009 +0000
+++ b/utils.c	Sun Dec 06 00:03:53 2009 +0000
@@ -1311,6 +1311,7 @@
     int64_t av_uninit(pos_min), av_uninit(pos_max), pos, pos_limit;
     int64_t ts_min, ts_max, ts;
     int index;
+    int64_t ret;
     AVStream *st;
 
     if (stream_index < 0)
@@ -1363,7 +1364,8 @@
         return -1;
 
     /* do the seek */
-    url_fseek(s->pb, pos, SEEK_SET);
+    if ((ret = url_fseek(s->pb, pos, SEEK_SET)) < 0)
+        return ret;
 
     av_update_cur_dts(s, st, ts);