diff options
| author | Russ Allbery <rra@stanford.edu> | 2013-03-15 11:32:06 -0700 |
|---|---|---|
| committer | Russ Allbery <rra@stanford.edu> | 2013-03-15 11:57:37 -0700 |
| commit | 0fb2fa09c3b5bd1f25c8400d0fc519fb0fca2aab (patch) | |
| tree | 2f78cf2ead0b7813f17927bba9ede8b2a0726849 /client | |
| parent | d5853241c9562fa51cdb990028642bf87a46fbea (diff) | |
Avoid passing uninitialized GSS-API statuses to error reporting
Caught by clang --analyze in the client open code. This shouldn't
matter since the error code wouldn't be TOKEN_FAIL_GSSAPI, but be
clean anyway.
Change-Id: I4c8c82f3d53603c06720b2e1d00ff8a559529879
Reviewed-on: https://gerrit.stanford.edu/949
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
Diffstat (limited to 'client')
| -rw-r--r-- | client/open.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/client/open.c b/client/open.c index 3d9bc67..65029ff 100644 --- a/client/open.c +++ b/client/open.c @@ -8,7 +8,7 @@ * * Written by Russ Allbery <rra@stanford.edu> * Based on work by Anton Ushakov - * Copyright 2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 + * Copyright 2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 * The Board of Trustees of the Leland Stanford Junior University * * See LICENSE for licensing terms. @@ -193,7 +193,7 @@ internal_open(struct remctl *r, const char *host, const char *principal) flags |= TOKEN_PROTOCOL; status = token_send(r->fd, flags, &send_tok, r->timeout); if (status != TOKEN_OK) { - internal_token_error(r, "sending token", status, major, minor); + internal_token_error(r, "sending token", status, 0, 0); gss_release_buffer(&minor, &send_tok); goto fail; } |