Commit 0e5e41ee authored by Christophe JAILLET's avatar Christophe JAILLET Committed by Martin K. Petersen
Browse files

scsi: virtio_scsi: Remove a useless function call 



'inq_result' is known to be NULL. There is no point calling kfree().

Signed-off-by: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/08740635cdb0f8293e57c557b22e048daae50961.1685345683.git.christophe.jaillet@wanadoo.fr


Reviewed-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
Reviewed-by: default avatarStefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 401f8ef3

drivers/scsi/virtio_scsi.c

+1 −3
Original line number Diff line number Diff line
@@ -338,10 +338,8 @@ static int virtscsi_rescan_hotunplug(struct virtio_scsi *vscsi) 
	int result, inquiry_len, inq_result_len = 256;

	char *inq_result = kmalloc(inq_result_len, GFP_KERNEL);



	if (!inq_result) {

		kfree(inq_result);

	if (!inq_result)

		return -ENOMEM;

	}



	shost_for_each_device(sdev, shost) {

		inquiry_len = sdev->inquiry_len ? sdev->inquiry_len : 36;