Use NULL instead of 0 where a pointer is expected
authorBart Van Assche <bart.vanassche@sandisk.com>
Fri, 21 Oct 2016 18:45:54 +0000 (11:45 -0700)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Thu, 3 Nov 2016 13:37:26 +0000 (14:37 +0100)
This was detected by sparse.

Signed-off-by: Bart Van Assche <bart.vanassche@sandisk.com>
kpartx/lopart.c
libmultipath/checkers/rbd.c
mpathpersist/main.h

index 023d5ae..14af34f 100644 (file)
@@ -228,7 +228,7 @@ find_unused_loop_device (void)
                                "       maybe /dev/loop# has the wrong major number?");
        } else
                fprintf(stderr, "mount: could not find any free loop device");
-       return 0;
+       return NULL;
 }
 
 extern int
index 745e240..481d860 100644 (file)
@@ -464,7 +464,7 @@ static int rbd_rm_blacklist(struct rbd_checker_context *ct)
        cmd[1] = NULL;
 
        ret = rados_mon_command(ct->cluster, (const char **)cmd, 1, "", 0,
-                               NULL, 0, &stat, &stat_len);
+                               NULL, NULL, &stat, &stat_len);
        if (ret < 0) {
                condlog(1, "rbd%d: repair failed to remove blacklist for %s %d",
                        ct->rbd_bus_id, ct->client_addr, ret);
index 7c31262..5c0e089 100644 (file)
@@ -1,28 +1,28 @@
 static struct option long_options[] = {
-       {"verbose", 1, 0, 'v'},
-       {"clear", 0, 0, 'C'},
-       {"device", 1, 0, 'd'},
-       {"help", 0, 0, 'h'},
-       {"hex", 0, 0, 'H'},
-       {"in", 0, 0, 'i'},
-       {"out", 0, 0, 'o'},
-       {"param-aptpl", 0, 0, 'Z'},
-       {"param-rk", 1, 0, 'K'},
-       {"param-sark", 1, 0, 'S'},
-       {"preempt", 0, 0, 'P'},
-       {"preempt-abort", 0, 0, 'A'},
-       {"prout-type", 1, 0, 'T'},
-       {"read-full-status", 0, 0, 's'},
-       {"read-keys", 0, 0, 'k'},
-       {"read-reservation", 0, 0, 'r'},
-       {"register", 0, 0, 'G'},
-       {"register-ignore", 0, 0, 'I'},
-       {"release", 0, 0, 'L'},
-       {"report-capabilities", 0, 0, 'c'},
-       {"reserve", 0, 0, 'R'},
-       {"transport-id", 1, 0, 'X'},
-       {"alloc-length", 1, 0, 'l'},
-       {0, 0, 0, 0}
+       {"verbose", 1, NULL, 'v'},
+       {"clear", 0, NULL, 'C'},
+       {"device", 1, NULL, 'd'},
+       {"help", 0, NULL, 'h'},
+       {"hex", 0, NULL, 'H'},
+       {"in", 0, NULL, 'i'},
+       {"out", 0, NULL, 'o'},
+       {"param-aptpl", 0, NULL, 'Z'},
+       {"param-rk", 1, NULL, 'K'},
+       {"param-sark", 1, NULL, 'S'},
+       {"preempt", 0, NULL, 'P'},
+       {"preempt-abort", 0, NULL, 'A'},
+       {"prout-type", 1, NULL, 'T'},
+       {"read-full-status", 0, NULL, 's'},
+       {"read-keys", 0, NULL, 'k'},
+       {"read-reservation", 0, NULL, 'r'},
+       {"register", 0, NULL, 'G'},
+       {"register-ignore", 0, NULL, 'I'},
+       {"release", 0, NULL, 'L'},
+       {"report-capabilities", 0, NULL, 'c'},
+       {"reserve", 0, NULL, 'R'},
+       {"transport-id", 1, NULL, 'X'},
+       {"alloc-length", 1, NULL, 'l'},
+       {NULL, 0, NULL, 0}
 };
 
 static void usage(void);