summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorQuentin Young <qlyoung@cumulusnetworks.com>2017-05-26 17:21:36 +0200
committerQuentin Young <qlyoung@cumulusnetworks.com>2017-05-26 17:22:24 +0200
commit000fe2ca4c02010c7f9c63b9b86e7e29c9ceacd0 (patch)
tree3d3b2b2cc5d3cd34298d537ea73b16a01072e519
parent*: do not take address of packed member (diff)
downloadfrr-000fe2ca4c02010c7f9c63b9b86e7e29c9ceacd0.tar.xz
frr-000fe2ca4c02010c7f9c63b9b86e7e29c9ceacd0.zip
bgpd: remove struct capability_orf_entry
virtually unused Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
-rw-r--r--bgpd/bgp_open.c23
-rw-r--r--bgpd/bgp_open.h10
2 files changed, 12 insertions, 21 deletions
diff --git a/bgpd/bgp_open.c b/bgpd/bgp_open.c
index 437472966..cbec1a832 100644
--- a/bgpd/bgp_open.c
+++ b/bgpd/bgp_open.c
@@ -266,7 +266,8 @@ static int
bgp_capability_orf_entry (struct peer *peer, struct capability_header *hdr)
{
struct stream *s = BGP_INPUT (peer);
- struct capability_orf_entry entry;
+ struct capability_mp_data mpc;
+ u_char num;
iana_afi_t pkt_afi;
afi_t afi;
safi_t pkt_safi, safi;
@@ -277,14 +278,14 @@ bgp_capability_orf_entry (struct peer *peer, struct capability_header *hdr)
int i;
/* ORF Entry header */
- bgp_capability_mp_data (s, &entry.mpc);
- entry.num = stream_getc (s);
- pkt_afi = entry.mpc.afi;
- pkt_safi = entry.mpc.safi;
+ bgp_capability_mp_data (s, &mpc);
+ num = stream_getc (s);
+ pkt_afi = mpc.afi;
+ pkt_safi = mpc.safi;
if (bgp_debug_neighbor_events(peer))
zlog_debug ("%s ORF Cap entry for afi/safi: %u/%u",
- peer->host, entry.mpc.afi, entry.mpc.safi);
+ peer->host, mpc.afi, mpc.safi);
/* Convert AFI, SAFI to internal values, check. */
if (bgp_map_afi_safi_iana2int (pkt_afi, pkt_safi, &afi, &safi))
@@ -295,20 +296,20 @@ bgp_capability_orf_entry (struct peer *peer, struct capability_header *hdr)
return 0;
}
- entry.mpc.afi = pkt_afi;
- entry.mpc.safi = safi;
+ mpc.afi = pkt_afi;
+ mpc.safi = safi;
/* validate number field */
- if (CAPABILITY_CODE_ORF_LEN + (entry.num * 2) > hdr->length)
+ if (CAPABILITY_CODE_ORF_LEN + (num * 2) > hdr->length)
{
zlog_info ("%s ORF Capability entry length error,"
" Cap length %u, num %u",
- peer->host, hdr->length, entry.num);
+ peer->host, hdr->length, num);
bgp_notify_send (peer, BGP_NOTIFY_OPEN_ERR, BGP_NOTIFY_OPEN_MALFORMED_ATTR);
return -1;
}
- for (i = 0 ; i < entry.num ; i++)
+ for (i = 0 ; i < num ; i++)
{
type = stream_getc(s);
mode = stream_getc(s);
diff --git a/bgpd/bgp_open.h b/bgpd/bgp_open.h
index 4a9bcb2f7..0b0d9023e 100644
--- a/bgpd/bgp_open.h
+++ b/bgpd/bgp_open.h
@@ -36,16 +36,6 @@ struct capability_mp_data
safi_t safi;
};
-struct capability_orf_entry
-{
- struct capability_mp_data mpc;
- u_char num;
- struct {
- u_char type;
- u_char mode;
- } orfs[];
-};
-
struct capability_as4
{
uint32_t as4;