diff options
| author | Lars Mueller <lmuelle@suse.com> | 2012-04-20 07:59:15 -0400 |
|---|---|---|
| committer | Jeff Layton <jlayton@samba.org> | 2012-04-20 07:59:15 -0400 |
| commit | 0aa12de5c1565d56a240d7b0dd814316f4ea81f3 (patch) | |
| tree | ee902433e955f49a5350e7d2e00821fd9b6b8fe2 /mount.cifs.c | |
| parent | e5f124c10fa8e582c5df61017d6f6c2b10c397dc (diff) | |
| download | cifs-utils-0aa12de5c1565d56a240d7b0dd814316f4ea81f3.tar.gz cifs-utils-0aa12de5c1565d56a240d7b0dd814316f4ea81f3.tar.bz2 cifs-utils-0aa12de5c1565d56a240d7b0dd814316f4ea81f3.zip | |
mount.cifs: toggle_dac_capability() stores return code
the build process of the cifs-utils for Mandriva 2011 made me notice of
the unused variable rc in toggle_dac_capability() of mount.cifs.c.
A bit up in the code we store the return value and do not make use of it
while calling return.
The attached patch intends to fix this.
The failing build result is still visible at
https://build.opensuse.org/package/live_build_log?arch=x86_64&package=cifs-utils&project=network%3Asamba%3ASTABLE&repository=Mandriva_2011
Acked-by: Suresh Jayaraman <sjayaraman@suse.com>
Signed-off-by: Lars Mueller <lmuelle@suse.com>
Diffstat (limited to 'mount.cifs.c')
| -rw-r--r-- | mount.cifs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mount.cifs.c b/mount.cifs.c index a363b61..06715dd 100644 --- a/mount.cifs.c +++ b/mount.cifs.c @@ -577,7 +577,7 @@ toggle_dac_capability(int writable, int enable) } free_caps: cap_free(caps); - return 0; + return rc; } #else /* HAVE_LIBCAP */ static int |