Re: [PATCH v2 1/2] scsi: qedf: Replace kmalloc_array() with kcalloc()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Jiasheng Jiang <jiashengjiangcool@xxxxxxxxx>
Subject: Re: [PATCH v2 1/2] scsi: qedf: Replace kmalloc_array() with kcalloc()
From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
Date: Fri, 7 Feb 2025 16:09:58 +0100
Cc: GR-QLogic-Storage-Upstream@xxxxxxxxxxx, James.Bottomley@xxxxxxxxxxxxxxxxxxxxx, arun.easi@xxxxxxxxxx, bvanassche@xxxxxxx, jhasan@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, manish.rangankar@xxxxxxxxxx, markus.elfring@xxxxxx, martin.petersen@xxxxxxxxxx, nilesh.javali@xxxxxxxxxx, skashyap@xxxxxxxxxxx, stable@xxxxxxxxxxxxxxx
In-reply-to: <20250206192000.17827-1-jiashengjiangcool@gmail.com>

On Thu, Feb 06, 2025 at 07:19:59PM +0000, Jiasheng Jiang wrote:
> Replace kmalloc_array() with kcalloc() to avoid old (dirty) data being
> used/freed.

"Potentially" being freed.  It will not be used.  And this is only for
an error path that obviously no one has hit before.

Please explain this much better.

thanks,

greg k-h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Follow-Ups:
- Re: [PATCH v2 1/2] scsi: qedf: Replace kmalloc_array() with kcalloc()
  - From: Jiasheng Jiang
- [PATCH v3 1/2] scsi: qedf: Replace kmalloc_array() with kcalloc()
  - From: Jiasheng Jiang

References:
- Re: [PATCH 1/2] scsi: qedf: Replace kmalloc_array() with kcalloc()
  - From: Greg KH
- [PATCH v2 1/2] scsi: qedf: Replace kmalloc_array() with kcalloc()
  - From: Jiasheng Jiang

Prev by Date: Re: blktests failures with v6.14-rc1 kernel
Next by Date: [PATCH v3 2/2] scsi: qedf: Add check for bdt_info
Previous by thread: [PATCH v2 2/2] scsi: qedf: Add check for bdt_info
Next by thread: [PATCH v3 1/2] scsi: qedf: Replace kmalloc_array() with kcalloc()
Index(es):
- Date
- Thread

[Index of Archives] [SCSI Target Devel] [Linux SCSI Target Infrastructure] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Samba] [Device Mapper]