Browse Source

ffprobe: fix potential NULL pointer dereference

Found by Coverity, should fix CID 733741.
tags/n1.1
Stefano Sabatini 13 years ago
parent
commit
29d46d7bce
1 changed files with 2 additions and 2 deletions
  1. +2
    -2
      ffprobe.c

+ 2
- 2
ffprobe.c View File

@@ -1095,13 +1095,13 @@ static void json_print_section_header(WriterContext *wctx)
json->indent_level++; json->indent_level++;
if (section->flags & SECTION_FLAG_IS_ARRAY) { if (section->flags & SECTION_FLAG_IS_ARRAY) {
printf("\"%s\": [\n", buf.str); printf("\"%s\": [\n", buf.str);
} else if (!(parent_section->flags & SECTION_FLAG_IS_ARRAY)) {
} else if (parent_section && !(parent_section->flags & SECTION_FLAG_IS_ARRAY)) {
printf("\"%s\": {%s", buf.str, json->item_start_end); printf("\"%s\": {%s", buf.str, json->item_start_end);
} else { } else {
printf("{%s", json->item_start_end); printf("{%s", json->item_start_end);


/* this is required so the parser can distinguish between packets and frames */ /* this is required so the parser can distinguish between packets and frames */
if (parent_section->id == SECTION_ID_PACKETS_AND_FRAMES) {
if (parent_section && parent_section->id == SECTION_ID_PACKETS_AND_FRAMES) {
if (!json->compact) if (!json->compact)
JSON_INDENT(); JSON_INDENT();
printf("\"type\": \"%s\"%s", section->name, json->item_sep); printf("\"type\": \"%s\"%s", section->name, json->item_sep);


Loading…
Cancel
Save