tor

commit 362229f184c92712ee5a3027ce5d29024ca5b382
parent 5de66739724451801487dbec0eb210c0595cc00f
Author: David Goulet <dgoulet@torproject.org>
Date:   Tue, 27 Oct 2020 12:59:29 -0400

lib/metrics: Fix wrong macro expansion

The "METRICS_PREFIX" was not expanded but rather used as a litteral. Fix that
by just removing the define and using "tor_" directly.

Reviewed-by: Alexander Færøy <ahf@torproject.org>
Signed-off-by: David Goulet <dgoulet@torproject.org>

Diffstat:
M
src/lib/metrics/metrics_common.h
 | 
6
+-----
M
src/test/test_hs_metrics.c
 | 
2
+-

2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/src/lib/metrics/metrics_common.h b/src/lib/metrics/metrics_common.h
@@ -11,14 +11,10 @@
 
 #include "lib/cc/torint.h"
 
-/** Prefix to every metrics exposed. This is insures that the metrics are
- * always in the same namespace. */
-#define METRICS_PREFIX tor_
-
 /** Helper macro that must be used to construct the right namespaced metrics
  * name. A name is a string so stringify the result. */
 #define METRICS_STR(val) #val
-#define METRICS_NAME(name) METRICS_STR(METRICS_PREFIX ## name)
+#define METRICS_NAME(name) METRICS_STR(tor_ ## name)
 
 /** Format output type. */
 typedef enum {
diff --git a/src/test/test_hs_metrics.c b/src/test/test_hs_metrics.c
@@ -44,7 +44,7 @@ test_metrics(void *arg)
 
   /* Confirm the entry value. */
   const smartlist_t *entries = metrics_store_get_all(service->metrics.store,
-                                                     "hs_intro_num_total");
+                                                     "tor_hs_intro_num_total");
   tt_assert(entries);
   tt_int_op(smartlist_len(entries), OP_EQ, 1);
   const metrics_store_entry_t *entry = smartlist_get(entries, 0);