Commit 5f947da7 authored by Piotr Pelplinski's avatar Piotr Pelplinski Committed by Ben Walker
Browse files

blobstore: move free xattr's to separate function 



Signed-off-by: default avatarPiotr Pelplinski <piotr.pelplinski@intel.com>
Change-Id: I8b570802b05b8e03802d3c2b68a1e7644ea548ac
Reviewed-on: https://review.gerrithub.io/396572


Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent 69c9bb01

lib/blob/blobstore.c

+13 −7
Original line number Diff line number Diff line
@@ -166,10 +166,21 @@ _spdk_blob_alloc(struct spdk_blob_store *bs, spdk_blob_id id) 
}



static void

_spdk_blob_free(struct spdk_blob_data *blob)

_spdk_xattrs_free(struct spdk_xattr_tailq *xattrs)

{

	struct spdk_xattr 	*xattr, *xattr_tmp;



	TAILQ_FOREACH_SAFE(xattr, xattrs, link, xattr_tmp) {

		TAILQ_REMOVE(xattrs, xattr, link);

		free(xattr->name);

		free(xattr->value);

		free(xattr);

	}

}



static void

_spdk_blob_free(struct spdk_blob_data *blob)

{

	assert(blob != NULL);



	free(blob->active.clusters);
@@ -177,12 +188,7 @@ _spdk_blob_free(struct spdk_blob_data *blob) 
	free(blob->active.pages);

	free(blob->clean.pages);



	TAILQ_FOREACH_SAFE(xattr, &blob->xattrs, link, xattr_tmp) {

		TAILQ_REMOVE(&blob->xattrs, xattr, link);

		free(xattr->name);

		free(xattr->value);

		free(xattr);

	}

	_spdk_xattrs_free(&blob->xattrs);



	free(blob);

}