Plug another memory leak.

Again, if ibv_get_device_list() returns a non-null pointer, and sets the
"number of devices" value to 0, we still have to free the list pointed
to by the non-null pointer.

1 files changed, 5 insertions, 1 deletions

diff --git a/pcap-rdmasniff.c b/pcap-rdmasniff.c
index 9cb56abb..032b80f8 100644
--- a/pcap-rdmasniff.c
+++ b/pcap-rdmasniff.c
@@ -368,7 +368,11 @@ rdmasniff_create(const char *device, char *ebuf, int *is_ours)
 	*is_ours = 0;
 
 	dev_list = ibv_get_device_list(&numdev);
-	if (!dev_list || !numdev) {
+	if (!dev_list) {
+		return NULL;
+	}
+	if (!numdev) {
+		ibv_free_device_list(dev_list);
 		return NULL;
 	}