summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorpaul <paul>2002-12-13 20:56:48 +0000
committerpaul <paul>2002-12-13 20:56:48 +0000
commit30961a15cf3837b62c65d4c0ecf3ee05f4b6bf55 (patch)
treeb70d64c8b4d08d745311262cf3366bda4b2260a1
parent9483e15540359cbd90b3f6481f1a7390a92167a7 (diff)
Greg Troxel <gdt@ir.bbn.com>
[zebra 15715] FIX for ospf md5 authentication problem, finally! fix copy of ospf packet buffer
-rw-r--r--ospfd/ospf_packet.c15
1 files changed, 13 insertions, 2 deletions
diff --git a/ospfd/ospf_packet.c b/ospfd/ospf_packet.c
index 95d184dc..62b93fef 100644
--- a/ospfd/ospf_packet.c
+++ b/ospfd/ospf_packet.c
@@ -226,7 +226,13 @@ ospf_packet_dup (struct ospf_packet *op)
{
struct ospf_packet *new;
- new = ospf_packet_new (op->length);
+ if (stream_get_endp(op->s) != op->length) {
+ zlog_warn ("ospf_packet_dup stream %d ospf_packet %d size mismatch",
+ STREAM_SIZE(op->s), op->length);
+ }
+
+ /* Reserve space for MD5 authentication that may be added later. */
+ new = ospf_packet_new (stream_get_endp(op->s) + OSPF_AUTH_MD5_SIZE);
ospf_stream_copy (new->s, op->s);
new->dst = op->dst;
@@ -359,7 +365,12 @@ ospf_make_md5_digest (struct ospf_interface *oi, struct ospf_packet *op)
stream_set_putp (op->s, oldputp);
/* We do *NOT* increment the OSPF header length. */
- op->length += OSPF_AUTH_MD5_SIZE;
+ op->length = ntohs (ospfh->length) + OSPF_AUTH_MD5_SIZE;
+
+ if (stream_get_endp(op->s) != op->length) {
+ zlog_warn("ospf_make_md5_digest: length mismatch stream %d ospf_packet %d",
+ stream_get_endp(op->s), op->length);
+ }
return OSPF_AUTH_MD5_SIZE;
}