diff options
| author | Anton Khirnov <anton@khirnov.net> | 2016-03-21 16:50:11 +0100 |
|---|---|---|
| committer | Anton Khirnov <anton@khirnov.net> | 2016-04-24 10:06:23 +0200 |
| commit | 71d3305c2711d4f6ec8b92db09ff64cf4e19a58e (patch) | |
| tree | 4ff64d1b03b31d5f8b19311e27bca0cd9bd148cd /libavcodec | |
| parent | a6e27f7add2698fdd89911632b570c3d0c3f2aaa (diff) | |
| download | ffmpeg-71d3305c2711d4f6ec8b92db09ff64cf4e19a58e.tar.gz | |
h264_parse: make sure the ref count is zeroed on all failure paths
Diffstat (limited to 'libavcodec')
| -rw-r--r-- | libavcodec/h264_parse.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/libavcodec/h264_parse.c b/libavcodec/h264_parse.c index 6230d677b5..bd1a50ec17 100644 --- a/libavcodec/h264_parse.c +++ b/libavcodec/h264_parse.c @@ -194,11 +194,11 @@ int ff_h264_parse_ref_count(int *plist_count, int ref_count[2], if (num_ref_idx_active_override_flag) { ref_count[0] = get_ue_golomb(gb) + 1; if (ref_count[0] < 1) - return AVERROR_INVALIDDATA; + goto fail; if (slice_type_nos == AV_PICTURE_TYPE_B) { ref_count[1] = get_ue_golomb(gb) + 1; if (ref_count[1] < 1) - return AVERROR_INVALIDDATA; + goto fail; } } @@ -213,12 +213,15 @@ int ff_h264_parse_ref_count(int *plist_count, int ref_count[2], max_refs = picture_structure == PICT_FRAME ? 16 : 32; - if (ref_count[0] > max_refs || ref_count[1] > max_refs) { - ref_count[0] = ref_count[1] = 0; - return AVERROR_INVALIDDATA; - } + if (ref_count[0] > max_refs || ref_count[1] > max_refs) + goto fail; *plist_count = list_count; return 0; +fail: + *plist_count = 0; + ref_count[0] = 0; + ref_count[1] = 0; + return AVERROR_INVALIDDATA; } |