diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-04-18 10:49:09 +0800 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2013-04-30 18:14:15 -0400 |
| commit | 1eb6d6223aaac87538a0e9aa0d30980df224914c (patch) | |
| tree | d0fd804d6c6ba727de43f35502058bebf6f324ef /net/sunrpc/auth_gss | |
| parent | 2a6cf944c2f8ad5a7ef599ed275b85fa56eba3fc (diff) | |
svcauth_gss: fix error return code in rsc_parse()
Fix to return a negative error code from the error handling
case instead of 0, as returned elsewhere in this function.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net/sunrpc/auth_gss')
| -rw-r--r-- | net/sunrpc/auth_gss/svcauth_gss.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/sunrpc/auth_gss/svcauth_gss.c b/net/sunrpc/auth_gss/svcauth_gss.c index b70ac1cec8f..b1924e53e8c 100644 --- a/net/sunrpc/auth_gss/svcauth_gss.c +++ b/net/sunrpc/auth_gss/svcauth_gss.c @@ -506,8 +506,10 @@ static int rsc_parse(struct cache_detail *cd, len = qword_get(&mesg, buf, mlen); if (len > 0) { rsci.cred.cr_principal = kstrdup(buf, GFP_KERNEL); - if (!rsci.cred.cr_principal) + if (!rsci.cred.cr_principal) { + status = -ENOMEM; goto out; + } } } |