diff options
author | J. Bruce Fields <bfields@redhat.com> | 2013-10-08 15:53:07 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2013-10-08 15:56:23 -0400 |
commit | 3be34555fa294483202483c4b171a8d335447187 (patch) | |
tree | cdcf8e2787d05c7fe4acfc42400e7242dbc9bc4e /net | |
parent | c66080ad0a9295eaf0246a46949619f92c75faaa (diff) |
svcrpc: fix error-handling on badd gssproxy downcall
For every other problem here we bail out with an error, but here for
some reason we're setting a negative cache entry (with, note, an
undefined expiry).
It seems simplest just to bail out in the same way as we do in other
cases.
Cc: Simo Sorce <simo@redhat.com>
Reported-by: Andi Kleen <andi@firstfloor.org>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/sunrpc/auth_gss/svcauth_gss.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/sunrpc/auth_gss/svcauth_gss.c b/net/sunrpc/auth_gss/svcauth_gss.c index 09fb638bcaa4..008cdade5aae 100644 --- a/net/sunrpc/auth_gss/svcauth_gss.c +++ b/net/sunrpc/auth_gss/svcauth_gss.c | |||
@@ -1167,8 +1167,8 @@ static int gss_proxy_save_rsc(struct cache_detail *cd, | |||
1167 | if (!ud->found_creds) { | 1167 | if (!ud->found_creds) { |
1168 | /* userspace seem buggy, we should always get at least a | 1168 | /* userspace seem buggy, we should always get at least a |
1169 | * mapping to nobody */ | 1169 | * mapping to nobody */ |
1170 | dprintk("RPC: No creds found, marking Negative!\n"); | 1170 | dprintk("RPC: No creds found!\n"); |
1171 | set_bit(CACHE_NEGATIVE, &rsci.h.flags); | 1171 | goto out; |
1172 | } else { | 1172 | } else { |
1173 | 1173 | ||
1174 | /* steal creds */ | 1174 | /* steal creds */ |