Re: [PATCH][next] net: aquantia: fix unsigned numvecs comparison with less than zero

From: Igor Russkikh
Date: Thu May 10 2018 - 11:36:42 EST



Thanks, Colin!

> The comparison of numvecs < 0 is always false because numvecs is a u32
> and hence the error return from a failed call to pci_alloc_irq_vectores
> is never detected. Fix this by using the signed int ret to handle the
> error return and assign numvecs to err.
>
> Detected by CoverityScan, CID#1468650 ("Unsigned compared against 0")
>
> Fixes: a09bd81b5413 ("net: aquantia: Limit number of vectors to actually allocated irqs")
> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>
> ---

Acked-by: Igor Russkikh <igor.russkikh@xxxxxxxxxxxx>