[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20220824123010.51763-6-manivannan.sadhasivam@linaro.org>
Date: Wed, 24 Aug 2022 18:00:10 +0530
From: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
To: kishon@...com, gregkh@...uxfoundation.org, lpieralisi@...nel.org
Cc: linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org,
mie@...l.co.jp, kw@...ux.com,
Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Subject: [PATCH v2 5/5] tools: PCI: Fix memory leak
Memory allocated for "test" needs to be freed at the end of the main().
Fixes: 3f2ed8134834 ("tools: PCI: Add a userspace tool to test PCI endpoint")
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
---
tools/pci/pcitest.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/tools/pci/pcitest.c b/tools/pci/pcitest.c
index a4e5b17cc3b5..a416a66802f3 100644
--- a/tools/pci/pcitest.c
+++ b/tools/pci/pcitest.c
@@ -157,7 +157,7 @@ static int run_test(struct pci_test *test)
int main(int argc, char **argv)
{
- int c;
+ int c, ret;
struct pci_test *test;
test = calloc(1, sizeof(*test));
@@ -249,5 +249,8 @@ int main(int argc, char **argv)
return -EINVAL;
}
- return run_test(test);
+ ret = run_test(test);
+ free(test);
+
+ return ret;
}
--
2.25.1
Powered by blists - more mailing lists