]> git.proxmox.com Git - mirror_spl-debian.git/commitdiff
Fix two minor compiler warnings
authorBrian Behlendorf <behlendorf1@llnl.gov>
Mon, 26 Jul 2010 17:24:26 +0000 (10:24 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Mon, 26 Jul 2010 17:24:26 +0000 (10:24 -0700)
In cmd/splat.c there was a comparison between an __u32 and an int.  To
resolve the issue simply use a __u32 and strtoul() when converting the
provided user string.

In module/spl/spl-vnode.c we should explicitly cast nd->last.name to
a const char * which is what is expected by the prototype.

cmd/splat.c
module/spl/spl-vnode.c

index c0bb7d8d4c23ca698b39772f9c58907936750a62..2fa0c50c06e6c2082357ba496c6922772687d972 100644 (file)
@@ -316,14 +316,15 @@ static test_t *test_find(char *sub_str, char *test_str)
        ListIterator si, ti;
        subsystem_t *sub;
        test_t *test;
-       int sub_num, test_num;
+       __u32 sub_num, test_num;
 
-       /* No error checking here because it may not be a number, it's
+       /*
+        * No error checking here because it may not be a number, it's
         * perfectly OK for it to be a string.  Since we're just using
         * it for comparison purposes this is all very safe.
         */
-       sub_num = strtol(sub_str, NULL, 0);
-       test_num = strtol(test_str, NULL, 0);
+       sub_num = strtoul(sub_str, NULL, 0);
+       test_num = strtoul(test_str, NULL, 0);
 
         si = list_iterator_create(subsystems);
 
index 9bfead8cf44a0c431d1106ebdd341981e9e0882c..dd759bf7fb81fd384805262a322ab3dfc9d4fce0 100644 (file)
@@ -257,7 +257,8 @@ EXPORT_SYMBOL(vn_seek);
 static struct dentry *
 vn_lookup_hash(struct nameidata *nd)
 {
-       return lookup_one_len(nd->last.name, nd->nd_dentry, nd->last.len);
+       return lookup_one_len((const char *)nd->last.name,
+                             nd->nd_dentry, nd->last.len);
 } /* lookup_hash() */
 
 static void