diff options
| -rw-r--r-- | changes/ticket27630 | 3 | ||||
| -rw-r--r-- | src/feature/nodelist/networkstatus.c | 2 | ||||
| -rw-r--r-- | src/feature/nodelist/routerparse.c | 3 |
3 files changed, 5 insertions, 3 deletions
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/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 <b>flavorname</b>. If - * <b>unverified</b> is false, get the one we haven't verified. Return NULL if + * <b>unverified</b> 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, 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; } |