mirror of
https://github.com/samba-team/samba.git
synced 2025-01-10 01:18:15 +03:00
auth/spnego: do basic state_position checking in gensec_spnego_update_in()
Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Andreas Schneider <asn@samba.org> Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org> Autobuild-Date(master): Thu Jun 29 20:15:05 CEST 2017 on sn-devel-144
This commit is contained in:
parent
e9f1daa6f4
commit
6cddaa577b
@ -1383,14 +1383,9 @@ static struct tevent_req *gensec_spnego_update_send(TALLOC_CTX *mem_ctx,
|
|||||||
&spnego_state->out_frag);
|
&spnego_state->out_frag);
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case SPNEGO_DONE:
|
|
||||||
/* We should not be called after we are 'done' */
|
|
||||||
tevent_req_nterror(req, NT_STATUS_INVALID_PARAMETER);
|
|
||||||
return tevent_req_post(req, ev);
|
|
||||||
|
|
||||||
default:
|
default:
|
||||||
tevent_req_nterror(req, NT_STATUS_INVALID_PARAMETER);
|
smb_panic(__location__);
|
||||||
return tevent_req_post(req, ev);
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (NT_STATUS_IS_OK(status)) {
|
if (NT_STATUS_IS_OK(status)) {
|
||||||
@ -1434,6 +1429,23 @@ static NTSTATUS gensec_spnego_update_in(struct gensec_security *gensec_security,
|
|||||||
|
|
||||||
*full_in = data_blob_null;
|
*full_in = data_blob_null;
|
||||||
|
|
||||||
|
switch (spnego_state->state_position) {
|
||||||
|
case SPNEGO_FALLBACK:
|
||||||
|
*full_in = in;
|
||||||
|
spnego_state->in_needed = 0;
|
||||||
|
return NT_STATUS_OK;
|
||||||
|
|
||||||
|
case SPNEGO_CLIENT_START:
|
||||||
|
case SPNEGO_CLIENT_TARG:
|
||||||
|
case SPNEGO_SERVER_START:
|
||||||
|
case SPNEGO_SERVER_TARG:
|
||||||
|
break;
|
||||||
|
|
||||||
|
case SPNEGO_DONE:
|
||||||
|
default:
|
||||||
|
return NT_STATUS_INVALID_PARAMETER;
|
||||||
|
}
|
||||||
|
|
||||||
if (spnego_state->in_needed == 0) {
|
if (spnego_state->in_needed == 0) {
|
||||||
size_t size = 0;
|
size_t size = 0;
|
||||||
int ret;
|
int ret;
|
||||||
|
Loading…
Reference in New Issue
Block a user