]> git.puffer.fish Git - matthieu/frr.git/commitdiff
bgpd: No need to check for NULL before calling XFREE() in RPKI code
authorDonatas Abraitis <donatas@opensourcerouting.org>
Tue, 5 Apr 2022 08:12:22 +0000 (11:12 +0300)
committerDonatas Abraitis <donatas@opensourcerouting.org>
Tue, 5 Apr 2022 08:14:34 +0000 (11:14 +0300)
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
bgpd/bgp_rpki.c

index cbccc4423fd68c77c96c71e820381b860ab6872f..a7bc89e6485f98785fe8dd8eea319b0b1c549b99 100644 (file)
@@ -903,9 +903,8 @@ static void free_cache(struct cache *cache)
        if (cache->type == TCP) {
                XFREE(MTYPE_BGP_RPKI_CACHE, cache->tr_config.tcp_config->host);
                XFREE(MTYPE_BGP_RPKI_CACHE, cache->tr_config.tcp_config->port);
-               if (cache->tr_config.tcp_config->bindaddr)
-                       XFREE(MTYPE_BGP_RPKI_CACHE,
-                             cache->tr_config.tcp_config->bindaddr);
+               XFREE(MTYPE_BGP_RPKI_CACHE,
+                     cache->tr_config.tcp_config->bindaddr);
                XFREE(MTYPE_BGP_RPKI_CACHE, cache->tr_config.tcp_config);
        }
 #if defined(FOUND_SSH)
@@ -917,9 +916,8 @@ static void free_cache(struct cache *cache)
                      cache->tr_config.ssh_config->client_privkey_path);
                XFREE(MTYPE_BGP_RPKI_CACHE,
                      cache->tr_config.ssh_config->server_hostkey_path);
-               if (cache->tr_config.ssh_config->bindaddr)
-                       XFREE(MTYPE_BGP_RPKI_CACHE,
-                             cache->tr_config.ssh_config->bindaddr);
+               XFREE(MTYPE_BGP_RPKI_CACHE,
+                     cache->tr_config.ssh_config->bindaddr);
                XFREE(MTYPE_BGP_RPKI_CACHE, cache->tr_config.ssh_config);
        }
 #endif