Make xdprval() oops on bad argument

diff --git a/src/lib/commands/xdump.c b/src/lib/commands/xdump.c
index adc278a43f61e124cbb6410179adea7f3b46d759..92d4762f7886ee37ae06e40001613d9de7e9e56b 100644 (file)
--- a/src/lib/commands/xdump.c
+++ b/src/lib/commands/xdump.c
@@ -106,12 +106,20 @@ xdeval(struct valstr *val,
     return val;                        /* FIXME nstr_exec_val() should return VAL */
 }
 
-/* Dump VAL prefixed with SEP, return " ".  */
+/*
+ * Dump VAL prefixed with SEP, return " ".
+ * VAL must be evaluated.
+ */
 static char *
 xdprval(struct valstr *val, char *sep)
 {
     unsigned char *s, *e, *l;
 
+    if (CANT_HAPPEN(val->val_cat != NSC_VAL)) {
+       pr("%snil", sep);
+       return " ";
+    }
+
     switch (val->val_type) {
     case NSC_LONG:
        pr("%s%ld", sep, val->val_as.lng);