Jan Synacek 78d35a
From bd99ada53a580dd28bc2c6bfcd2c974e0696ed94 Mon Sep 17 00:00:00 2001
Jan Synacek 78d35a
From: Jan Synacek <jsynacek@redhat.com>
Jan Synacek 78d35a
Date: Tue, 14 Feb 2017 10:21:53 +0100
Jan Synacek 78d35a
Subject: [PATCH] machinectl: make sure that inability to get OS version isn't
Jan Synacek 78d35a
 consider fatal for machine list
Jan Synacek 78d35a
Jan Synacek 78d35a
Backported from: 4b2419165ce409ee55ce96a926302f89685f2293
Jan Synacek 78d35a
Resolves: #1419501
Jan Synacek 78d35a
---
Jan Synacek 78d35a
 src/machine/machinectl.c | 16 ++++++----------
Jan Synacek 78d35a
 1 file changed, 6 insertions(+), 10 deletions(-)
Jan Synacek 78d35a
Jan Synacek 78d35a
diff --git a/src/machine/machinectl.c b/src/machine/machinectl.c
Jan Synacek 78d35a
index 7b9be3b..7fe1da1 100644
Jan Synacek 78d35a
--- a/src/machine/machinectl.c
Jan Synacek 78d35a
+++ b/src/machine/machinectl.c
Jan Synacek 78d35a
@@ -242,24 +242,20 @@ static int list_machines(int argc, char *argv[], void *userdata) {
Jan Synacek 78d35a
                 if (name[0] == '.' && !arg_all)
Jan Synacek 78d35a
                         continue;
Jan Synacek 78d35a
 
Jan Synacek 78d35a
-                if (!GREEDY_REALLOC(machines, n_allocated, n_machines + 1)) {
Jan Synacek 78d35a
+                if (!GREEDY_REALLOC0(machines, n_allocated, n_machines + 1)) {
Jan Synacek 78d35a
                         r = log_oom();
Jan Synacek 78d35a
                         goto out;
Jan Synacek 78d35a
                 }
Jan Synacek 78d35a
 
Jan Synacek 78d35a
-                machines[n_machines].os = NULL;
Jan Synacek 78d35a
-                machines[n_machines].version_id = NULL;
Jan Synacek 78d35a
-                r = get_os_release_property(bus, name,
Jan Synacek 78d35a
-                                "ID\0" "VERSION_ID\0",
Jan Synacek 78d35a
-                                &machines[n_machines].os,
Jan Synacek 78d35a
-                                &machines[n_machines].version_id);
Jan Synacek 78d35a
-                if (r < 0)
Jan Synacek 78d35a
-                        goto out;
Jan Synacek 78d35a
-
Jan Synacek 78d35a
                 machines[n_machines].name = name;
Jan Synacek 78d35a
                 machines[n_machines].class = class;
Jan Synacek 78d35a
                 machines[n_machines].service = service;
Jan Synacek 78d35a
 
Jan Synacek 78d35a
+                (void) get_os_release_property(bus, name,
Jan Synacek 78d35a
+                                "ID\0" "VERSION_ID\0",
Jan Synacek 78d35a
+                                &machines[n_machines].os,
Jan Synacek 78d35a
+                                &machines[n_machines].version_id);
Jan Synacek 78d35a
+
Jan Synacek 78d35a
                 l = strlen(name);
Jan Synacek 78d35a
                 if (l > max_name)
Jan Synacek 78d35a
                         max_name = l;
Jan Synacek 78d35a
-- 
Jan Synacek 78d35a
2.9.3
Jan Synacek 78d35a