lavfi/movie: return proper error code in case of av_get_token() allocation failure
authorStefano Sabatini <stefasab@gmail.com>
Tue, 6 Nov 2012 22:28:43 +0000 (23:28 +0100)
committerStefano Sabatini <stefasab@gmail.com>
Tue, 6 Nov 2012 22:31:16 +0000 (23:31 +0100)
Also slightly clarify logic, and should fix coverity issue CID 717771.

libavfilter/src_movie.c

index 8ebad6d..87b6d2a 100644 (file)
@@ -197,9 +197,12 @@ static av_cold int movie_common_init(AVFilterContext *ctx, const char *args, con
     movie->class = class;
     av_opt_set_defaults(movie);
 
-    if (args)
+    if (args) {
         movie->file_name = av_get_token(&args, ":");
-    if (!movie->file_name || !*movie->file_name) {
+        if (!movie->file_name)
+            return AVERROR(ENOMEM);
+    }
+    if (!args || !*movie->file_name) {
         av_log(ctx, AV_LOG_ERROR, "No filename provided!\n");
         return AVERROR(EINVAL);
     }