Re: [PATCH] media: cedrus: skip invalid H.264 reference list entries

From: Nicolas Dufresne

Date: Thu Apr 09 2026 - 10:01:24 EST


Le jeudi 09 avril 2026 à 22:30 +0800, Pengpeng Hou a écrit :
> Hi Paul,
>
> Thanks, that makes sense.
>
> I agree Cedrus should not silently skip an invalid index in the active
> portion of ref_pic_list0/ref_pic_list1.
>
> I'll respin this to keep the check at the Cedrus use site rather than
> cedrus_try_ctrl(), but return -EINVAL when one of the active reference
> entries points past V4L2_H264_NUM_DPB_ENTRIES. Entries beyond
> num_ref_idx_l0_active_minus1 / num_ref_idx_l1_active_minus1 will still
> be ignored as before.

Please, let the discussion continue before respinning.

Nicolas

>
> Thanks,
> Pengpeng
>

Attachment: signature.asc
Description: This is a digitally signed message part