|
 |
3e5111 |
From 25fbc5c619a4277086b0bce0e3a55dd4099d4ab2 Mon Sep 17 00:00:00 2001
|
|
|
3e5111 |
Message-Id: <25fbc5c619a4277086b0bce0e3a55dd4099d4ab2@dist-git>
|
|
|
3e5111 |
From: Martin Kletzander <mkletzan@redhat.com>
|
|
|
3e5111 |
Date: Tue, 25 Apr 2017 13:41:20 +0200
|
|
|
3e5111 |
Subject: [PATCH] Remove pointless check for !ret in
|
|
|
3e5111 |
virDomainNetDefCoalesceParseXML
|
|
|
3e5111 |
|
|
|
3e5111 |
https://bugzilla.redhat.com/show_bug.cgi?id=1414627
|
|
|
3e5111 |
|
|
|
3e5111 |
It was left there after removing a macro it was part of in first
|
|
|
3e5111 |
version or so. Now it will always be NULL.
|
|
|
3e5111 |
|
|
|
3e5111 |
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
|
|
|
3e5111 |
(cherry picked from commit b2763f189c5b306a685021b4fede9e2cee8528de)
|
|
|
3e5111 |
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
|
|
|
3e5111 |
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
|
|
|
3e5111 |
---
|
|
|
3e5111 |
src/conf/domain_conf.c | 2 +-
|
|
|
3e5111 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
3e5111 |
|
|
|
3e5111 |
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
|
|
|
3e5111 |
index 47022c329..2d5d00cf2 100644
|
|
|
3e5111 |
--- a/src/conf/domain_conf.c
|
|
|
3e5111 |
+++ b/src/conf/domain_conf.c
|
|
|
3e5111 |
@@ -6776,7 +6776,7 @@ virDomainNetDefCoalesceParseXML(xmlNodePtr node,
|
|
|
3e5111 |
if (!str)
|
|
|
3e5111 |
goto cleanup;
|
|
|
3e5111 |
|
|
|
3e5111 |
- if (!ret && VIR_ALLOC(ret) < 0)
|
|
|
3e5111 |
+ if (VIR_ALLOC(ret) < 0)
|
|
|
3e5111 |
goto cleanup;
|
|
|
3e5111 |
|
|
|
3e5111 |
if (virStrToLong_ullp(str, NULL, 10, &tmp) < 0) {
|
|
|
3e5111 |
--
|
|
|
3e5111 |
2.12.2
|
|
|
3e5111 |
|