diff options
| author |   Trond Myklebust <Trond.Myklebust@netapp.com> | 2009-12-18 16:28:12 -0500 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2009-12-18 16:28:12 -0500 |
| commit | b891e4a05ef6beac85465295a032431577c66b16 (patch) | |
| tree | 832ee6823036cdcfc7a2b52d7960faebbaa035cc /net | |
| parent | SUNRPC: Fix up an error return value in gss_import_sec_context_kerberos() (diff) | |
| download | linux-dev-b891e4a05ef6beac85465295a032431577c66b16.tar.xz linux-dev-b891e4a05ef6beac85465295a032431577c66b16.zip | |
SUNRPC: Fix the return value in gss_import_sec_context()
If the context allocation fails, it will return GSS_S_FAILURE, which is
neither a valid error code, nor is it even negative.
Return ENOMEM instead...
Reported-by: Jeff Layton <jlayton@redhat.com>
Cc: stable@kernel.org
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/sunrpc/auth_gss/gss_mech_switch.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/auth_gss/gss_mech_switch.c b/net/sunrpc/auth_gss/gss_mech_switch.c index 6efbb0cd3c7c..76e4c6f4ac3c 100644 --- a/net/sunrpc/auth_gss/gss_mech_switch.c +++ b/net/sunrpc/auth_gss/gss_mech_switch.c @@ -252,7 +252,7 @@ gss_import_sec_context(const void *input_token, size_t bufsize, struct gss_ctx **ctx_id) { if (!(*ctx_id = kzalloc(sizeof(**ctx_id), GFP_KERNEL))) - return GSS_S_FAILURE; + return -ENOMEM; (*ctx_id)->mech_type = gss_mech_get(mech); return mech->gm_ops |