| From 08575947727d4916b997f47577ee3feffa23eeb9 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Tue, 1 Jun 2021 18:01:55 +0800 |
| Subject: crypto: nitrox - fix unchecked variable in nitrox_register_interrupts |
| |
| From: Tong Tiangen <tongtiangen@huawei.com> |
| |
| [ Upstream commit 57c126661f50b884d3812e7db6e00f2e778eccfb ] |
| |
| 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> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| drivers/crypto/cavium/nitrox/nitrox_isr.c | 4 ++++ |
| 1 file changed, 4 insertions(+) |
| |
| diff --git a/drivers/crypto/cavium/nitrox/nitrox_isr.c b/drivers/crypto/cavium/nitrox/nitrox_isr.c |
| index 3dec570a190a..10e3408bf704 100644 |
| --- a/drivers/crypto/cavium/nitrox/nitrox_isr.c |
| +++ b/drivers/crypto/cavium/nitrox/nitrox_isr.c |
| @@ -306,6 +306,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); |
| -- |
| 2.30.2 |
| |