[FFmpeg-cvslog] lavf/segment: remove silly assert(), improve feedback

Stefano Sabatini git at videolan.org
Sun Dec 9 20:47:40 CET 2012


ffmpeg | branch: master | Stefano Sabatini <stefasab at gmail.com> | Sun Dec  9 20:39:27 2012 +0100| [ad47ac20ae3e8ac52fa23f6fa520a3124cc515cd] | committer: Stefano Sabatini

lavf/segment: remove silly assert(), improve feedback

The code should not assert in case of invalid user input. Fails
gracefully instead.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=ad47ac20ae3e8ac52fa23f6fa520a3124cc515cd
---

 libavformat/segment.c |   10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/libavformat/segment.c b/libavformat/segment.c
index 1ad410e..15c87db 100644
--- a/libavformat/segment.c
+++ b/libavformat/segment.c
@@ -29,7 +29,6 @@
 #include "avformat.h"
 #include "internal.h"
 
-#include "libavutil/avassert.h"
 #include "libavutil/log.h"
 #include "libavutil/opt.h"
 #include "libavutil/avstring.h"
@@ -274,13 +273,18 @@ static int parse_times(void *log_ctx, int64_t **times, int *nb_times,
     for (i = 0; i < *nb_times; i++) {
         int64_t t;
         char *tstr = av_strtok(p, ",", &saveptr);
-        av_assert0(tstr);
         p = NULL;
 
+        if (!tstr || !tstr[0]) {
+            av_log(log_ctx, AV_LOG_ERROR, "Empty time specification in times list %s\n",
+                   times_str);
+            FAIL(AVERROR(EINVAL));
+        }
+
         ret = av_parse_time(&t, tstr, 1);
         if (ret < 0) {
             av_log(log_ctx, AV_LOG_ERROR,
-                   "Invalid time duration specification in %s\n", p);
+                   "Invalid time duration specification '%s' in times list %s\n", tstr, times_str);
             FAIL(AVERROR(EINVAL));
         }
         (*times)[i] = t;



More information about the ffmpeg-cvslog mailing list