[PATCH 03/10] virlease: Rework virLeaseReadCustomLeaseFile()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



There are some variables which are used only inside the single
loop the function has. Let's declare them inside the loop body to
make that obvious. Also, fix indendation.

Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
---
 src/util/virlease.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/util/virlease.c b/src/util/virlease.c
index aeb605862f..f01cf5b9bd 100644
--- a/src/util/virlease.c
+++ b/src/util/virlease.c
@@ -50,11 +50,7 @@ virLeaseReadCustomLeaseFile(virJSONValuePtr leases_array_new,
 {
     g_autofree char *lease_entries = NULL;
     g_autoptr(virJSONValue) leases_array = NULL;
-    long long expirytime;
     int custom_lease_file_len = 0;
-    virJSONValuePtr lease_tmp = NULL;
-    const char *ip_tmp = NULL;
-    const char *server_duid_tmp = NULL;
     size_t i;
 
     /* Read entire contents */
@@ -83,7 +79,11 @@ virLeaseReadCustomLeaseFile(virJSONValuePtr leases_array_new,
 
     i = 0;
     while (i < virJSONValueArraySize(leases_array)) {
-        if (!(lease_tmp = virJSONValueArrayGet(leases_array, i))) {
+        virJSONValuePtr lease_tmp = virJSONValueArrayGet(leases_array, i);
+        long long expirytime;
+        const char *ip_tmp = NULL;
+
+        if (!lease_tmp) {
             virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                            _("failed to parse json"));
             return -1;
@@ -103,9 +103,10 @@ virLeaseReadCustomLeaseFile(virJSONValuePtr leases_array_new,
         }
 
         if (server_duid && strchr(ip_tmp, ':')) {
+            const char *server_duid_tmp = NULL;
+
             /* This is an ipv6 lease */
-            if ((server_duid_tmp
-                 = virJSONValueObjectGetString(lease_tmp, "server-duid"))) {
+            if ((server_duid_tmp = virJSONValueObjectGetString(lease_tmp, "server-duid"))) {
                 if (!*server_duid)
                     *server_duid = g_strdup(server_duid_tmp);
             } else {
-- 
2.26.2




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux