From e9ffd9890f6073bc2fc3be36be2fd521a4f3af96 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Thu, 21 Mar 2013 07:47:42 -0400 Subject: Remove check for whether bridge_line->digest is NULL It can never be NULL, since it's an array in bridge_line_t. Introduced in 266f8cddd87f. Found by coverity; this is CID 992691. Bug not in any released Tor. --- src/or/entrynodes.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'src/or/entrynodes.c') diff --git a/src/or/entrynodes.c b/src/or/entrynodes.c index 1ce78e5127..48eb309ce0 100644 --- a/src/or/entrynodes.c +++ b/src/or/entrynodes.c @@ -1766,8 +1766,7 @@ bridge_add_from_config(bridge_line_t *bridge_line) b = tor_malloc_zero(sizeof(bridge_info_t)); tor_addr_copy(&b->addr, &bridge_line->addr); b->port = bridge_line->port; - if (bridge_line->digest) - memcpy(b->identity, bridge_line->digest, DIGEST_LEN); + memcpy(b->identity, bridge_line->digest, DIGEST_LEN); if (bridge_line->transport_name) b->transport_name = bridge_line->transport_name; b->fetch_status.schedule = DL_SCHED_BRIDGE; -- cgit v1.2.3-54-g00ecf