From ab91302fd04d3ee03a2855c1ffb8570923f8d40c Mon Sep 17 00:00:00 2001 From: cypherpunks Date: Tue, 11 Sep 2018 00:41:32 +0000 Subject: [PATCH 1/2] nodelist: use strcmpstart() instead of strncmp() in rend_parse_v2_service_descriptor. Fix on c58675ca728f12b42f65e5b8964ae695c2e0ec2d --- changes/ticket27630 | 3 +++ src/feature/nodelist/routerparse.c | 3 +-- 2 files changed, 4 insertions(+), 2 deletions(-) create mode 100644 changes/ticket27630 diff --git a/changes/ticket27630 b/changes/ticket27630 new file mode 100644 index 0000000000..8a5f5d6b5b --- /dev/null +++ b/changes/ticket27630 @@ -0,0 +1,3 @@ + o Code simplification and refactoring: + - Use the simpler strcmpstart() helper in rend_parse_v2_service_descriptor + instead of strncmp(). Closes ticket 27630. diff --git a/src/feature/nodelist/routerparse.c b/src/feature/nodelist/routerparse.c index 73d320de40..b76b2974fa 100644 --- a/src/feature/nodelist/routerparse.c +++ b/src/feature/nodelist/routerparse.c @@ -5167,8 +5167,7 @@ rend_parse_v2_service_descriptor(rend_service_descriptor_t **parsed_out, tor_assert(desc); /* Check if desc starts correctly. */ - if (strncmp(desc, "rendezvous-service-descriptor ", - strlen("rendezvous-service-descriptor "))) { + if (strcmpstart(desc, "rendezvous-service-descriptor ")) { log_info(LD_REND, "Descriptor does not start correctly."); goto err; } From cb9fa3b04c7d35eb04bb5f695606bfafe97c88d1 Mon Sep 17 00:00:00 2001 From: cypherpunks Date: Tue, 11 Sep 2018 22:21:09 +0000 Subject: [PATCH 2/2] nodelist: fix docs for networkstatus_read_cached_consensus_impl Fix on c12d2cb2dcfe52aadf528a70433b468327472eaf. --- src/feature/nodelist/networkstatus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/feature/nodelist/networkstatus.c b/src/feature/nodelist/networkstatus.c index 6492b828b1..6e386628d7 100644 --- a/src/feature/nodelist/networkstatus.c +++ b/src/feature/nodelist/networkstatus.c @@ -201,7 +201,7 @@ networkstatus_reset_download_failures(void) /** * Read and and return the cached consensus of type flavorname. If - * unverified is false, get the one we haven't verified. Return NULL if + * unverified is true, get the one we haven't verified. Return NULL if * the file isn't there. */ static char * networkstatus_read_cached_consensus_impl(int flav,