Zbigniew Jędrzejewski-Szmek 62fe94
From 4fe1be9ce2e0cca6354a4167f0a1a7e1f943c91c Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Lennart Poettering <lennart@poettering.net>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Thu, 21 Aug 2014 19:10:26 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] systemctl: in list-unit-files, always show legend, even if we
Zbigniew Jędrzejewski-Szmek 62fe94
 know about no unit files
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/systemctl/systemctl.c | 14 +++++---------
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 5 insertions(+), 9 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c
Zbigniew Jędrzejewski-Szmek 62fe94
index 072f615ad5..d9b8bee28d 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/systemctl/systemctl.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/systemctl/systemctl.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -1351,11 +1351,8 @@ static int list_unit_files(sd_bus *bus, char **args) {
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
                 n_units = hashmap_size(h);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-                if (n_units == 0)
Zbigniew Jędrzejewski-Szmek 62fe94
-                        return 0;
Zbigniew Jędrzejewski-Szmek 62fe94
-
Zbigniew Jędrzejewski-Szmek 62fe94
                 units = new(UnitFileList, n_units);
Zbigniew Jędrzejewski-Szmek 62fe94
-                if (!units) {
Zbigniew Jędrzejewski-Szmek 62fe94
+                if (!units && n_units > 0) {
Zbigniew Jędrzejewski-Szmek 62fe94
                         unit_file_list_free(h);
Zbigniew Jędrzejewski-Szmek 62fe94
                         return log_oom();
Zbigniew Jędrzejewski-Szmek 62fe94
                 }
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -1411,14 +1408,13 @@ static int list_unit_files(sd_bus *bus, char **args) {
Zbigniew Jędrzejewski-Szmek 62fe94
                         return bus_log_parse_error(r);
Zbigniew Jędrzejewski-Szmek 62fe94
         }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (c > 0) {
Zbigniew Jędrzejewski-Szmek 62fe94
-                qsort(units, c, sizeof(UnitFileList), compare_unit_file_list);
Zbigniew Jędrzejewski-Szmek 62fe94
-                output_unit_file_list(units, c);
Zbigniew Jędrzejewski-Szmek 62fe94
-        }
Zbigniew Jędrzejewski-Szmek 62fe94
+        qsort_safe(units, c, sizeof(UnitFileList), compare_unit_file_list);
Zbigniew Jędrzejewski-Szmek 62fe94
+        output_unit_file_list(units, c);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (avoid_bus())
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (avoid_bus()) {
Zbigniew Jędrzejewski-Szmek 62fe94
                 for (unit = units; unit < units + c; unit++)
Zbigniew Jędrzejewski-Szmek 62fe94
                         free(unit->path);
Zbigniew Jędrzejewski-Szmek 62fe94
+        }
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         return 0;
Zbigniew Jędrzejewski-Szmek 62fe94
 }