From 2b440236f99524cc42e50b98f8b6b1510330c32e Mon Sep 17 00:00:00 2001 From: Andre Guedes Date: Fri, 7 Feb 2014 19:28:44 -0300 Subject: [PATCH] amptest: Fix memory leak --- tools/amptest.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/tools/amptest.c b/tools/amptest.c index 16f15bc87..6192f7e67 100644 --- a/tools/amptest.c +++ b/tools/amptest.c @@ -496,6 +496,7 @@ static bool find_amp_controller(void) struct hci_dev_list_req *dl; struct hci_dev_req *dr; int fd, i; + bool result; fd = socket(AF_BLUETOOTH, SOCK_RAW, BTPROTO_HCI); if (fd < 0) { @@ -515,8 +516,8 @@ static bool find_amp_controller(void) if (ioctl(fd, HCIGETDEVLIST, (void *) dl) < 0) { perror("Failed to get HCI device list"); - close(fd); - return false; + result = false; + goto done; } for (i = 0; i< dl->dev_num; i++) { @@ -541,9 +542,12 @@ static bool find_amp_controller(void) } } - close(fd); + result = true; - return true; +done: + free(dl); + close(fd); + return result; } int main(int argc ,char *argv[]) -- 2.47.3