OSDN Git Service

scsi: elx: libefc: Prefer kcalloc() over open coded arithmetic
authorLen Baker <len.baker@gmx.com>
Sun, 5 Sep 2021 06:24:48 +0000 (08:24 +0200)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 15 Sep 2021 03:33:22 +0000 (23:33 -0400)
commit0a5e20fc8ca7ed10b8865421a7c1e8d460840956
tree271e0cb65a2f40b5395fecad77e1f2b31203f56b
parent0d6b26795bd2b9f1d7f0d43a21a386438586bc69
scsi: elx: libefc: Prefer kcalloc() over open coded arithmetic

As noted in the "Deprecated Interfaces, Language Features, Attributes, and
Conventions" documentation [1], size calculations (especially
multiplication) should not be performed in memory allocator (or similar)
function arguments due to the risk of them overflowing. This could lead to
values wrapping around and a smaller allocation being made than the caller
was expecting. Using those allocations could lead to linear overflows of
heap memory and other misbehaviors.

Use the purpose specific kcalloc() function instead of the argument count *
size in the kzalloc() function.

[1] https://www.kernel.org/doc/html/v5.14/process/deprecated.html#open-coded-arithmetic-in-allocator-arguments

Link: https://lore.kernel.org/r/20210905062448.6587-1-len.baker@gmx.com
Reviewed-by: James Smart <jsmart2021@gmail.com>
Signed-off-by: Len Baker <len.baker@gmx.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/elx/libefc/efc_fabric.c