summaryrefslogtreecommitdiff
path: root/source4/libcli
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2004-07-06 23:20:23 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 12:56:53 -0500
commite59e82b90204107a3b6e2e6e3fc52d48634adc7a (patch)
tree77e28ca5e14aecca23c82f4b5402bfba14decc6e /source4/libcli
parent7c139a6815b6c8f877f69eec0b3299416630e101 (diff)
downloadsamba-e59e82b90204107a3b6e2e6e3fc52d48634adc7a.tar.gz
samba-e59e82b90204107a3b6e2e6e3fc52d48634adc7a.tar.bz2
samba-e59e82b90204107a3b6e2e6e3fc52d48634adc7a.zip
r1372: Remove the 'default' case from the SPENGO state machine, and fix up
some compiler warnings that allowed us to see. Andrew Bartlett (This used to be commit 1a6c2018dd49519e6fccdd5a7f35d70b67d45275)
Diffstat (limited to 'source4/libcli')
-rw-r--r--source4/libcli/auth/spnego.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/source4/libcli/auth/spnego.c b/source4/libcli/auth/spnego.c
index b680fe170e..502d30b30f 100644
--- a/source4/libcli/auth/spnego.c
+++ b/source4/libcli/auth/spnego.c
@@ -40,7 +40,7 @@ struct spnego_state {
TALLOC_CTX *mem_ctx;
uint_t ref_count;
enum spnego_message_type expected_packet;
- enum spnego_message_type state_position;
+ enum spnego_state_position state_position;
enum spnego_negResult result;
struct gensec_security *sub_sec_security;
};
@@ -314,8 +314,9 @@ static NTSTATUS gensec_spnego_update(struct gensec_security *gensec_security, TA
} else {
/* server needs to send NegTargetInit */
}
+ return NT_STATUS_INVALID_PARAMETER;
}
-
+
case SPNEGO_CLIENT_START:
{
/* The server offers a list of mechanisms */
@@ -534,11 +535,10 @@ static NTSTATUS gensec_spnego_update(struct gensec_security *gensec_security, TA
return nt_status;
}
- default:
- spnego_free_data(&spnego);
- DEBUG(1, ("Invalid SPENGO request: %d\n", spnego.type));
- return NT_STATUS_INVALID_PARAMETER;
+ case SPNEGO_DONE:
+ return NT_STATUS_OK;
}
+ return NT_STATUS_INVALID_PARAMETER;
}
static void gensec_spnego_end(struct gensec_security *gensec_security)