On 23/08/2020 18:31, James Almer wrote:
On 8/23/2020 1:06 PM, Mark Thompson wrote:
The OBU must be in both the spatial and temporal layers for the
operating point, not just one of them.
---
libavcodec/cbs_av1_syntax_template.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/cbs_av1_syntax_template.c
b/libavcodec/cbs_av1_syntax_template.c
index 2c976a9574..a281aba8ac 100644
--- a/libavcodec/cbs_av1_syntax_template.c
+++ b/libavcodec/cbs_av1_syntax_template.c
@@ -1390,7 +1390,7 @@ static int
FUNC(uncompressed_header)(CodedBitstreamContext *ctx, RWContext *rw,
int in_temporal_layer = (op_pt_idc >>
priv->temporal_id ) & 1;
int in_spatial_layer = (op_pt_idc >>
(priv->spatial_id + 8)) & 1;
if (seq->operating_point_idc[i] == 0 ||
- in_temporal_layer || in_spatial_layer) {
+ (in_temporal_layer && in_spatial_layer)) {
fbs(seq->decoder_model_info.buffer_removal_time_length_minus_1 + 1,
buffer_removal_time[i], 1, i);
}
LGTM, can confirm this fixes parsing https://0x0.st/zF2s.ivf
Thank you for the sample! Applied.
- Mark
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
