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 b20e3b2..821ecec 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;