From adb7cf05d31021409dcdfd56147925e6731136bc Mon Sep 17 00:00:00 2001 From: wanghailiangx Date: Fri, 25 Sep 2020 08:26:20 -0400 Subject: [PATCH] test/deallocated : Fix memory leak about context. Here code returns but context isn't freed. And we can change to break here also. Signed-off-by: yidong0635 Change-Id: I8f2173ae3599c13c99c7d70c19e2f8ce0c023f38 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4396 Tested-by: SPDK CI Jenkins Reviewed-by: Ben Walker Reviewed-by: Jim Harris --- test/nvme/deallocated_value/deallocated_value.c | 1 + 1 file changed, 1 insertion(+) diff --git a/test/nvme/deallocated_value/deallocated_value.c b/test/nvme/deallocated_value/deallocated_value.c index 91600e83e..8544df5c7 100644 --- a/test/nvme/deallocated_value/deallocated_value.c +++ b/test/nvme/deallocated_value/deallocated_value.c @@ -265,6 +265,7 @@ deallocate_test(void) ns_entry->qpair = spdk_nvme_ctrlr_alloc_io_qpair(ns_entry->ctrlr, NULL, 0); if (ns_entry->qpair == NULL) { printf("ERROR: spdk_nvme_ctrlr_alloc_io_qpair() failed.\n"); + cleanup(&context); return; }