scsi: qedf: Replace alloction API and add null check

[PATCH 0/2] scsi: qedf: Replace alloction API and add null check

Jiasheng Jiang posted 2 patches 10 months, 1 week ago

Download series mbox

drivers/scsi/qedf/qedf_io.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)

Expand all Fold all

[PATCH 0/2] scsi: qedf: Replace alloction API and add null check

Posted by Jiasheng Jiang 10 months, 1 week ago

This patch series improves memory safety in the qedf SCSI driver by:

1. Replace kmalloc_array() with kcalloc() to avoid old (dirty) data being
   used/freed.
2. Add a check for "bdt_info". Otherwise, if one of the allocations
   for "cmgr->io_bdt_pool[i]" fails, "bdt_info->bd_tbl" will cause a NULL
   pointer dereference.

### Changelog:
#### v2:
- Replace kzalloc() with kcalloc().

Jiasheng Jiang (2):
  scsi: qedf: Replace kmalloc_array() with kcalloc()
  scsi: qedf: Add check for bdt_info

 drivers/scsi/qedf/qedf_io.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

-- 
2.25.1

Re: [PATCH v5? 0/2] scsi: qedf: Replace alloction API and add null check

Posted by Markus Elfring 10 months, 1 week ago

…
> ### Changelog:
> #### v2:
…

Why did you overlook to increment version numbers once more?
https://lore.kernel.org/all/?q=%22This+looks+like+a+new+version+of+a+previously+submitted+patch%22
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.13#n605

Regards,
Markus