crypto: nitrox - fix unchecked variable in nitrox_register_interrupts
authorTong Tiangen <tongtiangen@huawei.com>
Tue, 1 Jun 2021 10:01:55 +0000 (18:01 +0800)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 11 Jun 2021 07:02:14 +0000 (15:02 +0800)
Function nitrox_register_interrupts leaves variable 'nr_vecs' unchecked, which
would be use as kcalloc parameter later.

Fixes: 5155e118dda9 ("crypto: cavium/nitrox - use pci_alloc_irq_vectors() while enabling MSI-X.")
Signed-off-by: Tong Tiangen <tongtiangen@huawei.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/cavium/nitrox/nitrox_isr.c

index c288c4b51783da60b2a188b44fe8ef7b4c53ae4e..f19e520da6d0ca1c2e08c8db25e3e9bf32965ba5 100644 (file)
@@ -307,6 +307,10 @@ int nitrox_register_interrupts(struct nitrox_device *ndev)
         * Entry 192: NPS_CORE_INT_ACTIVE
         */
        nr_vecs = pci_msix_vec_count(pdev);
+       if (nr_vecs < 0) {
+               dev_err(DEV(ndev), "Error in getting vec count %d\n", nr_vecs);
+               return nr_vecs;
+       }
 
        /* Enable MSI-X */
        ret = pci_alloc_irq_vectors(pdev, nr_vecs, nr_vecs, PCI_IRQ_MSIX);