Lowercase BridgeDistribution value before adding it to the descriptor.

This patch makes sure we lowercase the value of BridgeDistribution
before we add it to the descriptor as `bridge-distribution-request`.

See: https://bugs.torproject.org/32753

diff --git a/src/feature/relay/router.c b/src/feature/relay/router.c
index 7ad0d1bd05c4d34ccb430d7720fcffa8f92f8967..2b86232279eeb49c8c74e7ba11fdb6d01aeac125 100644 (file)
--- a/src/feature/relay/router.c
+++ b/src/feature/relay/router.c
@@ -2908,14 +2908,21 @@ router_dump_router_to_string(routerinfo_t *router,
   }
 
   if (options->BridgeRelay) {
-    const char *bd;
+    char *bd = NULL;
+
     if (options->BridgeDistribution && strlen(options->BridgeDistribution)) {
-      bd = options->BridgeDistribution;
+      bd = tor_strdup(options->BridgeDistribution);
     } else {
-      bd = "any";
+      bd = tor_strdup("any");
     }
+
+    // Make sure our value is lowercased in the descriptor instead of just
+    // forwarding what the user wrote in their torrc directly.
+    tor_strlower(bd);
+
     smartlist_add_asprintf(chunks, "bridge-distribution-request %s\n",
                            escaped(bd));
+    tor_free(bd);
   }
 
   if (router->onion_curve25519_pkey) {