Fix segfault with -filters option
authorBobby Bingham <uhmmmm@gmail.com>
Wed, 19 Sep 2012 04:27:43 +0000 (23:27 -0500)
committerBobby Bingham <uhmmmm@gmail.com>
Wed, 19 Sep 2012 04:27:43 +0000 (23:27 -0500)
Filters now use null pointers to indicate having no input/output pads,
rather than empty lists of pads. We can't assume pad is non-null
anymore.

Signed-off-by: Bobby Bingham <uhmmmm@gmail.com>
cmdutils.c

index 3ed143e..bd4ba4a 100644 (file)
@@ -1041,7 +1041,7 @@ int show_filters(void *optctx, const char *opt, const char *arg)
                 *(descr_cur++) = '>';
             }
             pad = i ? (*filter)->outputs : (*filter)->inputs;
-            for (j = 0; pad[j].name; j++) {
+            for (j = 0; pad && pad[j].name; j++) {
                 if (descr_cur >= descr + sizeof(descr) - 4)
                     break;
                 *(descr_cur++) = get_media_type_char(pad[j].type);