tor

commit 25af8ada30b84a0df19763bbc226ae209c38a1a0
parent a3342b382a1a87bbe41aa057761cd83daeadbfc7
Author: Nick Mathewson <nickm@torproject.org>
Date:   Tue, 17 Sep 2019 19:26:45 -0400

Correct the syntax description for the MAPADDRESS command.

In 0.4.1.1-alpha I introduced a bug where we would require and
ignore a single positional argument.

Fixes bug 31772.

Diffstat:
A
changes/ticket31772
 | 
4
++++
M
src/feature/control/control_cmd.c
 | 
4
+++-

2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/changes/ticket31772 b/changes/ticket31772
@@ -0,0 +1,4 @@
+  o Minor bugfixes (controller protocol):
+    - Fix the MAPADDRESS controller command to accept one or more
+      arguments. Previously, it required two or more arguments, and ignored
+      the first. Fixes bug 31772; bugfix on 0.4.1.1-alpha.
diff --git a/src/feature/control/control_cmd.c b/src/feature/control/control_cmd.c
@@ -639,7 +639,9 @@ address_is_invalid_mapaddress_target(const char *addr)
 }
 
 static const control_cmd_syntax_t mapaddress_syntax = {
-  .max_args=1,
+  // no positional arguments are expected
+  .max_args=0,
+  // an arbitrary number of K=V entries are supported.
   .accept_keywords=true,
 };