Cleanup in packet.c. Deleted unused parameters.

diff --git a/proto/ospf/dbdes.c b/proto/ospf/dbdes.c
index 3d8ec50a8266f7656c89f4b284b9a7bb76cd981b..54dd71fe2abf410dfcefb70f925e542c43970158 100644 (file)
--- a/proto/ospf/dbdes.c
+++ b/proto/ospf/dbdes.c
@@ -182,26 +182,11 @@ ospf_dbdes_reqladd(struct ospf_dbdes_packet *ps, struct ospf_neighbor *n)
 
 void
 ospf_dbdes_receive(struct ospf_dbdes_packet *ps,
-                  struct ospf_iface *ifa, u16 size)
+                  struct ospf_iface *ifa, struct ospf_neighbor *n)
 {
   struct proto *p = (struct proto *) ifa->proto;
-  u32 nrid, myrid = p->cf->global->router_id;
-  struct ospf_neighbor *n;
-
-  nrid = ntohl(((struct ospf_packet *) ps)->routerid);
-
-
-  if ((n = find_neigh(ifa, nrid)) == NULL)
-  {
-    OSPF_TRACE(D_PACKETS, "Received dbdes from unknown neigbor! %I.", nrid);
-    return;
-  }
-
-  if (ifa->iface->mtu < size)
-  {
-    OSPF_TRACE(D_PACKETS, "Received dbdes larger than MTU from %I!", n->ip);
-    return;
-  }
+  u32 myrid = p->cf->global->router_id;
+  unsigned int size = ntohs(ps->ospf_packet.length);
 
   OSPF_TRACE(D_PACKETS, "Received dbdes from %I via %s.", n->ip,
             ifa->iface->name);

diff --git a/proto/ospf/dbdes.h b/proto/ospf/dbdes.h
index b332ecb2ee283abe3889cbcff1c5cb0341d2e983..c5ffe59092e76e05f4d2527353dd6185256b79ad 100644 (file)
--- a/proto/ospf/dbdes.h
+++ b/proto/ospf/dbdes.h
@@ -12,6 +12,6 @@
 
 void ospf_dbdes_send(struct ospf_neighbor *n);
 void ospf_dbdes_receive(struct ospf_dbdes_packet *ps,
-                       struct ospf_iface *ifa, u16 size);
+                       struct ospf_iface *ifa, struct ospf_neighbor *n);
 
 #endif /* _BIRD_OSPF_DBDES_H_ */

