In this case we explicitly don't care what these functions return, so declare a couple of unused variables to catch the results. Signed-off-by: Jeff Layton <jlayton@xxxxxxxxx> --- mount.cifs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mount.cifs.c b/mount.cifs.c index f843bb4..3fd472a 100644 --- a/mount.cifs.c +++ b/mount.cifs.c @@ -1964,8 +1964,8 @@ restore_privs: if (dacrc) rc = rc ? rc : dacrc; } else { - setfsuid(oldfsuid); - setfsgid(oldfsgid); + uid_t __attribute__((unused)) uignore = setfsuid(oldfsuid); + gid_t __attribute__((unused)) gignore = setfsgid(oldfsgid); } return rc; -- 1.7.11.4 -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html