resolved: simplify min_ttl check

rr is asserted upon a few lines above, no need to check for null.
Coverity-found issue, CID 1450844

  CID 1450844:  Null pointer dereferences  (REVERSE_INULL)
  Null-checking "rr" suggests that it may be null, but it has already
    been dereferenced on all paths leading to the check.

diff --git a/src/resolve/resolved-dns-cache.c b/src/resolve/resolved-dns-cache.c
index 9b2e7115c0a247634f8d59622c8a557694528f07..c019dc38c9dd41cce12713158cd7b4f7c5498b98 100644 (file)
--- a/src/resolve/resolved-dns-cache.c
+++ b/src/resolve/resolved-dns-cache.c
@@ -439,9 +439,7 @@ static int dns_cache_put_positive(
         /* Determine the minimal TTL of all RRs in the answer plus the one by the main RR we are supposed to
          * cache. Since we cache whole answers to questions we should never return answers where only some
          * RRs are still valid, hence find the lowest here */
-        min_ttl = dns_answer_min_ttl(answer);
-        if (rr)
-                min_ttl = MIN(min_ttl, rr->ttl);
+        min_ttl = MIN(dns_answer_min_ttl(answer), rr->ttl);
 
         /* New TTL is 0? Delete this specific entry... */
         if (min_ttl <= 0) {