diff options
| author | Martin Mares <mj@ucw.cz> | 1999-12-16 13:51:43 +0000 |
|---|---|---|
| committer | Martin Mares <mj@ucw.cz> | 1999-12-16 13:51:43 +0000 |
| commit | 476e10842503b51331a7994b6e25c91b20eb8e71 (patch) | |
| tree | e9858a156a032f27a213c8932b5fc5021b2343d3 /sysdep/linux/netlink | |
| parent | e693ddff874890a9e5c990f6ca75d2e2358d065a (diff) | |
Minor cleanups.
Diffstat (limited to 'sysdep/linux/netlink')
| -rw-r--r-- | sysdep/linux/netlink/netlink.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/sysdep/linux/netlink/netlink.c b/sysdep/linux/netlink/netlink.c index 25d8fc03..11973e93 100644 --- a/sysdep/linux/netlink/netlink.c +++ b/sysdep/linux/netlink/netlink.c @@ -30,7 +30,7 @@ #include <linux/netlink.h> #include <linux/rtnetlink.h> -#ifndef MSG_TRUNC /* FIXME: Hack to circumvent omissions in glibc includes */ +#ifndef MSG_TRUNC /* Hack: Several versions of glibc miss this one :( */ #define MSG_TRUNC 0x20 #endif @@ -143,7 +143,7 @@ nl_error(struct nlmsghdr *h) e = (struct nlmsgerr *) NLMSG_DATA(h); ec = -e->error; if (ec) - log(L_WARN "Netlink: %s", strerror(ec)); /* FIXME: Shut up? */ + log(L_WARN "Netlink: %s", strerror(ec)); return ec; } @@ -472,7 +472,6 @@ nl_send_route(struct krt_proto *p, rte *e, int new) r.h.nlmsg_type = new ? RTM_NEWROUTE : RTM_DELROUTE; r.h.nlmsg_len = NLMSG_LENGTH(sizeof(struct rtmsg)); r.h.nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK | (new ? NLM_F_CREATE|NLM_F_REPLACE : 0); - /* FIXME: Do we really need to process ACKs? */ r.r.rtm_family = BIRD_AF; r.r.rtm_dst_len = net->n.pxlen; @@ -666,7 +665,7 @@ nl_parse_route(struct nlmsghdr *h, int scan) if (ng) ra.iface = ng->iface; else - /* FIXME: Remove this warning? */ + /* FIXME: Remove this warning? Handle it somehow... */ log(L_WARN "Kernel told us to use non-neighbor %I for %I/%d", ra.gw, net->n.prefix, net->n.pxlen); } else |