[libvirt] [PATCH 2/9] vsh-table: allow empty columns

Ján Tomko jtomko at redhat.com
Tue Feb 12 09:21:20 UTC 2019


Trivially implement this by deleting the bogus check in
vshTableSafeEncode.

Now it returns an empty string for an empty string instead
of returning NULL without setting an error.

Signed-off-by: Ján Tomko <jtomko at redhat.com>
---
 tests/vshtabletest.c | 4 +++-
 tools/vsh-table.c    | 3 ---
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/tests/vshtabletest.c b/tests/vshtabletest.c
index 79285b14a9..b07db3cf23 100644
--- a/tests/vshtabletest.c
+++ b/tests/vshtabletest.c
@@ -291,7 +291,8 @@ testNTables(const void *opaque ATTRIBUTE_UNUSED)
         " Id   Name       Status\n"
         "--------------------------\n"
         " 1    fedora28   running\n"
-        " 2    rhel7.5    running\n";
+        " 2    rhel7.5    running\n"
+        " 3    gazpacho   \n";
     const char *exp2 =
         " Id   Name   Status\n"
         "---------------------\n";
@@ -311,6 +312,7 @@ testNTables(const void *opaque ATTRIBUTE_UNUSED)
         goto cleanup;
     vshTableRowAppend(table1, "1", "fedora28", "running", NULL);
     vshTableRowAppend(table1, "2", "rhel7.5", "running", NULL);
+    vshTableRowAppend(table1, "3", "gazpacho", "", NULL);
     act1 = vshTablePrintToString(table1, true);
 
     table2 = vshTableNew("Id", "Name", "Status", NULL);
diff --git a/tools/vsh-table.c b/tools/vsh-table.c
index fda8f15879..8bd6d99778 100644
--- a/tools/vsh-table.c
+++ b/tools/vsh-table.c
@@ -219,9 +219,6 @@ vshTableSafeEncode(const char *s, size_t *width)
 
     memset(&st, 0, sizeof(st));
 
-    if (!sz)
-        return NULL;
-
     if (VIR_ALLOC_N(buf, (sz * HEX_ENCODE_LENGTH) + 1) < 0)
         return NULL;
 
-- 
2.19.2




More information about the libvir-list mailing list