diff options
author | Andi Kleen <andi@firstfloor.org> | 2010-07-20 15:24:27 -0700 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2010-07-29 19:32:17 -0400 |
commit | 69049961014992f50b10d6c3cd3cd172d4aae5ac (patch) | |
tree | d6717d7bc3eae733374cc9acb3a278a82498dced /fs/nfsd/nfsproc.c | |
parent | f9d7562fdb9dc0ada3a7aba5dbbe9d965e2a105d (diff) | |
download | linux-stable-69049961014992f50b10d6c3cd3cd172d4aae5ac.tar.gz linux-stable-69049961014992f50b10d6c3cd3cd172d4aae5ac.tar.bz2 linux-stable-69049961014992f50b10d6c3cd3cd172d4aae5ac.zip |
gcc-4.6: nfsd: fix initialized but not read warnings
Fixes at least one real minor bug: the nfs4 recovery dir sysctl
would not return its status properly.
Also I finished Al's 1e41568d7378d ("Take ima_path_check() in nfsd
past dentry_open() in nfsd_open()") commit, it moved the IMA
code, but left the old path initializer in there.
The rest is just dead code removed I think, although I was not
fully sure about the "is_borc" stuff. Some more review
would be still good.
Found by gcc 4.6's new warnings.
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Neil Brown <neilb@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/nfsproc.c')
-rw-r--r-- | fs/nfsd/nfsproc.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c index a047ad6111ef..1edb78b96c5f 100644 --- a/fs/nfsd/nfsproc.c +++ b/fs/nfsd/nfsproc.c @@ -290,7 +290,6 @@ nfsd_proc_create(struct svc_rqst *rqstp, struct nfsd_createargs *argp, * gospel of sun micro */ if (type != S_IFREG) { - int is_borc = 0; if (type != S_IFBLK && type != S_IFCHR) { rdev = 0; } else if (type == S_IFCHR && !(attr->ia_valid & ATTR_SIZE)) { @@ -298,7 +297,6 @@ nfsd_proc_create(struct svc_rqst *rqstp, struct nfsd_createargs *argp, type = S_IFIFO; } else { /* Okay, char or block special */ - is_borc = 1; if (!rdev) rdev = wanted; } |