From: Roland Dreier <rolandd@cisco.com>
Date: Wed, 17 Jan 2007 21:41:33 +0000 (-0800)
Subject: Check return of calloc() in ibv_get_device_list()
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=596803fbae8ff5e2e50b5a99b3e84ee6b3603779;p=~shefty%2Flibibverbs.git

Check return of calloc() in ibv_get_device_list()

Don't blindly copy device pointers if calloc() returns NULL.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
(cherry picked from commit 789728f7be1e6c7f22380ae739d7b692d2c08d0f)
---

diff --git a/src/device.c b/src/device.c
index 90e3b15..30c81ba 100644
--- a/src/device.c
+++ b/src/device.c
@@ -62,9 +62,13 @@ struct ibv_device **ibv_get_device_list(int *num)
 		num_devices = ibverbs_init(&device_list);
 
 	l = calloc(num_devices + 1, sizeof (struct ibv_device *));
+	if (!l)
+		goto out;
+
 	for (i = 0; i < num_devices; ++i)
 		l[i] = device_list[i];
 
+out:
 	pthread_mutex_unlock(&device_list_lock);
 
 	if (num)