[libvirt] [PATCH 21/23] Avoid crash on OOM in virportallocatortest
Daniel P. Berrange
berrange at redhat.com
Wed Sep 25 14:51:16 UTC 2013
From: "Daniel P. Berrange" <berrange at redhat.com>
The virportallocatortest did not check if the object
allocation failed in all cases. This lead to a crash
on OOM in the testsuite
Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
---
tests/virportallocatortest.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/tests/virportallocatortest.c b/tests/virportallocatortest.c
index 1a0cdfa..615fa15 100644
--- a/tests/virportallocatortest.c
+++ b/tests/virportallocatortest.c
@@ -67,6 +67,9 @@ static int testAllocAll(const void *args ATTRIBUTE_UNUSED)
int ret = -1;
unsigned short p1, p2, p3, p4, p5, p6, p7;
+ if (!alloc)
+ return -1;
+
if (virPortAllocatorAcquire(alloc, &p1) < 0)
goto cleanup;
if (p1 != 5901) {
@@ -137,6 +140,9 @@ static int testAllocReuse(const void *args ATTRIBUTE_UNUSED)
int ret = -1;
unsigned short p1, p2, p3, p4;
+ if (!alloc)
+ return -1;
+
if (virPortAllocatorAcquire(alloc, &p1) < 0)
goto cleanup;
if (p1 != 5901) {
--
1.8.3.1
More information about the libvir-list
mailing list