[tor-commits] [tor/maint-0.4.3] In typed_var_kvassign, include config names in error messages.

nickm at torproject.org nickm at torproject.org
Thu Mar 5 13:48:47 UTC 2020


commit 2c792d1e0e6f92c33287ba8e87055142745a6532
Author: Nick Mathewson <nickm at torproject.org>
Date:   Wed Feb 26 14:16:55 2020 -0500

    In typed_var_kvassign, include config names in error messages.
    
    This should improve the usability of our configuration error messages.
---
 src/lib/confmgt/typedvar.c | 11 ++++++++++-
 src/test/test_options.c    |  3 ++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/lib/confmgt/typedvar.c b/src/lib/confmgt/typedvar.c
index 3a188db95..1955302cd 100644
--- a/src/lib/confmgt/typedvar.c
+++ b/src/lib/confmgt/typedvar.c
@@ -24,6 +24,7 @@
 #include "lib/log/log.h"
 #include "lib/log/util_bug.h"
 #include "lib/malloc/malloc.h"
+#include "lib/string/printf.h"
 #include "lib/string/util_string.h"
 
 #include "lib/confmgt/var_type_def_st.h"
@@ -75,7 +76,15 @@ typed_var_kvassign(void *target, const config_line_t *line,
     return def->fns->kv_parse(target, line, errmsg, def->params);
   }
 
-  return typed_var_assign(target, line->value, errmsg, def);
+  int rv = typed_var_assign(target, line->value, errmsg, def);
+  if (rv < 0 && *errmsg != NULL) {
+    /* typed_var_assign() didn't know the line's keyword, but we do.
+     * Let's add it to the error message. */
+    char *oldmsg = *errmsg;
+    tor_asprintf(errmsg, "Could not parse %s: %s", line->key, oldmsg);
+    tor_free(oldmsg);
+  }
+  return rv;
 }
 
 /**
diff --git a/src/test/test_options.c b/src/test/test_options.c
index 129a8299e..636d3c0e5 100644
--- a/src/test/test_options.c
+++ b/src/test/test_options.c
@@ -4269,7 +4269,8 @@ test_options_trial_assign(void *arg)
   v = options_trial_assign(lines, 0, &msg);
   tt_int_op(v, OP_EQ, SETOPT_ERR_PARSE);
   tt_str_op(msg, OP_EQ,
-            "Unrecognized value ambidextrous. Allowed values are 0 and 1.");
+            "Could not parse UseBridges: Unrecognized value ambidextrous. "
+            "Allowed values are 0 and 1.");
   tor_free(msg);
   config_free_lines(lines);
 





More information about the tor-commits mailing list