summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--bgpd/bgp_open.c23
-rw-r--r--bgpd/bgp_open.h12
-rw-r--r--eigrpd/eigrp_fsm.c4
-rw-r--r--eigrpd/eigrp_interface.c4
-rw-r--r--eigrpd/eigrp_network.c38
-rw-r--r--eigrpd/eigrp_network.h4
-rw-r--r--eigrpd/eigrp_topology.c10
-rw-r--r--eigrpd/eigrp_update.c2
-rw-r--r--eigrpd/eigrp_zebra.c2
-rw-r--r--isisd/isis_pdu.c3
-rw-r--r--isisd/iso_checksum.c11
-rw-r--r--isisd/iso_checksum.h2
12 files changed, 51 insertions, 64 deletions
diff --git a/bgpd/bgp_open.c b/bgpd/bgp_open.c
index 4374729668..cbec1a8320 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 459b966dd2..0b0d9023e0 100644
--- a/bgpd/bgp_open.h
+++ b/bgpd/bgp_open.h
@@ -36,18 +36,6 @@ struct capability_mp_data
safi_t safi;
};
-#pragma pack(1)
-struct capability_orf_entry
-{
- struct capability_mp_data mpc;
- u_char num;
- struct {
- u_char type;
- u_char mode;
- } orfs[];
-} __attribute__ ((packed));
-#pragma pack()
-
struct capability_as4
{
uint32_t as4;
diff --git a/eigrpd/eigrp_fsm.c b/eigrpd/eigrp_fsm.c
index ad4eb70181..44a4747588 100644
--- a/eigrpd/eigrp_fsm.c
+++ b/eigrpd/eigrp_fsm.c
@@ -434,8 +434,8 @@ int eigrp_fsm_event_keep_state(struct eigrp_fsm_action_message *msg)
if (prefix->state == EIGRP_FSM_STATE_PASSIVE)
{
- if (!eigrp_metrics_is_same(&prefix->reported_metric,
- &((struct eigrp_neighbor_entry *) prefix->entries->head->data)->total_metric))
+ if (!eigrp_metrics_is_same(prefix->reported_metric,
+ ((struct eigrp_neighbor_entry *) prefix->entries->head->data)->total_metric))
{
prefix->rdistance =
prefix->fdistance =
diff --git a/eigrpd/eigrp_interface.c b/eigrpd/eigrp_interface.c
index 14829be31a..26643452cb 100644
--- a/eigrpd/eigrp_interface.c
+++ b/eigrpd/eigrp_interface.c
@@ -310,7 +310,7 @@ eigrp_if_up (struct eigrp_interface *ei)
pe->nt = EIGRP_TOPOLOGY_TYPE_CONNECTED;
pe->state = EIGRP_FSM_STATE_PASSIVE;
- pe->fdistance = eigrp_calculate_metrics (eigrp, &metric);
+ pe->fdistance = eigrp_calculate_metrics (eigrp, metric);
pe->req_action |= EIGRP_FSM_NEED_UPDATE;
eigrp_prefix_entry_add (eigrp->topology_table, pe);
listnode_add(eigrp->topology_changes_internalIPV4, pe);
@@ -319,7 +319,7 @@ eigrp_if_up (struct eigrp_interface *ei)
ne->ei = ei;
ne->reported_metric = metric;
ne->total_metric = metric;
- ne->distance = eigrp_calculate_metrics (eigrp, &metric);
+ ne->distance = eigrp_calculate_metrics (eigrp, metric);
ne->reported_distance = 0;
ne->prefix = pe;
ne->adv_router = eigrp->neighbor_self;
diff --git a/eigrpd/eigrp_network.c b/eigrpd/eigrp_network.c
index 45d91025d1..cfed11a9eb 100644
--- a/eigrpd/eigrp_network.c
+++ b/eigrpd/eigrp_network.c
@@ -387,29 +387,29 @@ eigrp_network_unset(struct eigrp *eigrp, struct prefix_ipv4 *p)
}
u_int32_t
-eigrp_calculate_metrics(struct eigrp *eigrp, struct eigrp_metrics *metric)
+eigrp_calculate_metrics(struct eigrp *eigrp, struct eigrp_metrics metric)
{
uint64_t temp_metric;
temp_metric = 0;
- if(metric->delay == EIGRP_MAX_METRIC)
+ if(metric.delay == EIGRP_MAX_METRIC)
return EIGRP_MAX_METRIC;
// EIGRP Metric = {K1*BW+[(K2*BW)/(256-load)]+(K3*delay)}*{K5/(reliability+K4)}
if (eigrp->k_values[0])
- temp_metric += (eigrp->k_values[0] * metric->bandwith);
+ temp_metric += (eigrp->k_values[0] * metric.bandwith);
if (eigrp->k_values[1])
- temp_metric += ((eigrp->k_values[1] * metric->bandwith)
- / (256 - metric->load));
+ temp_metric += ((eigrp->k_values[1] * metric.bandwith)
+ / (256 - metric.load));
if (eigrp->k_values[2])
- temp_metric += (eigrp->k_values[2] * metric->delay);
+ temp_metric += (eigrp->k_values[2] * metric.delay);
if (eigrp->k_values[3] && !eigrp->k_values[4])
temp_metric *= eigrp->k_values[3];
if (!eigrp->k_values[3] && eigrp->k_values[4])
- temp_metric *= (eigrp->k_values[4] / metric->reliability);
+ temp_metric *= (eigrp->k_values[4] / metric.reliability);
if (eigrp->k_values[3] && eigrp->k_values[4])
- temp_metric *= ((eigrp->k_values[4] / metric->reliability)
+ temp_metric *= ((eigrp->k_values[4] / metric.reliability)
+ eigrp->k_values[3]);
if (temp_metric <= EIGRP_MAX_METRIC)
@@ -432,21 +432,21 @@ eigrp_calculate_total_metrics(struct eigrp *eigrp,
entry->total_metric.bandwith =
entry->total_metric.bandwith > bw ? bw : entry->total_metric.bandwith;
- return eigrp_calculate_metrics(eigrp, &entry->total_metric);
+ return eigrp_calculate_metrics(eigrp, entry->total_metric);
}
u_char
-eigrp_metrics_is_same(struct eigrp_metrics *metric1,
- struct eigrp_metrics *metric2)
+eigrp_metrics_is_same(struct eigrp_metrics metric1,
+ struct eigrp_metrics metric2)
{
- if ((metric1->bandwith == metric2->bandwith)
- && (metric1->delay == metric2->delay)
- && (metric1->hop_count == metric2->hop_count)
- && (metric1->load == metric2->load)
- && (metric1->reliability == metric2->reliability)
- && (metric1->mtu[0] == metric2->mtu[0])
- && (metric1->mtu[1] == metric2->mtu[1])
- && (metric1->mtu[2] == metric2->mtu[2]))
+ if ((metric1.bandwith == metric2.bandwith)
+ && (metric1.delay == metric2.delay)
+ && (metric1.hop_count == metric2.hop_count)
+ && (metric1.load == metric2.load)
+ && (metric1.reliability == metric2.reliability)
+ && (metric1.mtu[0] == metric2.mtu[0])
+ && (metric1.mtu[1] == metric2.mtu[1])
+ && (metric1.mtu[2] == metric2.mtu[2]))
return 1;
return 0; // if different
diff --git a/eigrpd/eigrp_network.h b/eigrpd/eigrp_network.h
index bcca2609a8..82ac455c96 100644
--- a/eigrpd/eigrp_network.h
+++ b/eigrpd/eigrp_network.h
@@ -43,9 +43,9 @@ extern int eigrp_if_drop_allspfrouters (struct eigrp *top, struct prefix *p,
unsigned int ifindex);
extern void eigrp_adjust_sndbuflen (struct eigrp *, unsigned int);
-extern u_int32_t eigrp_calculate_metrics (struct eigrp *, struct eigrp_metrics *);
+extern u_int32_t eigrp_calculate_metrics (struct eigrp *, struct eigrp_metrics);
extern u_int32_t eigrp_calculate_total_metrics (struct eigrp *, struct eigrp_neighbor_entry *);
-extern u_char eigrp_metrics_is_same(struct eigrp_metrics *,struct eigrp_metrics *);
+extern u_char eigrp_metrics_is_same(struct eigrp_metrics, struct eigrp_metrics);
extern void eigrp_external_routes_refresh (struct eigrp *, int);
#endif /* EIGRP_NETWORK_H_ */
diff --git a/eigrpd/eigrp_topology.c b/eigrpd/eigrp_topology.c
index d422b450ed..245b6adce8 100644
--- a/eigrpd/eigrp_topology.c
+++ b/eigrpd/eigrp_topology.c
@@ -423,24 +423,24 @@ eigrp_topology_update_distance(struct eigrp_fsm_action_message *msg)
if (msg->data_type == EIGRP_TLV_IPv4_INT)
{
int_data = msg->data.ipv4_int_type;
- if (eigrp_metrics_is_same(&int_data->metric,&entry->reported_metric))
+ if (eigrp_metrics_is_same(int_data->metric, entry->reported_metric))
{
return 0; // No change
}
change =
entry->reported_distance
- < eigrp_calculate_metrics(eigrp, &int_data->metric) ? 1 :
+ < eigrp_calculate_metrics(eigrp, int_data->metric) ? 1 :
entry->reported_distance
- > eigrp_calculate_metrics(eigrp, &int_data->metric) ? 2 : 3; // Increase : Decrease : No change
+ > eigrp_calculate_metrics(eigrp, int_data->metric) ? 2 : 3; // Increase : Decrease : No change
entry->reported_metric = int_data->metric;
entry->reported_distance =
- eigrp_calculate_metrics(eigrp, &int_data->metric);
+ eigrp_calculate_metrics(eigrp, int_data->metric);
entry->distance = eigrp_calculate_total_metrics(eigrp, entry);
}
else
{
ext_data = msg->data.ipv4_ext_data;
- if (eigrp_metrics_is_same (&ext_data->metric, &entry->reported_metric))
+ if (eigrp_metrics_is_same (ext_data->metric, entry->reported_metric))
return 0;
}
/*
diff --git a/eigrpd/eigrp_update.c b/eigrpd/eigrp_update.c
index 98b2defea0..164907885b 100644
--- a/eigrpd/eigrp_update.c
+++ b/eigrpd/eigrp_update.c
@@ -344,7 +344,7 @@ eigrp_update_receive (struct eigrp *eigrp, struct ip *iph, struct eigrp_header *
ne->reported_metric = tlv->metric;
ne->reported_distance =
eigrp_calculate_metrics(eigrp,
- &tlv->metric);
+ tlv->metric);
/*
* Filtering
*/
diff --git a/eigrpd/eigrp_zebra.c b/eigrpd/eigrp_zebra.c
index ba6ecf2452..2baad75265 100644
--- a/eigrpd/eigrp_zebra.c
+++ b/eigrpd/eigrp_zebra.c
@@ -509,7 +509,7 @@ eigrp_redistribute_set (struct eigrp *eigrp, int type, struct eigrp_metrics metr
if (eigrp_is_type_redistributed (type))
{
- if (eigrp_metrics_is_same(&metric, &eigrp->dmetric[type]))
+ if (eigrp_metrics_is_same(metric, eigrp->dmetric[type]))
{
eigrp->dmetric[type] = metric;
}
diff --git a/isisd/isis_pdu.c b/isisd/isis_pdu.c
index 69beade477..d62682b10f 100644
--- a/isisd/isis_pdu.c
+++ b/isisd/isis_pdu.c
@@ -1315,7 +1315,8 @@ process_lsp (int level, struct isis_circuit *circuit, const u_char *ssnpa)
/* Checksum sanity check - FIXME: move to correct place */
/* 12 = sysid+pdu+remtime */
if (iso_csum_verify (STREAM_PNT (circuit->rcv_stream) + 4,
- pdu_len - 12, &hdr->checksum))
+ pdu_len - 12, hdr->checksum,
+ offsetof(struct isis_link_state_hdr, checksum) - 4))
{
zlog_debug ("ISIS-Upd (%s): LSP %s invalid LSP checksum 0x%04x",
circuit->area->area_tag,
diff --git a/isisd/iso_checksum.c b/isisd/iso_checksum.c
index d036c0ba76..70b6b91edb 100644
--- a/isisd/iso_checksum.c
+++ b/isisd/iso_checksum.c
@@ -46,14 +46,14 @@
*/
int
-iso_csum_verify (u_char * buffer, int len, uint16_t * csum)
+iso_csum_verify (u_char * buffer, int len, uint16_t csum, int offset)
{
u_int16_t checksum;
u_int32_t c0;
u_int32_t c1;
- c0 = *csum & 0xff00;
- c1 = *csum & 0x00ff;
+ c0 = csum & 0xff00;
+ c1 = csum & 0x00ff;
/*
* If both are zero return correct
@@ -67,11 +67,8 @@ iso_csum_verify (u_char * buffer, int len, uint16_t * csum)
if (c0 == 0 || c1 == 0)
return 1;
- /* Offset of checksum from the start of the buffer */
- int offset = (u_char *) csum - buffer;
-
checksum = fletcher_checksum(buffer, len, offset);
- if (checksum == *csum)
+ if (checksum == csum)
return 0;
return 1;
}
diff --git a/isisd/iso_checksum.h b/isisd/iso_checksum.h
index cca6ee24a4..50f6a7d560 100644
--- a/isisd/iso_checksum.h
+++ b/isisd/iso_checksum.h
@@ -23,6 +23,6 @@
#ifndef _ZEBRA_ISO_CSUM_H
#define _ZEBRA_ISO_CSUM_H
-int iso_csum_verify (u_char * buffer, int len, uint16_t * csum);
+int iso_csum_verify (u_char * buffer, int len, uint16_t csum, int offset);
#endif /* _ZEBRA_ISO_CSUM_H */
generated by cgit v1.2.3 (git 2.39.1) at 2025-11-25 03:56:36 +0000