On Tue, 2020-10-06 at 16:47 +0300, Dan Carpenter wrote:
> After a list_for_each_entry() loop, the list iterator is always non-NULL
> so these conditions don't work. If the "waiter" is not found then this
> results in an out of bounds access.
>
> I have fixed it by introducing a new "found" variable. In one case, I
> used an else statement for readability.
>
> Fixes: 46e4b9ec4fa4 ("staging: vchiq_arm: use list_for_each_entry when accessing bulk_waiter_list")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> ---
Reviewed-by: Nicolas Saenz Julienne <nsaenzjulienne@xxxxxxx>
Attachment:
signature.asc
Description: This is a digitally signed message part
