yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone

Blame SOURCES/kvm-util-Drop-superfluous-conditionals-around-g_free.patch

9ae3a8
From 7c92e003601bf83d662f5e21a4bc095de875e906 Mon Sep 17 00:00:00 2001
9ae3a8
From: Markus Armbruster <armbru@redhat.com>
9ae3a8
Date: Tue, 8 Sep 2015 18:06:20 +0200
9ae3a8
Subject: [PATCH 2/7] util: Drop superfluous conditionals around g_free()
9ae3a8
9ae3a8
Message-id: <1441735585-23432-3-git-send-email-armbru@redhat.com>
9ae3a8
Patchwork-id: 67707
9ae3a8
O-Subject: [RHEL-7.2 qemu-kvm PATCH 2/7] util: Drop superfluous conditionals around g_free()
9ae3a8
Bugzilla: 1218919
9ae3a8
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
9ae3a8
RH-Acked-by: Fam Zheng <famz@redhat.com>
9ae3a8
RH-Acked-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
9ae3a8
9ae3a8
Signed-off-by: Markus Armbruster <armbru@redhat.com>
9ae3a8
Reviewed-by: Eric Blake <eblake@redhat.com>
9ae3a8
Reviewed-by: Fam Zheng <famz@redhat.com>
9ae3a8
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
9ae3a8
(cherry picked from commit 44c2286b5df1f50fc9fedabd7b1a5a8022e583fc)
9ae3a8
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
9ae3a8
---
9ae3a8
 util/uri.c | 48 ++++++++++++++++++++++--------------------------
9ae3a8
 1 file changed, 22 insertions(+), 26 deletions(-)
9ae3a8
9ae3a8
diff --git a/util/uri.c b/util/uri.c
9ae3a8
index bbf2832..01dc09e 100644
9ae3a8
--- a/util/uri.c
9ae3a8
+++ b/util/uri.c
9ae3a8
@@ -225,7 +225,7 @@ rfc3986_parse_scheme(URI *uri, const char **str) {
9ae3a8
     while (ISA_ALPHA(cur) || ISA_DIGIT(cur) ||
9ae3a8
            (*cur == '+') || (*cur == '-') || (*cur == '.')) cur++;
9ae3a8
     if (uri != NULL) {
9ae3a8
-	if (uri->scheme != NULL) g_free(uri->scheme);
9ae3a8
+        g_free(uri->scheme);
9ae3a8
 	uri->scheme = g_strndup(*str, cur - *str);
9ae3a8
     }
9ae3a8
     *str = cur;
9ae3a8
@@ -262,8 +262,7 @@ rfc3986_parse_fragment(URI *uri, const char **str)
9ae3a8
            ((uri != NULL) && (uri->cleanup & 1) && (IS_UNWISE(cur))))
9ae3a8
         NEXT(cur);
9ae3a8
     if (uri != NULL) {
9ae3a8
-        if (uri->fragment != NULL)
9ae3a8
-            g_free(uri->fragment);
9ae3a8
+        g_free(uri->fragment);
9ae3a8
 	if (uri->cleanup & 2)
9ae3a8
 	    uri->fragment = g_strndup(*str, cur - *str);
9ae3a8
 	else
9ae3a8
@@ -298,8 +297,7 @@ rfc3986_parse_query(URI *uri, const char **str)
9ae3a8
            ((uri != NULL) && (uri->cleanup & 1) && (IS_UNWISE(cur))))
9ae3a8
         NEXT(cur);
9ae3a8
     if (uri != NULL) {
9ae3a8
-	if (uri->query != NULL)
9ae3a8
-	    g_free (uri->query);
9ae3a8
+        g_free(uri->query);
9ae3a8
 	uri->query = g_strndup (*str, cur - *str);
9ae3a8
     }
9ae3a8
     *str = cur;
9ae3a8
@@ -360,7 +358,7 @@ rfc3986_parse_user_info(URI *uri, const char **str)
9ae3a8
 	NEXT(cur);
