Hand-tune the new tor_memcmp instances in 0.2.3

diff --git a/src/or/microdesc.c b/src/or/microdesc.c
index 86a53d202f9edf40ec0eaf27f9570ea9d6132c65..ecab799612ebc2db5b20451b137fbdabce9a813a 100644 (file)
--- a/src/or/microdesc.c
+++ b/src/or/microdesc.c
@@ -465,7 +465,7 @@ microdesc_cache_rebuild(microdesc_cache_t *cache, int force)
     tor_assert(md->saved_location == SAVED_IN_CACHE);
     md->body = (char*)cache->cache_content->data + md->off;
     if (PREDICT_UNLIKELY(
-                 md->bodylen < 9 || tor_memcmp(md->body, "onion-key", 9) != 0)) {
+             md->bodylen < 9 || fast_memneq(md->body, "onion-key", 9) != 0)) {
       /* XXXX023 once bug 2022 is solved, we can kill this block and turn it
        * into just the tor_assert(!memcmp) */
       off_t avail = cache->cache_content->size - md->off;

diff --git a/src/or/nodelist.c b/src/or/nodelist.c
index ed7fe9c7a2e03124c9f9e84286d645c729e07b8b..0856b3117c1f1981e865d1a96d1606074a994e56 100644 (file)
--- a/src/or/nodelist.c
+++ b/src/or/nodelist.c
@@ -352,7 +352,7 @@ nodelist_assert_ok(void)
     SMARTLIST_FOREACH_BEGIN(rl->routers, routerinfo_t *, ri) {
       const node_t *node = node_get_by_id(ri->cache_info.identity_digest);
       tor_assert(node && node->ri == ri);
-      tor_assert(tor_memeq(ri->cache_info.identity_digest,
+      tor_assert(fast_memeq(ri->cache_info.identity_digest,
                              node->identity, DIGEST_LEN));
       tor_assert(! digestmap_get(dm, node->identity));
       digestmap_set(dm, node->identity, (void*)node);
@@ -364,7 +364,7 @@ nodelist_assert_ok(void)
     SMARTLIST_FOREACH_BEGIN(ns->routerstatus_list, routerstatus_t *, rs) {
       const node_t *node = node_get_by_id(rs->identity_digest);
       tor_assert(node && node->rs == rs);
-      tor_assert(tor_memeq(rs->identity_digest, node->identity, DIGEST_LEN));
+      tor_assert(fast_memeq(rs->identity_digest, node->identity, DIGEST_LEN));
       digestmap_set(dm, node->identity, (void*)node);
       if (ns->flavor == FLAV_MICRODESC) {
         /* If it's a microdesc consensus, every entry that has a
@@ -422,7 +422,7 @@ node_get_by_hex_id(const char *hex_id)
       if (nn_char == '=') {
         const char *named_id =
           networkstatus_get_router_digest_by_nickname(nn_buf);
-        if (!named_id || tor_memcmp(named_id, digest_buf, DIGEST_LEN))
+        if (!named_id || tor_memneq(named_id, digest_buf, DIGEST_LEN))
           return NULL;
       }
     }