On Thu, 23 Aug 2012 10:18:25 -0400 Jeff Layton <[email protected]> wrote:
> 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 <[email protected]> > --- > 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; Committed... -- Jeff Layton <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
