aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFrank Plowman <post@frankplowman.com>2024-02-25 22:30:42 +0000
committerJames Almer <jamrial@gmail.com>2024-02-25 19:55:01 -0300
commit8b6219a99d80cabf87c50170c009fe93092e32bd (patch)
tree66bcda4e1040da56c0398a5323fe72ff63365be6
parent53ab7ff67e7ee9e7cae5cb0449203a7951cbe029 (diff)
downloadffmpeg-8b6219a99d80cabf87c50170c009fe93092e32bd.tar.gz
lavc/vvc: Read subpic ID when only one subpicture is present
Previously, the following syntax elements were not read in the case sps_num_subpics_minus is 0: * sps_subpic_id_len_minus1 * sps_subpic_id_mapping_explicitly_signalled_flag * sps_subpic_id_mapping_present_flag * sps_subpic_id[i] This was causing failures to decode bitstreams, for example the DVB's "VVC HDR UHDTV1 OpenGOP 3840x2160 50fps HLG10 PiP" V&V bitstream. Patch fixes this by moving the reads for these syntax elements out a scope. Signed-off-by: Frank Plowman <post@frankplowman.com> Signed-off-by: James Almer <jamrial@gmail.com>
-rw-r--r--libavcodec/cbs_h266_syntax_template.c36
1 files changed, 18 insertions, 18 deletions
diff --git a/libavcodec/cbs_h266_syntax_template.c b/libavcodec/cbs_h266_syntax_template.c
index 9cb2d6553a..49fb12ba77 100644
--- a/libavcodec/cbs_h266_syntax_template.c
+++ b/libavcodec/cbs_h266_syntax_template.c
@@ -1213,30 +1213,30 @@ static int FUNC(sps)(CodedBitstreamContext *ctx, RWContext *rw,
infer(sps_loop_filter_across_subpic_enabled_flag[i], 0);
}
}
- ue(sps_subpic_id_len_minus1, 0, 15);
- if ((1 << (current->sps_subpic_id_len_minus1 + 1)) <
- current->sps_num_subpics_minus1 + 1) {
- av_log(ctx->log_ctx, AV_LOG_ERROR,
- "sps_subpic_id_len_minus1(%d) is too small\n",
- current->sps_subpic_id_len_minus1);
- return AVERROR_INVALIDDATA;
- }
- flag(sps_subpic_id_mapping_explicitly_signalled_flag);
- if (current->sps_subpic_id_mapping_explicitly_signalled_flag) {
- flag(sps_subpic_id_mapping_present_flag);
- if (current->sps_subpic_id_mapping_present_flag) {
- for (i = 0; i <= current->sps_num_subpics_minus1; i++) {
- ubs(current->sps_subpic_id_len_minus1 + 1,
- sps_subpic_id[i], 1, i);
- }
- }
- }
} else {
infer(sps_subpic_ctu_top_left_x[0], 0);
infer(sps_subpic_ctu_top_left_y[0], 0);
infer(sps_subpic_width_minus1[0], tmp_width_val - 1);
infer(sps_subpic_height_minus1[0], tmp_height_val - 1);
}
+ ue(sps_subpic_id_len_minus1, 0, 15);
+ if ((1 << (current->sps_subpic_id_len_minus1 + 1)) <
+ current->sps_num_subpics_minus1 + 1) {
+ av_log(ctx->log_ctx, AV_LOG_ERROR,
+ "sps_subpic_id_len_minus1(%d) is too small\n",
+ current->sps_subpic_id_len_minus1);
+ return AVERROR_INVALIDDATA;
+ }
+ flag(sps_subpic_id_mapping_explicitly_signalled_flag);
+ if (current->sps_subpic_id_mapping_explicitly_signalled_flag) {
+ flag(sps_subpic_id_mapping_present_flag);
+ if (current->sps_subpic_id_mapping_present_flag) {
+ for (i = 0; i <= current->sps_num_subpics_minus1; i++) {
+ ubs(current->sps_subpic_id_len_minus1 + 1,
+ sps_subpic_id[i], 1, i);
+ }
+ }
+ }
} else {
infer(sps_num_subpics_minus1, 0);
infer(sps_independent_subpics_flag, 1);