]> git.puffer.fish Git - mirror/frr.git/commitdiff
Merge pull request #3869 from qlyoung/cocci-fixes
authorDavid Lamparter <equinox@diac24.net>
Wed, 6 Mar 2019 14:54:44 +0000 (15:54 +0100)
committerGitHub <noreply@github.com>
Wed, 6 Mar 2019 14:54:44 +0000 (15:54 +0100)
Assorted Coccinelle fixes

17 files changed:
1  2 
bgpd/bgp_aspath.c
bgpd/bgp_attr.c
bgpd/bgp_attr_evpn.c
bgpd/bgp_community.c
bgpd/bgp_ecommunity.c
bgpd/bgp_evpn.c
bgpd/bgp_lcommunity.c
bgpd/bgp_route.c
bgpd/bgp_vty.c
bgpd/bgp_zebra.c
bgpd/bgpd.c
bgpd/rfapi/rfapi_import.c
lib/if_rmap.c
lib/prefix.c
zebra/if_netlink.c
zebra/zebra_rib.c
zebra/zebra_vxlan.c

Simple merge
diff --cc bgpd/bgp_attr.c
Simple merge
Simple merge
Simple merge
Simple merge
diff --cc bgpd/bgp_evpn.c
Simple merge
Simple merge
Simple merge
diff --cc bgpd/bgp_vty.c
Simple merge
Simple merge
diff --cc bgpd/bgpd.c
Simple merge
Simple merge
diff --cc lib/if_rmap.c
index 53bbcbfe57b6548b542444031303779a0ab0263e,f8e500f43d514b014651a5d41b1919dc903ebca2..955c1417c4f47680480c04862e7e4572c1347cfb
@@@ -63,10 -62,9 +60,9 @@@ struct if_rmap *if_rmap_lookup(struct i
        /* temporary copy */
        key.ifname = (ifname) ? XSTRDUP(MTYPE_IF_RMAP_NAME, ifname) : NULL;
  
 -      if_rmap = hash_lookup(ifrmaphash, &key);
 +      if_rmap = hash_lookup(ctx->ifrmaphash, &key);
  
-       if (key.ifname)
-               XFREE(MTYPE_IF_RMAP_NAME, key.ifname);
+       XFREE(MTYPE_IF_RMAP_NAME, key.ifname);
  
        return if_rmap;
  }
@@@ -104,10 -98,9 +100,9 @@@ static struct if_rmap *if_rmap_get(stru
        /* temporary copy */
        key.ifname = (ifname) ? XSTRDUP(MTYPE_IF_RMAP_NAME, ifname) : NULL;
  
 -      ret = hash_get(ifrmaphash, &key, if_rmap_hash_alloc);
 +      ret = hash_get(ctx->ifrmaphash, &key, if_rmap_hash_alloc);
  
-       if (key.ifname)
-               XFREE(MTYPE_IF_RMAP_NAME, key.ifname);
+       XFREE(MTYPE_IF_RMAP_NAME, key.ifname);
  
        return ret;
  }
@@@ -133,12 -125,10 +128,10 @@@ static struct if_rmap *if_rmap_set(stru
  {
        struct if_rmap *if_rmap;
  
 -      if_rmap = if_rmap_get(ifname);
 +      if_rmap = if_rmap_get(ctx, ifname);
  
        if (type == IF_RMAP_IN) {
-               if (if_rmap->routemap[IF_RMAP_IN])
-                       XFREE(MTYPE_IF_RMAP_NAME,
-                             if_rmap->routemap[IF_RMAP_IN]);
+               XFREE(MTYPE_IF_RMAP_NAME, if_rmap->routemap[IF_RMAP_IN]);
                if_rmap->routemap[IF_RMAP_IN] =
                        XSTRDUP(MTYPE_IF_RMAP_NAME, routemap_name);
        }
diff --cc lib/prefix.c
Simple merge
Simple merge
Simple merge
Simple merge