]> git.proxmox.com Git - mirror_iproute2.git/commitdiff
ipmacsec: fix warning on 32bit platform
authorStephen Hemminger <stephen@networkplumber.org>
Mon, 10 Dec 2018 21:47:58 +0000 (13:47 -0800)
committerStephen Hemminger <stephen@networkplumber.org>
Mon, 10 Dec 2018 21:47:58 +0000 (13:47 -0800)
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>
ip/ipmacsec.c

index 9b991065e4836a52438bff34dad544b492be555c..54cd2b8c97e46f0188d753645ec5e7751cc9d2e5 100644 (file)
@@ -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);
        }
 }