From b9b170a9dd640dbde0a707b972fdb0c611e68df5 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Thu, 20 Oct 2011 11:53:40 +0200 Subject: s3-seal use gensec_[un]wrap() instead of gensec_[un]seal_packet() This should not make a difference for NTLMSSP as it still calls the low level ntlmssp_[un]seal_packet() functions with the same input parameters. If we convert the gss-api/krb5 based code to gensec we have to use gensec_[un]wrap() as the wire format is different compared to gensec_[un]seal_packet() there. Andrew Bartlett Split from another commit by Stefan Metzmacher --- source3/libsmb/smb_seal.c | 95 ++++++++++++++++++++--------------------------- 1 file changed, 41 insertions(+), 54 deletions(-) diff --git a/source3/libsmb/smb_seal.c b/source3/libsmb/smb_seal.c index f767f16be50..0f415d0a906 100644 --- a/source3/libsmb/smb_seal.c +++ b/source3/libsmb/smb_seal.c @@ -82,42 +82,44 @@ bool common_encryption_on(struct smb_trans_enc_state *es) static NTSTATUS common_ntlm_decrypt_buffer(struct auth_ntlmssp_state *auth_ntlmssp_state, char *buf) { + struct gensec_security *gensec = auth_ntlmssp_state->gensec_security; NTSTATUS status; size_t buf_len = smb_len_nbt(buf) + 4; /* Don't forget the 4 length bytes. */ - size_t data_len; - char *inbuf; - DATA_BLOB sig; + DATA_BLOB in_buf, out_buf; + TALLOC_CTX *frame; - if (buf_len < 8 + NTLMSSP_SIG_SIZE) { + if (buf_len < 8) { return NT_STATUS_BUFFER_TOO_SMALL; } - inbuf = (char *)smb_xmemdup(buf, buf_len); - - /* Adjust for the signature. */ - data_len = buf_len - 8 - NTLMSSP_SIG_SIZE; + frame = talloc_stackframe(); - /* Point at the signature. */ - sig = data_blob_const(inbuf+8, NTLMSSP_SIG_SIZE); + in_buf = data_blob_const(buf + 8, buf_len - 8); - status = gensec_unseal_packet(auth_ntlmssp_state->gensec_security, - (unsigned char *)inbuf + 8 + NTLMSSP_SIG_SIZE, /* 4 byte len + 0xFF 'E' */ - data_len, - (unsigned char *)inbuf + 8 + NTLMSSP_SIG_SIZE, - data_len, - &sig); + status = gensec_unwrap(gensec, frame, &in_buf, &out_buf); if (!NT_STATUS_IS_OK(status)) { - SAFE_FREE(inbuf); + DEBUG(0,("common_ntlm_decrypt_buffer: gensec_unwrap failed. Error %s\n", + nt_errstr(status))); + TALLOC_FREE(frame); return status; } - memcpy(buf + 8, inbuf + 8 + NTLMSSP_SIG_SIZE, data_len); + if (out_buf.length > in_buf.length) { + DEBUG(0,("common_ntlm_decrypt_buffer: gensec_unwrap size (%u) too large (%u) !\n", + (unsigned int)out_buf.length, + (unsigned int)in_buf.length )); + TALLOC_FREE(frame); + return NT_STATUS_INVALID_PARAMETER; + } + + memcpy(buf + 8, out_buf.data, out_buf.length); /* Reset the length and overwrite the header. */ - smb_setlen_nbt(buf,data_len + 4); + smb_setlen_nbt(buf, out_buf.length + 4); + + TALLOC_FREE(frame); - SAFE_FREE(inbuf); return NT_STATUS_OK; } @@ -133,55 +135,40 @@ static NTSTATUS common_ntlm_encrypt_buffer(struct auth_ntlmssp_state *auth_ntlms char *buf, char **ppbuf_out) { + struct gensec_security *gensec = auth_ntlmssp_state->gensec_security; NTSTATUS status; - char *buf_out; - size_t data_len = smb_len_nbt(buf) - 4; /* Ignore the 0xFF SMB bytes. */ - DATA_BLOB sig; + DATA_BLOB in_buf, out_buf; + size_t buf_len = smb_len_nbt(buf) + 4; /* Don't forget the 4 length bytes. */ TALLOC_CTX *frame; + *ppbuf_out = NULL; - if (data_len == 0) { + if (buf_len < 8) { return NT_STATUS_BUFFER_TOO_SMALL; } + in_buf = data_blob_const(buf + 8, buf_len - 8); frame = talloc_stackframe(); - /* - * We know smb_len_nbt can't return a value > 128k, so no int overflow - * check needed. - */ - buf_out = (char *)malloc(8 + NTLMSSP_SIG_SIZE + data_len); - if (buf_out == NULL) { + status = gensec_wrap(gensec, frame, &in_buf, &out_buf); + if (!NT_STATUS_IS_OK(status)) { + DEBUG(0,("common_ntlm_encrypt_buffer: gensec_wrap failed. Error %s\n", + nt_errstr(status))); TALLOC_FREE(frame); - return NT_STATUS_NO_MEMORY; + return status; } - /* Copy the data from the original buffer. */ - - memcpy(buf_out + 8 + NTLMSSP_SIG_SIZE, buf + 8, data_len); - - smb_set_enclen(buf_out, smb_len_nbt(buf) + NTLMSSP_SIG_SIZE, enc_ctx_num); - - ZERO_STRUCT(sig); + *ppbuf_out = (char *)malloc(out_buf.length + 8); /* We know this can't wrap. */ + if (!*ppbuf_out) { + TALLOC_FREE(frame); + return NT_STATUS_NO_MEMORY; + } - status = gensec_seal_packet(auth_ntlmssp_state->gensec_security, - frame, - (unsigned char *)buf_out + 8 + NTLMSSP_SIG_SIZE, /* 4 byte len + 0xFF 'S' */ - data_len, - (unsigned char *)buf_out + 8 + NTLMSSP_SIG_SIZE, - data_len, - &sig); + memcpy(*ppbuf_out+8, out_buf.data, out_buf.length); + smb_set_enclen(*ppbuf_out, out_buf.length + 4, enc_ctx_num); - if (!NT_STATUS_IS_OK(status)) { - talloc_free(frame); - SAFE_FREE(buf_out); - return status; - } + TALLOC_FREE(frame); - /* First 16 data bytes are signature for SSPI compatibility. */ - memcpy(buf_out + 8, sig.data, NTLMSSP_SIG_SIZE); - talloc_free(frame); - *ppbuf_out = buf_out; return NT_STATUS_OK; } -- cgit