Commit ea88aefb authored by Dariusz Stojaczyk's avatar Dariusz Stojaczyk Committed by Jim Harris
Browse files

test/bdevperf: fix memory leak if opening a descriptor fails 



This was probably overlooked when developing 57d174ff.

Fixes: 57d174ff ("bdev: add spdk_bdev_open/close")
Change-Id: Idb30287e740ac0300a5c7dc9fad7e06693f58330
Signed-off-by: default avatarDariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/398585


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 25191ba7

test/lib/bdev/bdevperf/bdevperf.c

+2 −0
Original line number Diff line number Diff line
@@ -206,6 +206,8 @@ bdevperf_construct_targets(void) 
		rc = spdk_bdev_open(bdev, true, NULL, NULL, &target->bdev_desc);

		if (rc != 0) {

			SPDK_ERRLOG("Could not open leaf bdev %s, error=%d\n", spdk_bdev_get_name(bdev), rc);

			free(target->name);

			free(target);

			bdev = spdk_bdev_next_leaf(bdev);

			continue;

		}