ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bdev/virtio/scsi: fix bdev->name memory leak

Browse Source 
This must have been overlooked before.

Change-Id: I631c7b926970d045e9e91cada535acd605e05b6e
Signed-off-by: Dariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/406334
Tested-by: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: Pawel Wodkowski <pawelx.wodkowski@intel.com>
Reviewed-by: Daniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
This commit is contained in:
Dariusz Stojaczyk 2018-04-04 14:29:57 +02:00 committed by Jim Harris
parent be98deff3e
commit 20d8fec039
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/bdev/virtio/bdev_virtio_scsi.c
View File

@ -664,6 +664,7 @@ bdev_virtio_disk_destruct(void *ctx)
struct virtio_scsi_dev *svdev = disk->svdev;
TAILQ_REMOVE(&svdev->luns, disk, link);
free(disk->bdev.name);
free(disk);
if (svdev->removed && TAILQ_EMPTY(&svdev->luns)) {
@ -1292,6 +1293,7 @@ virtio_scsi_dev_add_tgt(struct virtio_scsi_dev *svdev, struct virtio_scsi_scan_i
rc = spdk_bdev_register(&disk->bdev);
if (rc) {
SPDK_ERRLOG("Failed to register bdev name=%s\n", disk->bdev.name);
free(bdev->name);
free(disk);
return rc;
}