Partially reverts
04f92904ea6c41517ff7154910c16ef4c3bc646b
"userv-ipif: Always request routes from userv, regardless of link quality"
It turns out that this check is necessary to avoid bringing up a route
for a "netlink" stanza in the configuration file which is never used.
In particular, this avoids bringing up a netlink for (a) sites which
are not mentioned in the config file (b) the site on which secnet is
running.
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
c->deliver=NULL;
c->dst=NULL;
c->name=name;
c->deliver=NULL;
c->dst=NULL;
c->name=name;
- c->link_quality=LINK_QUALITY_DOWN;
+ c->link_quality=LINK_QUALITY_UNUSED;
c->mtu=mtu?mtu:st->mtu;
c->options=options;
c->outcount=0;
c->mtu=mtu?mtu:st->mtu;
c->options=options;
c->outcount=0;
typedef void netlink_deliver_fn(void *st, struct buffer_if *buf);
/* site code can tell netlink when outgoing packets will be dropped,
so netlink can generate appropriate ICMP and make routing decisions */
typedef void netlink_deliver_fn(void *st, struct buffer_if *buf);
/* site code can tell netlink when outgoing packets will be dropped,
so netlink can generate appropriate ICMP and make routing decisions */
-#define LINK_QUALITY_DOWN 0 /* No chance of a packet being delivered */
-#define LINK_QUALITY_DOWN_STALE_ADDRESS 1 /* Link down, old address information */
-#define LINK_QUALITY_DOWN_CURRENT_ADDRESS 2 /* Link down, current address information */
-#define LINK_QUALITY_UP 3 /* Link active */
+#define LINK_QUALITY_UNUSED 0 /* This link is unused, do not make this netlink */
+#define LINK_QUALITY_DOWN 1 /* No chance of a packet being delivered right away*/
+#define LINK_QUALITY_DOWN_STALE_ADDRESS 2 /* Link down, old address information */
+#define LINK_QUALITY_DOWN_CURRENT_ADDRESS 3 /* Link down, current address information */
+#define LINK_QUALITY_UP 4 /* Link active */
#define MAXIMUM_LINK_QUALITY 3
typedef void netlink_link_quality_fn(void *st, uint32_t quality);
typedef void netlink_register_fn(void *st, netlink_deliver_fn *deliver,
#define MAXIMUM_LINK_QUALITY 3
typedef void netlink_link_quality_fn(void *st, uint32_t quality);
typedef void netlink_register_fn(void *st, netlink_deliver_fn *deliver,
allnets=ipset_new();
for (r=st->slip.nl.clients; r; r=r->next) {
allnets=ipset_new();
for (r=st->slip.nl.clients; r; r=r->next) {
- struct ipset *nan;
- r->kup=True;
- nan=ipset_union(allnets,r->networks);
- ipset_free(allnets);
- allnets=nan;
+ if (r->link_quality > LINK_QUALITY_UNUSED) {
+ struct ipset *nan;
+ r->kup=True;
+ nan=ipset_union(allnets,r->networks);
+ ipset_free(allnets);
+ allnets=nan;
+ }
}
snets=ipset_to_subnet_list(allnets);
ipset_free(allnets);
}
snets=ipset_to_subnet_list(allnets);
ipset_free(allnets);