]> granicus.if.org Git - zfs/commitdiff
Fix 'statement with no effect' warning
authorBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 23 Feb 2011 21:03:19 +0000 (13:03 -0800)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Wed, 23 Feb 2011 21:03:19 +0000 (13:03 -0800)
Because the secpolicy_* macros are all currently defined to (0).
And because the caller of this function does not check the return
code.  The compiler complains that this statement has no effect
which is correct and OK.  To suppress the warning explictly cast
the result to (void).

module/zfs/zfs_vnops.c

index b20e3b2c3a29bce19aaef0461fe7efe581b0947d..821ececde35f59c0359e3024b79b015ec382c7b7 100644 (file)
@@ -2274,7 +2274,7 @@ top:
                                /*
                                 * Remove setuid/setgid for non-privileged users
                                 */
-                               secpolicy_setid_clear(attr, cr);
+                               (void) secpolicy_setid_clear(attr, cr);
                                trim_mask = (mask & (ATTR_UID|ATTR_GID));
                        } else {
                                need_policy =  TRUE;