summaryrefslogtreecommitdiffstats
path: root/bgpd/bgp_attr.c
diff options
context:
space:
mode:
authorJorge Boncompte <jbonor@gmail.com>2017-08-04 19:33:15 +0200
committerJorge Boncompte <jbonor@gmail.com>2017-08-04 19:33:15 +0200
commit5ddb31a1fffca4a5a767cd7cf06ef7a7b6ca17ae (patch)
treeb1cd2684744b8782eaea6e171544a6e8df0c7758 /bgpd/bgp_attr.c
parentMerge pull request #906 from opensourcerouting/vty-close (diff)
downloadfrr-5ddb31a1fffca4a5a767cd7cf06ef7a7b6ca17ae.tar.xz
frr-5ddb31a1fffca4a5a767cd7cf06ef7a7b6ca17ae.zip
bgpd: remove bogus change in attrhash_key_make()
Commit c8e7b895 ("bgpd: use Jenkins hash for BGP transit, cluster and attr hashes") changed attrhash_key_make() to use Jenkins hash, commit c8f3fe30 ("bgpd: Remove AS Path limit/TTL functionality") introduced a bogus change with a snippet of code that was deleted in the first one. Signed-off-by: Jorge Boncompte <jbonor@gmail.com>
Diffstat (limited to '')
-rw-r--r--bgpd/bgp_attr.c6
1 files changed, 0 insertions, 6 deletions
diff --git a/bgpd/bgp_attr.c b/bgpd/bgp_attr.c
index ca9a50db5..dd1879763 100644
--- a/bgpd/bgp_attr.c
+++ b/bgpd/bgp_attr.c
@@ -566,12 +566,6 @@ unsigned int attrhash_key_make(void *p)
MIX(attr->nexthop.s_addr);
MIX(attr->med);
MIX(attr->local_pref);
-
- key += attr->origin;
- key += attr->nexthop.s_addr;
- key += attr->med;
- key += attr->local_pref;
-
MIX(attr->aggregator_as);
MIX(attr->aggregator_addr.s_addr);
MIX(attr->weight);