Commit 36a5cda1 authored by Tomasz Zawadzki's avatar Tomasz Zawadzki
Browse files

lib/lvol: always use lvol_free instead of free 



Right now lvol_free() just frees the lvol,
thus not much functional change happens with this patch.

Later in the series, more sanity checks for debugging
will be added to lvol_free().

Change-Id: Ifd934c68134f0944b1b69d51470502190edb756f
Signed-off-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18687


Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarBen Walker <ben@nvidia.com>
Community-CI: Mellanox Build Bot
parent 7920c642

lib/lvol/lvol.c

+3 −3
Original line number Diff line number Diff line
@@ -217,7 +217,7 @@ load_next_lvol(void *cb_arg, struct spdk_blob *blob, int lvolerrno) 
		} else {

			TAILQ_FOREACH_SAFE(lvol, &lvs->lvols, link, tmp) {

				TAILQ_REMOVE(&lvs->lvols, lvol, link);

				free(lvol);

				lvol_free(lvol);

			}

			lvs_free(lvs);

			spdk_bs_unload(bs, bs_unload_with_error_cb, req);
@@ -1059,7 +1059,7 @@ lvol_create_open_cb(void *cb_arg, struct spdk_blob *blob, int lvolerrno) 
	TAILQ_REMOVE(&req->lvol->lvol_store->pending_lvols, req->lvol, link);



	if (lvolerrno < 0) {

		free(lvol);

		lvol_free(lvol);

		req->cb_fn(req->cb_arg, NULL, lvolerrno);

		free(req);

		return;
@@ -1086,7 +1086,7 @@ lvol_create_cb(void *cb_arg, spdk_blob_id blobid, int lvolerrno) 


	if (lvolerrno < 0) {

		TAILQ_REMOVE(&req->lvol->lvol_store->pending_lvols, req->lvol, link);

		free(req->lvol);

		lvol_free(req->lvol);

		assert(req->cb_fn != NULL);

		req->cb_fn(req->cb_arg, NULL, lvolerrno);

		free(req);