diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2015-04-19 14:54:03 +0200 |
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2015-04-21 10:19:01 +0200 |
| commit | 3cf4053a9e832408fad33a8246ecbd189f23a956 (patch) | |
| tree | c55451f64ca56d5df10cbedb66b3aa1e3928ba8a /ospf6d | |
| parent | 72c69d434840598a158747ba9f69dad536f96cea (diff) | |
| download | quagga-3cf4053a9e832408fad33a8246ecbd189f23a956.tar.bz2 quagga-3cf4053a9e832408fad33a8246ecbd189f23a956.tar.xz | |
ospf6d: fix pointer arithmetic warning
caddr_t was signed; this buffer size comparison is better done in
unsigned.
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'ospf6d')
| -rw-r--r-- | ospf6d/ospf6_intra.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ospf6d/ospf6_intra.c b/ospf6d/ospf6_intra.c index 1ef0b535..6606c96d 100644 --- a/ospf6d/ospf6_intra.c +++ b/ospf6d/ospf6_intra.c @@ -234,9 +234,9 @@ ospf6_router_lsa_originate (struct thread *thread) /* Multiple Router-LSA instance according to size limit setting */ if ( (oa->router_lsa_size_limit != 0) - && ((caddr_t) lsdesc + sizeof (struct ospf6_router_lsdesc) - - /* XXX warning: comparison between signed and unsigned */ - (caddr_t) buffer > oa->router_lsa_size_limit)) + && ((size_t)((char *)lsdesc - buffer) + + sizeof (struct ospf6_router_lsdesc) + > oa->router_lsa_size_limit)) { if ((caddr_t) lsdesc == (caddr_t) router_lsa + sizeof (struct ospf6_router_lsa)) |