diff options
| author | Jan Moskyto Matejka <mq@ucw.cz> | 2016-10-14 15:37:04 +0200 |
|---|---|---|
| committer | Ondrej Zajicek (work) <santiago@crfreenet.org> | 2016-11-01 14:52:54 +0100 |
| commit | 3e236955c9369475872b9b86a58502fa777b50b9 (patch) | |
| tree | d43b519c2289a353e56c32b42c402f9fa12d0e13 /proto/ospf/hello.c | |
| parent | 17fe57d8dcc89aea520788914b252cf49cf060ff (diff) | |
Build: switch on -Wextra, get rid of most of the warnings
There are several unresolved -Wmissing-field-initializers on older
versions of GCC than 5.1, all of them false positive.
Diffstat (limited to 'proto/ospf/hello.c')
| -rw-r--r-- | proto/ospf/hello.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/proto/ospf/hello.c b/proto/ospf/hello.c index 50cf1407..e00487dc 100644 --- a/proto/ospf/hello.c +++ b/proto/ospf/hello.c @@ -222,9 +222,12 @@ ospf_receive_hello(struct ospf_packet *pkt, struct ospf_iface *ifa, rcv_priority = ps->priority; int pxlen = u32_masklen(ntohl(ps->netmask)); + if (pxlen < 0) + DROP("prefix garbled", ntohl(ps->netmask)); + if ((ifa->type != OSPF_IT_VLINK) && (ifa->type != OSPF_IT_PTP) && - (pxlen != ifa->addr->pxlen)) + ((uint) pxlen != ifa->addr->pxlen)) DROP("prefix length mismatch", pxlen); neighbors = ps->neighbors; |