On 2/5/2025 12:10 PM, Zhao Zhili wrote:
> -    skip_bits1(gb); /* direct_depenency_all_layers_flag */
> -    direct_dep_type = get_bits_long(gb, direct_dep_type_len);
> -    if (direct_dep_type > HEVC_DEP_TYPE_BOTH) {
> -        av_log(avctx, AV_LOG_WARNING, "Unsupported direct_dep_type: %d\n",
> -               direct_dep_type);
> -        return AVERROR_PATCHWELCOME;
> +    /* direct_depenency_all_layers_flag */
> +    if (get_bits1(gb)) {
> +        direct_dep_type = get_bits_long(gb, direct_dep_type_len);
> +        if (direct_dep_type > HEVC_DEP_TYPE_BOTH) {
> +            av_log(avctx, AV_LOG_WARNING, "Unsupported direct_dep_type: %d\n",
> +                   direct_dep_type);
> +            return AVERROR_PATCHWELCOME;
> +        }
>       }

direct_dep_type is also coded if direct_depenency_all_layers_flag is 
false when (if I'm reading the spec right) vps->num_direct_ref_layers[1] 
is not 0.