Commit 3be34555 authored by J. Bruce Fields's avatar J. Bruce Fields

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: default avatarAndi Kleen <andi@firstfloor.org>
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent c66080ad
...@@ -1167,8 +1167,8 @@ static int gss_proxy_save_rsc(struct cache_detail *cd, ...@@ -1167,8 +1167,8 @@ static int gss_proxy_save_rsc(struct cache_detail *cd,
if (!ud->found_creds) { if (!ud->found_creds) {
/* userspace seem buggy, we should always get at least a /* userspace seem buggy, we should always get at least a
* mapping to nobody */ * mapping to nobody */
dprintk("RPC: No creds found, marking Negative!\n"); dprintk("RPC: No creds found!\n");
set_bit(CACHE_NEGATIVE, &rsci.h.flags); goto out;
} else { } else {
/* steal creds */ /* steal creds */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment