diff --git a/proto/rip/rip.c b/proto/rip/rip.c index f69127db..73e3098c 100644 --- a/proto/rip/rip.c +++ b/proto/rip/rip.c @@ -297,7 +297,7 @@ advertise_entry( struct proto *p, struct rip_block *b, ip_addr whotoldme ) } n = net_get( p->table, b->network, pxlen ); r = rte_get_temp(a); - r->u.rip.metric = ntohl(b->metric) + rif->patt->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; @@ -468,7 +468,7 @@ rip_timer(timer *t) struct iface *iface = rif->iface; if (!iface) continue; - if (rif->patt->mode & IM_QUIET) continue; + if (rif->mode & IM_QUIET) continue; if (!(iface->flags & IF_UP)) continue; rif->triggered = (P->tx_count % 6); @@ -577,15 +577,17 @@ static struct rip_interface * new_iface(struct proto *p, struct iface *new, unsigned long flags, struct iface_patt *patt ) { struct rip_interface *rif; + struct rip_patt *PATT = (struct rip_patt *) patt; rif = mb_allocz(p->pool, sizeof( struct rip_interface )); rif->iface = new; rif->proto = p; rif->busy = NULL; - rif->patt = (struct rip_patt *) patt; - - if (rif->patt) - rif->multicast = (!(rif->patt->mode & IM_BROADCAST)) && (flags & IF_MULTICAST); + if (PATT) { + rif->mode = PATT->mode; + rif->metric = PATT->metric; + rif->multicast = (!(PATT->mode & IM_BROADCAST)) && (flags & IF_MULTICAST); + } /* lookup multicasts over unnumbered links - no: rip is not defined over unnumbered links */ if (rif->multicast) @@ -627,7 +629,7 @@ new_iface(struct proto *p, struct iface *new, unsigned long flags, struct iface_ if (!ipa_nonzero(rif->sock->daddr)) { log( L_WARN "%s: interface %s is too strange for me", P_NAME, rif->iface ? rif->iface->name : "(dummy)" ); } else - if (!(rif->patt->mode & IM_NOLISTEN)) + if (!(rif->mode & IM_NOLISTEN)) if (sk_open(rif->sock)<0) { log( L_ERR "%s: could not listen on %s", P_NAME, rif->iface ? rif->iface->name : "(dummy)" ); /* Don't try to transmit into this one? Well, why not? This should not happen, anyway :-) */ @@ -868,8 +870,12 @@ static int rip_reconfigure(struct proto *p, struct proto_config *c) { struct rip_config *new = (struct rip_config *) c; - int generic = sizeof(struct proto_config) + sizeof(list); + int generic = sizeof(struct proto_config) + sizeof(list) /* + sizeof(struct password_item *) */; +#if 0 + if (!password_same()) + return 0; +#endif return !memcmp(((byte *) P_CF) + generic, ((byte *) new) + generic, sizeof(struct rip_proto_config) - generic); diff --git a/proto/rip/rip.h b/proto/rip/rip.h index cbddeaf6..442232db 100644 --- a/proto/rip/rip.h +++ b/proto/rip/rip.h @@ -101,7 +101,8 @@ struct rip_interface { struct iface *iface; sock *sock; struct rip_connection *busy; - struct rip_patt *patt; + int metric; /* You don't want to put struct rip_patt *patt here -- think about reconfigure */ + int mode; int triggered; struct object_lock *lock; int multicast; @@ -121,6 +122,7 @@ struct rip_patt { struct rip_proto_config { struct proto_config c; list iface_list; /* Patterns configured -- keep it first; see rip_reconfigure why */ + struct password_item *passwords; /* Passwords, keep second */ int infinity; /* User configurable data */ int port; @@ -128,7 +130,6 @@ struct rip_proto_config { int garbage_time; int timeout_time; - struct password_item *passwords; int authtype; #define AT_NONE 0 #define AT_PLAINTEXT 2