9ae3a8
     if (*cur == '@') {
9ae3a8
 	if (uri != NULL) {
9ae3a8
-	    if (uri->user != NULL) g_free(uri->user);
9ae3a8
+            g_free(uri->user);
9ae3a8
 	    if (uri->cleanup & 2)
9ae3a8
 		uri->user = g_strndup(*str, cur - *str);
9ae3a8
 	    else
9ae3a8
@@ -473,9 +471,9 @@ not_ipv4:
9ae3a8
         NEXT(cur);
9ae3a8
 found:
9ae3a8
     if (uri != NULL) {
9ae3a8
-	if (uri->authority != NULL) g_free(uri->authority);
9ae3a8
+        g_free(uri->authority);
9ae3a8
 	uri->authority = NULL;
9ae3a8
-	if (uri->server != NULL) g_free(uri->server);
9ae3a8
+        g_free(uri->server);
9ae3a8
 	if (cur != host) {
9ae3a8
 	    if (uri->cleanup & 2)
9ae3a8
 		uri->server = g_strndup(host, cur - host);
9ae3a8
@@ -585,7 +583,7 @@ rfc3986_parse_path_ab_empty(URI *uri, const char **str)
9ae3a8
 	if (ret != 0) return(ret);
9ae3a8
     }
9ae3a8
     if (uri != NULL) {
9ae3a8
-	if (uri->path != NULL) g_free(uri->path);
9ae3a8
+        g_free(uri->path);
9ae3a8
         if (*str != cur) {
9ae3a8
             if (uri->cleanup & 2)
9ae3a8
                 uri->path = g_strndup(*str, cur - *str);
9ae3a8
@@ -631,7 +629,7 @@ rfc3986_parse_path_absolute(URI *uri, const char **str)
9ae3a8
 	}
9ae3a8
     }
9ae3a8
     if (uri != NULL) {
9ae3a8
-	if (uri->path != NULL) g_free(uri->path);
9ae3a8
+        g_free(uri->path);
9ae3a8
         if (cur != *str) {
9ae3a8
             if (uri->cleanup & 2)
9ae3a8
                 uri->path = g_strndup(*str, cur - *str);
9ae3a8
@@ -673,7 +671,7 @@ rfc3986_parse_path_rootless(URI *uri, const char **str)
9ae3a8
 	if (ret != 0) return(ret);
9ae3a8
     }
9ae3a8
     if (uri != NULL) {
9ae3a8
-	if (uri->path != NULL) g_free(uri->path);
9ae3a8
+        g_free(uri->path);
9ae3a8
         if (cur != *str) {
9ae3a8
             if (uri->cleanup & 2)
9ae3a8
                 uri->path = g_strndup(*str, cur - *str);
9ae3a8
@@ -715,7 +713,7 @@ rfc3986_parse_path_no_scheme(URI *uri, const char **str)
9ae3a8
 	if (ret != 0) return(ret);
9ae3a8
     }
9ae3a8
     if (uri != NULL) {
9ae3a8
-	if (uri->path != NULL) g_free(uri->path);
9ae3a8
+        g_free(uri->path);
9ae3a8
         if (cur != *str) {
9ae3a8
             if (uri->cleanup & 2)
9ae3a8
                 uri->path = g_strndup(*str, cur - *str);
9ae3a8
@@ -769,7 +767,7 @@ rfc3986_parse_hier_part(URI *uri, const char **str)
9ae3a8
     } else {
9ae3a8
 	/* path-empty is effectively empty */
9ae3a8
 	if (uri != NULL) {
9ae3a8
-	    if (uri->path != NULL) g_free(uri->path);
9ae3a8
+            g_free(uri->path);
9ae3a8
 	    uri->path = NULL;
9ae3a8
 	}
9ae3a8
     }
9ae3a8
@@ -812,7 +810,7 @@ rfc3986_parse_relative_ref(URI *uri, const char *str) {
9ae3a8
     } else {
9ae3a8
 	/* path-empty is effectively empty */
9ae3a8
 	if (uri != NULL) {
9ae3a8
-	    if (uri->path != NULL) g_free(uri->path);
9ae3a8
+            g_free(uri->path);
9ae3a8
 	    uri->path = NULL;
9ae3a8
 	}
9ae3a8
     }
9ae3a8
@@ -1285,21 +1283,21 @@ static void
9ae3a8
 uri_clean(URI *uri) {
9ae3a8
     if (uri == NULL) return;
9ae3a8
 
9ae3a8
-    if (uri->scheme != NULL) g_free(uri->scheme);
9ae3a8
+    g_free(uri->scheme);
9ae3a8
     uri->scheme = NULL;
9ae3a8
-    if (uri->server != NULL) g_free(uri->server);
9ae3a8
+    g_free(uri->server);
9ae3a8
     uri->server = NULL;
9ae3a8
-    if (uri->user != NULL) g_free(uri->user);
9ae3a8
+    g_free(uri->user);
9ae3a8
     uri->user = NULL;
9ae3a8
-    if (uri->path != NULL) g_free(uri->path);
9ae3a8
+    g_free(uri->path);
9ae3a8
     uri->path = NULL;
9ae3a8
-    if (uri->fragment != NULL) g_free(uri->fragment);
9ae3a8
+    g_free(uri->fragment);
9ae3a8
     uri->fragment = NULL;
9ae3a8
-    if (uri->opaque != NULL) g_free(uri->opaque);
9ae3a8
+    g_free(uri->opaque);
9ae3a8
     uri->opaque = NULL;
9ae3a8
-    if (uri->authority != NULL) g_free(uri->authority);
9ae3a8
+    g_free(uri->authority);
9ae3a8
     uri->authority = NULL;
9ae3a8
-    if (uri->query != NULL) g_free(uri->query);
9ae3a8
+    g_free(uri->query);
9ae3a8
     uri->query = NULL;
9ae3a8
 }
9ae3a8
 
9ae3a8
@@ -1711,10 +1709,8 @@ uri_resolve(const char *uri, const char *base) {
9ae3a8
 	/*
9ae3a8
 	 * the base fragment must be ignored
9ae3a8
 	 */
9ae3a8
-	if (bas->fragment != NULL) {
9ae3a8
-	    g_free(bas->fragment);
9ae3a8
-	    bas->fragment = NULL;
9ae3a8
-	}
9ae3a8
+        g_free(bas->fragment);
9ae3a8
+        bas->fragment = NULL;
9ae3a8
 	val = uri_to_string(bas);
9ae3a8
 	goto done;
9ae3a8
     }
9ae3a8
-- 
9ae3a8
1.8.3.1
9ae3a8