Commit 7ee8a1c5 authored by Michael Niedermayer's avatar Michael Niedermayer

Merge commit '9a026c72'

* commit '9a026c72':
  h264: rebuild the default ref list if the reference count changes

Conflicts:
	libavcodec/h264.c
Merged-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
parents a4f8a564 9a026c72
...@@ -3382,11 +3382,12 @@ static int h264_slice_header_init(H264Context *h, int reinit) ...@@ -3382,11 +3382,12 @@ static int h264_slice_header_init(H264Context *h, int reinit)
int ff_set_ref_count(H264Context *h) int ff_set_ref_count(H264Context *h)
{ {
int ref_count[2], list_count;
int num_ref_idx_active_override_flag; int num_ref_idx_active_override_flag;
// set defaults, might be overridden a few lines later // set defaults, might be overridden a few lines later
h->ref_count[0] = h->pps.ref_count[0]; ref_count[0] = h->pps.ref_count[0];
h->ref_count[1] = h->pps.ref_count[1]; ref_count[1] = h->pps.ref_count[1];
if (h->slice_type_nos != AV_PICTURE_TYPE_I) { if (h->slice_type_nos != AV_PICTURE_TYPE_I) {
unsigned max[2]; unsigned max[2];
...@@ -3397,27 +3398,36 @@ int ff_set_ref_count(H264Context *h) ...@@ -3397,27 +3398,36 @@ int ff_set_ref_count(H264Context *h)
num_ref_idx_active_override_flag = get_bits1(&h->gb); num_ref_idx_active_override_flag = get_bits1(&h->gb);
if (num_ref_idx_active_override_flag) { if (num_ref_idx_active_override_flag) {
h->ref_count[0] = get_ue_golomb(&h->gb) + 1; ref_count[0] = get_ue_golomb(&h->gb) + 1;
if (h->slice_type_nos == AV_PICTURE_TYPE_B) { if (h->slice_type_nos == AV_PICTURE_TYPE_B) {
h->ref_count[1] = get_ue_golomb(&h->gb) + 1; ref_count[1] = get_ue_golomb(&h->gb) + 1;
} else } else
// full range is spec-ok in this case, even for frames // full range is spec-ok in this case, even for frames
h->ref_count[1] = 1; ref_count[1] = 1;
} }
if (h->ref_count[0]-1 > max[0] || h->ref_count[1]-1 > max[1]){ if (ref_count[0]-1 > max[0] || ref_count[1]-1 > max[1]){
av_log(h->avctx, AV_LOG_ERROR, "reference overflow %u > %u or %u > %u\n", h->ref_count[0]-1, max[0], h->ref_count[1]-1, max[1]); av_log(h->avctx, AV_LOG_ERROR, "reference overflow %u > %u or %u > %u\n", ref_count[0]-1, max[0], ref_count[1]-1, max[1]);
h->ref_count[0] = h->ref_count[1] = 0; h->ref_count[0] = h->ref_count[1] = 0;
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (h->slice_type_nos == AV_PICTURE_TYPE_B) if (h->slice_type_nos == AV_PICTURE_TYPE_B)
h->list_count = 2; list_count = 2;
else else
h->list_count = 1; list_count = 1;
} else { } else {
h->list_count = 0; list_count = 0;
h->ref_count[0] = h->ref_count[1] = 0; ref_count[0] = ref_count[1] = 0;
}
if (list_count != h->list_count ||
ref_count[0] != h->ref_count[0] ||
ref_count[1] != h->ref_count[1]) {
h->ref_count[0] = ref_count[0];
h->ref_count[1] = ref_count[1];
h->list_count = list_count;
return 1;
} }
return 0; return 0;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment