Commit 14ad24c7 authored by Zhiqiang Liu's avatar Zhiqiang Liu Committed by Tomasz Zawadzki
Browse files

idxd: fix one memleak problem in spdk_idxd_get_channel() 



In spdk_idxd_get_channel(), if chan->batch_base is allocated
faild, we should free chan before returning NULL.

Signed-off-by: default avatarZhiqiang Liu <liuzhiqiang26@huawei.com>
Change-Id: Ia652c334aead592429c1171da73d67160879686d
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/8301


Community-CI: Mellanox Build Bot
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 36304737

lib/idxd/idxd.c

+1 −0
Original line number Diff line number Diff line
@@ -108,6 +108,7 @@ spdk_idxd_get_channel(struct spdk_idxd_device *idxd) 
	chan->batch_base = calloc(NUM_BATCHES_PER_CHANNEL, sizeof(struct idxd_batch));

	if (chan->batch_base == NULL) {

		SPDK_ERRLOG("Failed to allocate batch pool\n");

		free(chan);

		return NULL;

	}