diff options
author | Stephen Hemminger <stephen@networkplumber.org> | 2018-12-10 13:47:58 -0800 |
---|---|---|
committer | Stephen Hemminger <stephen@networkplumber.org> | 2018-12-10 13:47:58 -0800 |
commit | 79940533c0e6f2686caf0d600996ed7bedf884a5 (patch) | |
tree | 32f5c57af0056461804c77e57f4cd2426e608f8d /ip/ipmacsec.c | |
parent | 028766aed21a4d8eb2e60c9ef667f75f9354a104 (diff) |
ipmacsec: fix warning on 32bit platform
On some 32 bit platforms, the printf was causing warning:
ipmacsec.c: In function ‘getattr_u64’:
ipmacsec.c:655:47: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘unsigned int’ [-Wformat=]
fprintf(stderr, "invalid attribute length %lu\n",
Resolve by computing length as size_t first.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Diffstat (limited to 'ip/ipmacsec.c')
-rw-r--r-- | ip/ipmacsec.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/ip/ipmacsec.c b/ip/ipmacsec.c index 9b991065..54cd2b8c 100644 --- a/ip/ipmacsec.c +++ b/ip/ipmacsec.c @@ -640,9 +640,11 @@ static void print_attrs(struct rtattr *attrs[]) } } -static __u64 getattr_u64(struct rtattr *stat) +static __u64 getattr_u64(const struct rtattr *stat) { - switch (RTA_PAYLOAD(stat)) { + size_t len = RTA_PAYLOAD(stat); + + switch (len) { case sizeof(__u64): return rta_getattr_u64(stat); case sizeof(__u32): @@ -652,8 +654,8 @@ static __u64 getattr_u64(struct rtattr *stat) case sizeof(__u8): return rta_getattr_u8(stat); default: - fprintf(stderr, "invalid attribute length %lu\n", - RTA_PAYLOAD(stat)); + fprintf(stderr, "invalid attribute length %zu\n", + len); exit(-1); } } |