Make static analysis tools happier about peerDigestLookup() (#1593)

Some tools thought that `key` could be nil. Refactor a bit to convince
them otherwise.

diff --git a/src/neighbors.cc b/src/neighbors.cc
index 54835f9128cbb68637d83cd17540f98026cff834..a2719f628f88eca4da62e6aaa9a1c52a51942fd0 100644 (file)
--- a/src/neighbors.cc
+++ b/src/neighbors.cc
@@ -707,9 +707,9 @@ peerDigestLookup(CachePeer * p, PeerSelector * ps)
 #if USE_CACHE_DIGESTS
     assert(ps);
     HttpRequest *request = ps->request;
-    const cache_key *key = request ? storeKeyPublicByRequest(request) : nullptr;
-    assert(p);
     assert(request);
+
+    assert(p);
     debugs(15, 5, "cache_peer " << *p);
     /* does the peeer have a valid digest? */
 
@@ -732,7 +732,7 @@ peerDigestLookup(CachePeer * p, PeerSelector * ps)
     assert(p->digest->cd);
     /* does digest predict a hit? */
 
-    if (!p->digest->cd->contains(key))
+    if (!p->digest->cd->contains(storeKeyPublicByRequest(request)))
         return LOOKUP_MISS;
 
     debugs(15, 5, "HIT for cache_peer " << *p);