diff options
author | Colin Ian King <colin.king@canonical.com> | 2020-11-02 13:46:01 +0000 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2020-11-03 17:51:56 -0800 |
commit | 873b807c989598a7db44e7669be3da97eaa8ecde (patch) | |
tree | 3ca68c287fec533aef4a8db797edb3cfe86cd418 /drivers/net/ethernet/marvell | |
parent | 1ca47431c29c63ffc521e17bdc97be5472599776 (diff) | |
download | lwn-873b807c989598a7db44e7669be3da97eaa8ecde.tar.gz lwn-873b807c989598a7db44e7669be3da97eaa8ecde.zip |
octeontx2-pf: Fix sizeof() mismatch
An incorrect sizeof() is being used, sizeof(u64 *) is not correct,
it should be sizeof(*sq->sqb_ptrs).
Addresses-Coverity: ("Sizeof not portable (SIZEOF_MISMATCH)")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20201102134601.698436-1-colin.king@canonical.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'drivers/net/ethernet/marvell')
-rw-r--r-- | drivers/net/ethernet/marvell/octeontx2/nic/otx2_common.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/marvell/octeontx2/nic/otx2_common.c b/drivers/net/ethernet/marvell/octeontx2/nic/otx2_common.c index fc765e86988e..9f3d6715748e 100644 --- a/drivers/net/ethernet/marvell/octeontx2/nic/otx2_common.c +++ b/drivers/net/ethernet/marvell/octeontx2/nic/otx2_common.c @@ -1239,7 +1239,7 @@ int otx2_sq_aura_pool_init(struct otx2_nic *pfvf) sq = &qset->sq[qidx]; sq->sqb_count = 0; - sq->sqb_ptrs = kcalloc(num_sqbs, sizeof(u64 *), GFP_KERNEL); + sq->sqb_ptrs = kcalloc(num_sqbs, sizeof(*sq->sqb_ptrs), GFP_KERNEL); if (!sq->sqb_ptrs) return -ENOMEM; |