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.carpen...@oracle.com>
> ---

Reviewed-by: Nicolas Saenz Julienne <nsaenzjulie...@suse.de>

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

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to