summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPavel Machek <pavel@ucw.cz>1999-03-17 11:20:23 +0100
committerPavel Machek <pavel@ucw.cz>1999-03-17 11:20:23 +0100
commit29df5739c455a954598c809c2e930abc41c8488e (patch)
tree581eab8699f5a365e5efd5136f12e9e89ffa1290
parentd36d838df5f726e1f6845fe0e6e5c188426ac00d (diff)
downloadbird-29df5739c455a954598c809c2e930abc41c8488e.tar
bird-29df5739c455a954598c809c2e930abc41c8488e.zip
Don't segfault on unknown interface.
-rw-r--r--proto/rip/rip.c33
1 files changed, 21 insertions, 12 deletions
diff --git a/proto/rip/rip.c b/proto/rip/rip.c
index 593f32b..fdbdacd 100644
--- a/proto/rip/rip.c
+++ b/proto/rip/rip.c
@@ -35,7 +35,7 @@ rip_reply(struct proto *p)
#endif
}
-#define P_NAME "rip/unknown" /* FIXME */
+#define P_NAME p->name
/*
* Output processing
@@ -88,8 +88,8 @@ rip_tx( sock *s )
packet->block[i].tag = htons( 0 ); /* FIXME: What should I set it to? */
packet->block[i].network = e->n.prefix;
packet->block[i].netmask = ipa_mkmask( e->n.pxlen );
- packet->block[i].nexthop = IPA_NONE; /* FIXME: How should I set it? */
- packet->block[i].metric = htonl( 1+ (e->metric?:1) );
+ packet->block[i].nexthop = IPA_NONE; /* FIXME: How should I set it? My own IP address? */
+ packet->block[i].metric = htonl( 1+ (e->metric?:1) ); /* FIXME: is this right? I already ++ metric at input side */
if (ipa_equal(e->whotoldme, s->daddr)) {
DBG( "(split horizont)" );
/* FIXME: should we do it in all cases? */
@@ -160,7 +160,7 @@ rip_sendto( struct proto *p, ip_addr daddr, int dport, struct rip_interface *rif
rip_tx(c->rif->sock);
}
-struct rip_interface*
+static struct rip_interface*
find_interface(struct proto *p, struct iface *what)
{
struct rip_interface *i;
@@ -190,7 +190,7 @@ advertise_entry( struct proto *p, struct rip_block *b, ip_addr whotoldme )
rte *r;
net *n;
neighbor *neighbor;
- struct rip_interface *i;
+ struct rip_interface *rif;
bzero(&A, sizeof(A));
A.proto = p;
@@ -202,7 +202,9 @@ advertise_entry( struct proto *p, struct rip_block *b, ip_addr whotoldme )
A.flags = 0;
A.gw = ipa_nonzero(b->nexthop) ? b->nexthop : whotoldme;
A.from = whotoldme;
-
+
+ /* FIXME: Check if destination looks valid - ie not net 0 or 127 */
+
neighbor = neigh_find( p, &A.gw, 0 );
if (!neighbor) {
log( L_ERR "%I asked me to route %I/%I using not-neighbor %I.", A.from, b->network, b->netmask, A.gw );
@@ -210,14 +212,23 @@ advertise_entry( struct proto *p, struct rip_block *b, ip_addr whotoldme )
}
A.iface = neighbor->iface;
- if (!(i = neighbor->data)) {
- i = neighbor->data = find_interface(p, A.iface);
+ if (!(rif = neighbor->data)) {
+ rif = neighbor->data = find_interface(p, A.iface);
}
+ if (!rif) {
+ bug("Route packet using unknown interface? No.");
+ return;
+ }
+
/* set to: interface of nexthop */
a = rta_lookup(&A);
- n = net_get( &master_table, 0, b->network, ipa_mklen( b->netmask )); /* FIXME: should verify that it really is netmask */
+ if (!ipa_equal( ipa_mkmask(ipa_mklen(b->netmask)), b->netmask)) {
+ log( L_ERR "%I asked me to route %I/%I, but that is not valid netmask.", A.from, b->network, b->netmask );
+ return;
+ }
+ n = net_get( &master_table, 0, b->network, ipa_mklen( b->netmask ));
r = rte_get_temp(a);
- r->u.rip.metric = ntohl(b->metric) + i->metric;
+ r->u.rip.metric = ntohl(b->metric) + rif->metric;
if (r->u.rip.metric > P_CF->infinity) r->u.rip.metric = P_CF->infinity;
r->u.rip.tag = ntohl(b->tag);
r->net = n;
@@ -239,8 +250,6 @@ process_block( struct proto *p, struct rip_block *block, ip_addr whotoldme )
return;
}
- /* FIXME: Check if destination looks valid - ie not net 0 or 127 */
-
debug( "block: %I tells me: %I/%I available, metric %d... ", whotoldme, network, block->netmask, metric );
advertise_entry( p, block, whotoldme );