diff --git a/proto/ospf/hello.c b/proto/ospf/hello.c
index 6e1fa92ad8d874575e27093e2d01177ed2d78274..a318a674e62589fae6c54926e1d7ca26b3f4c22a 100644 (file)
--- a/proto/ospf/hello.c
+++ b/proto/ospf/hello.c
@@ -10,18 +10,14 @@
 
 void
 ospf_hello_receive(struct ospf_hello_packet *ps,
-                  struct ospf_iface *ifa, int size, ip_addr faddr)
+                  struct ospf_iface *ifa, struct ospf_neighbor *n, ip_addr faddr)
 {
-  u32 nrid, *pnrid;
-  struct ospf_neighbor *n;
-  u8 i, twoway, oldpriority;
+  u32 *pnrid;
   ip_addr olddr, oldbdr;
   ip_addr mask;
   char *beg = "Bad OSPF hello packet from ", *rec = " received: ";
-  int eligible = 0;
   struct proto *p = (struct proto *) ifa->proto;
-
-  nrid = ntohl(((struct ospf_packet *) ps)->routerid);
+  unsigned int size = ntohs(ps->ospf_packet.length), i, twoway, oldpriority, eligible = 0;
 
   OSPF_TRACE(D_PACKETS, "Received hello from %I via %s", faddr,
             ifa->iface->name);
@@ -30,7 +26,7 @@ ospf_hello_receive(struct ospf_hello_packet *ps,
 
   if ((unsigned) ipa_mklen(mask) != ifa->iface->addr->pxlen)
   {
-    log(L_ERR, "%s%I%sbad netmask %I.", beg, nrid, rec, mask);
+    log(L_ERR "%s%I%sbad netmask %I.", beg, faddr, rec, mask);
     return;
   }
 
@@ -54,7 +50,7 @@ ospf_hello_receive(struct ospf_hello_packet *ps,
     return;
   }
 
-  if ((n = find_neigh(ifa, nrid)) == NULL)
+  if (!n)
   {
     if ((ifa->type == OSPF_IT_NBMA))
     {
@@ -92,7 +88,7 @@ ospf_hello_receive(struct ospf_hello_packet *ps,
 
     n = ospf_neighbor_new(ifa);
 
-    n->rid = nrid;
+    n->rid = ntohl(((struct ospf_packet *) ps)->routerid);
     n->ip = faddr;
     n->dr = ps->dr;
     ipa_ntoh(n->dr);

diff --git a/proto/ospf/hello.h b/proto/ospf/hello.h
index 5b41857c8094cfbcec69790dc680a67e40cc1efa..d535935232362ba9a57d201e5895cfdd52f07744 100644 (file)
--- a/proto/ospf/hello.h
+++ b/proto/ospf/hello.h
@@ -11,7 +11,7 @@
 #define _BIRD_OSPF_HELLO_H_
 
 void ospf_hello_receive(struct ospf_hello_packet *ps,
-                       struct ospf_iface *ifa, int size, ip_addr faddr);
-void ospf_hello_send(timer * timer, int poll, struct ospf_neighbor *dirn);
+                       struct ospf_iface *ifa, struct ospf_neighbor *n, ip_addr faddr);
+void ospf_hello_send(timer *timer, int poll, struct ospf_neighbor *dirn);
 
 #endif /* _BIRD_OSPF_HELLO_H_ */

diff --git a/proto/ospf/lsack.c b/proto/ospf/lsack.c
index 3a4e965276fb2560a743059bc0cd0f5d3c2c67a3..7d7238c2169df37141b3c0a7b8d1b4d5469a149b 100644 (file)
--- a/proto/ospf/lsack.c
+++ b/proto/ospf/lsack.c
@@ -119,25 +119,13 @@ ospf_lsack_send(struct ospf_neighbor *n, int queue)
 
 void
 ospf_lsack_receive(struct ospf_lsack_packet *ps,
-                  struct ospf_iface *ifa, u16 size)
+                  struct ospf_iface *ifa, struct ospf_neighbor *n)
 {
-  u32 nrid, myrid;
-  struct ospf_neighbor *n;
   struct ospf_lsa_header lsa, *plsa;
-  u16 nolsa, i;
+  u16 nolsa;
   struct top_hash_entry *en;
-  u16 lenn = ntohs(ps->ospf_packet.length);
   struct proto *p = (struct proto *) ifa->proto;
-
-  nrid = ntohl(ps->ospf_packet.routerid);
-
-  myrid = p->cf->global->router_id;
-
-  if ((n = find_neigh(ifa, nrid)) == NULL)
-  {
-    OSPF_TRACE(D_PACKETS, "Received LS ack from unknown neigbor! (%I)", nrid);
-    return;
-  }
+  unsigned int size = ntohs(ps->ospf_packet.length), i;
 
   OSPF_TRACE(D_PACKETS, "Received LS ack from %I", n->ip);
   ospf_neigh_sm(n, INM_HELLOREC);
@@ -145,10 +133,10 @@ ospf_lsack_receive(struct ospf_lsack_packet *ps,
   if (n->state < NEIGHBOR_EXCHANGE)
     return;
 
-  nolsa = (lenn - sizeof(struct ospf_lsack_packet)) /
+  nolsa = (size - sizeof(struct ospf_lsack_packet)) /
     sizeof(struct ospf_lsa_header);
 
-  if ((nolsa < 1) || ((lenn - sizeof(struct ospf_lsack_packet)) !=
+  if ((nolsa < 1) || ((size - sizeof(struct ospf_lsack_packet)) !=
                      (nolsa * sizeof(struct ospf_lsa_header))))
   {
     log(L_ERR "Received corrupted LS ack from %I", n->ip);

diff --git a/proto/ospf/lsack.h b/proto/ospf/lsack.h
index d56abc5463b78e51e4f40c8ff8ac7490a6bb023f..05cc22f09607d10f8a67e8f53fa2cea58a363794 100644 (file)
--- a/proto/ospf/lsack.h
+++ b/proto/ospf/lsack.h
@@ -17,7 +17,7 @@ struct lsah_n
 };
 
 void ospf_lsack_receive(struct ospf_lsack_packet *ps,
-                       struct ospf_iface *ifa, u16 size);
+                       struct ospf_iface *ifa, struct ospf_neighbor *n);
 void ospf_lsack_send(struct ospf_neighbor *n, int queue);
 void ospf_lsack_enqueue(struct ospf_neighbor *n, struct ospf_lsa_header *h,
                        int queue);

diff --git a/proto/ospf/lsreq.c b/proto/ospf/lsreq.c
index 046e14951490f3621c6302c79bd42335755a09e9..98d2d06c9d525564aaaa9482bf1b83ea7f2f624c 100644 (file)
--- a/proto/ospf/lsreq.c
+++ b/proto/ospf/lsreq.c
@@ -66,36 +66,27 @@ ospf_lsreq_send(struct ospf_neighbor *n)
 
 void
 ospf_lsreq_receive(struct ospf_lsreq_packet *ps,
-                  struct ospf_iface *ifa, u16 size)
+                  struct ospf_iface *ifa, struct ospf_neighbor *n)
 {
-  u32 nrid;
-  struct ospf_neighbor *n;
   struct ospf_lsreq_header *lsh;
   struct l_lsr_head *llsh;
   list uplist;
   slab *upslab;
-  int length, i, lsano;
+  unsigned int size = ntohs(ps->ospf_packet.length);
+  int i, lsano;
   struct proto *p = (struct proto *) ifa->proto;
 
-  nrid = ntohl(ps->ospf_packet.routerid);
-
-  if ((n = find_neigh(ifa, nrid)) == NULL)
-  {
-    OSPF_TRACE(D_PACKETS, "Received lsreq from unknown neighbor! (%I)", nrid);
-    return;
-  }
   if (n->state < NEIGHBOR_EXCHANGE)
     return;
 
   OSPF_TRACE(D_EVENTS, "Received LS req from neighbor: %I", n->ip);
   ospf_neigh_sm(n, INM_HELLOREC);
 
-  length = ntohs(ps->ospf_packet.length);
   lsh = (void *) (ps + 1);
   init_list(&uplist);
   upslab = sl_new(n->pool, sizeof(struct l_lsr_head));
 
-  lsano = (length - sizeof(struct ospf_lsreq_packet)) /
+  lsano = (size - sizeof(struct ospf_lsreq_packet)) /
     sizeof(struct ospf_lsreq_header);
   for (i = 0; i < lsano; lsh++, i++)
   {

diff --git a/proto/ospf/lsreq.h b/proto/ospf/lsreq.h
index 90a4e82bbce2218e7f53738835353e06fd902c1d..f917f05ce6c97b4ee6bad8463236cd942c360b90 100644 (file)
--- a/proto/ospf/lsreq.h
+++ b/proto/ospf/lsreq.h
@@ -12,6 +12,6 @@
 
 void ospf_lsreq_send(struct ospf_neighbor *n);
 void ospf_lsreq_receive(struct ospf_lsreq_packet *ps,
-                       struct ospf_iface *ifa, u16 size);
+                       struct ospf_iface *ifa, struct ospf_neighbor *n);
 
 #endif /* _BIRD_OSPF_LSREQ_H_ */

diff --git a/proto/ospf/lsupd.c b/proto/ospf/lsupd.c
index 52531bf5277a7f6bc9cccbd35fb43ca6b02906d0..2211891f4fac3683eb3591483e2e7ff71df01aa6 100644 (file)
--- a/proto/ospf/lsupd.c
+++ b/proto/ospf/lsupd.c
@@ -283,24 +283,15 @@ ospf_lsupd_send_list(struct ospf_neighbor *n, list * l)
 
 void
 ospf_lsupd_receive(struct ospf_lsupd_packet *ps,
-                  struct ospf_iface *ifa, u16 size)
+                  struct ospf_iface *ifa, struct ospf_neighbor *n)
 {
-  u32 area, nrid;
-  struct ospf_neighbor *n, *ntmp;
+  u32 area;
+  struct ospf_neighbor *ntmp;
   struct ospf_lsa_header *lsa;
   struct ospf_area *oa;
   struct proto_ospf *po = ifa->proto;
   struct proto *p = (struct proto *) po;
-  u8 i;
-  int sendreq = 1;
-
-  nrid = ntohl(ps->ospf_packet.routerid);
-
-  if ((n = find_neigh(ifa, nrid)) == NULL)
-  {
-    OSPF_TRACE(D_PACKETS, "Received lsupd from unknown neighbor! (%I)", nrid);
-    return;
-  }
+  unsigned int i, sendreq = 1, size = ntohs(ps->ospf_packet.length);
 
   if (n->state < NEIGHBOR_EXCHANGE)
   {
@@ -329,8 +320,8 @@ ospf_lsupd_receive(struct ospf_lsupd_packet *ps,
   {
     struct ospf_lsa_header lsatmp;
     struct top_hash_entry *lsadb;
-    int diff = ((u8 *) lsa) - ((u8 *) ps);
-    u16 chsum, lenn = ntohs(lsa->length);
+    unsigned diff = ((u8 *) lsa) - ((u8 *) ps), lenn = ntohs(lsa->length);
+    u16 chsum;
 
     if (((diff + sizeof(struct ospf_lsa_header)) >= size)
        || ((lenn + diff) > size))

diff --git a/proto/ospf/lsupd.h b/proto/ospf/lsupd.h
index 8c046759568ffa196da1e7de7255801248b61e1b..64d9aeff94af639cbe0bb2367d4de062a90380af 100644 (file)
--- a/proto/ospf/lsupd.h
+++ b/proto/ospf/lsupd.h
@@ -12,7 +12,7 @@
 
 void ospf_lsupd_send_list(struct ospf_neighbor *n, list * l);
 void ospf_lsupd_receive(struct ospf_lsupd_packet *ps,
-                       struct ospf_iface *ifa, u16 size);
+                       struct ospf_iface *ifa, struct ospf_neighbor *n);
 int ospf_lsupd_flood(struct ospf_neighbor *n, struct ospf_lsa_header *hn,
                     struct ospf_lsa_header *hh, struct ospf_iface *iff,
                     struct ospf_area *oa, int rtl);

diff --git a/proto/ospf/packet.c b/proto/ospf/packet.c
index 714ca8064cff4fd66597657be49eec05b3037ff4..ef3066ed339c762c14f6add20b1ac366fcee7182 100644 (file)
--- a/proto/ospf/packet.c
+++ b/proto/ospf/packet.c
@@ -84,6 +84,8 @@ ospf_rx_hook(sock * sk, int size)
   struct ospf_packet *ps;
   struct ospf_iface *ifa = (struct ospf_iface *) (sk->data);
   struct proto *p = (struct proto *) (ifa->proto);
+  struct ospf_neighbor *n;
+  char *mesg = "Bad OSPF packet from ";
 
   if (ifa->stub)
     return (1);
@@ -93,65 +95,71 @@ ospf_rx_hook(sock * sk, int size)
   ps = (struct ospf_packet *) ipv4_skip_header(sk->rbuf, &size);
   if (ps == NULL)
   {
-    log("%s: Bad OSPF packet received: bad IP header", p->name);
-    log("%s: Discarding", p->name);
-    return (1);
+    log(L_ERR "%s%I - bad IP header", mesg, sk->faddr);
+    return 1;
   }
 
   if ((unsigned) size < sizeof(struct ospf_packet))
   {
-    log("%s: Bad OSPF packet received: too short (%u bytes)", p->name, size);
-    log("%s: Discarding", p->name);
-    return (1);
+    log(L_ERR "%s%I - too short (%u bytes)", mesg, sk->faddr, size);
+    return 1;
   }
 
   if ((ntohs(ps->length) != size) || (size != (4 * (size / 4))))
   {
-    log("%s: Bad OSPF packet received: size field does not match", p->name);
-    log("%s: Discarding", p->name);
-    return (1);
+    log(L_ERR "%s%I - size field does not match", mesg, sk->faddr);
+    return 1;
   }
 
   if (ps->version != OSPF_VERSION)
   {
-    log("%s: Bad OSPF packet received: version %u", p->name, ps->version);
-    log("%s: Discarding", p->name);
-    return (1);
+    log(L_ERR "%s%I - version %u", mesg, sk->faddr, ps->version);
+    return 1;
   }
 
   if (!ipsum_verify(ps, 16, (void *) ps + sizeof(struct ospf_packet),
                    ntohs(ps->length) - sizeof(struct ospf_packet), NULL))
   {
-    log("%s: Bad OSPF packet received: bad checksum", p->name);
-    log("%s: Discarding", p->name);
-    return (1);
+    log(L_ERR "%s%I - bad checksum", mesg, sk->faddr);
+    return 1;
   }
 
   if (!ospf_rx_authenticate(ifa, ps))
   {
-    log("%s: Bad OSPF packet received: bad password", p->name);
-    return (1);
+    log(L_ERR "%s%I - bad password", mesg, sk->faddr);
+    return 1;
   }
 
   if (ntohl(ps->areaid) != ifa->an)
   {
-    log("%s: Bad OSPF packet received: other area %ld", p->name, ps->areaid);
-    log("%s: Discarding", p->name);
-    return (1);
+    log(L_ERR "%s%I - other area %ld", mesg, sk->faddr, ps->areaid);
+    return 1;
   }
 
   if (ntohl(ps->routerid) == p->cf->global->router_id)
   {
-    log("%s: Bad OSPF packet received: received my own router ID!", p->name);
-    log("%s: Discarding", p->name);
-    return (1);
+    log(L_ERR "%s%I - received my own router ID!", mesg, sk->faddr);
+    return 1;
   }
 
   if (ntohl(ps->routerid) == 0)
   {
-    log("%s: Bad OSPF packet received: Id 0.0.0.0 is not allowed.", p->name);
-    log("%s: Discarding", p->name);
-    return (1);
+    log(L_ERR "%s%I - router id = 0.0.0.0", mesg, sk->faddr);
+    return 1;
+  }
+
+  if ((unsigned) size > ifa->iface->mtu)
+  {
+    log(L_ERR "%s%I - received larger packet than MTU", mesg, sk->faddr);
+    return 1;
+  }
+
+  n = find_neigh(ifa, ntohl(((struct ospf_packet *) ps)->routerid));
+
+  if(!n && (ps->type != HELLO_P))
+  {
+    OSPF_TRACE(D_PACKETS, "Received non-hello packet from uknown neighbor (%I)", sk->faddr);
+    return 1;
   }
 
   /* Dump packet 
@@ -166,30 +174,29 @@ ospf_rx_hook(sock * sk, int size)
   {
   case HELLO_P:
     DBG("%s: Hello received.\n", p->name);
-    ospf_hello_receive((struct ospf_hello_packet *) ps, ifa, size, sk->faddr);
+    ospf_hello_receive((struct ospf_hello_packet *) ps, ifa, n, sk->faddr);
     break;
   case DBDES_P:
     DBG("%s: Database description received.\n", p->name);
-    ospf_dbdes_receive((struct ospf_dbdes_packet *) ps, ifa, size);
+    ospf_dbdes_receive((struct ospf_dbdes_packet *) ps, ifa, n);
     break;
   case LSREQ_P:
     DBG("%s: Link state request received.\n", p->name);
-    ospf_lsreq_receive((struct ospf_lsreq_packet *) ps, ifa, size);
+    ospf_lsreq_receive((struct ospf_lsreq_packet *) ps, ifa, n);
     break;
   case LSUPD_P:
     DBG("%s: Link state update received.\n", p->name);
-    ospf_lsupd_receive((struct ospf_lsupd_packet *) ps, ifa, size);
+    ospf_lsupd_receive((struct ospf_lsupd_packet *) ps, ifa, n);
     break;
   case LSACK_P:
     DBG("%s: Link state ack received.\n", p->name);
-    ospf_lsack_receive((struct ospf_lsack_packet *) ps, ifa, size);
+    ospf_lsack_receive((struct ospf_lsack_packet *) ps, ifa, n);
     break;
   default:
-    log("%s: Bad packet received: wrong type %u", p->name, ps->type);
-    log("%s: Discarding\n", p->name);
-    return (1);
+    log(L_ERR "%s%I - wrong type %u", mesg, sk->faddr, ps->type);
+    return 1;
   };
-  return (1);
+  return 1;
 }
 
 void