gh-63284: Add support for TLS-PSK (pre-shared key) to the ssl module (#103181)

Add support for TLS-PSK (pre-shared key) to the ssl module.

---------

Co-authored-by: Oleg Iarygin <oleg@arhadthedev.net>
Co-authored-by: Gregory P. Smith <greg@krypto.org>
This commit is contained in:
Grant Ramsay 2023-11-27 17:01:44 +13:00 • committed by GitHub
parent fb202af447
commit e954ac7205
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
10 changed files with 561 additions and 1 deletions

View file

@ -301,6 +301,8 @@ typedef struct {
BIO *keylog_bio;
/* Cached module state, also used in SSLSocket and SSLSession code. */
_sslmodulestate *state;
PyObject *psk_client_callback;
PyObject *psk_server_callback;
} PySSLContext;
typedef struct {
@ -3123,6 +3125,8 @@ _ssl__SSLContext_impl(PyTypeObject *type, int proto_version)
self->alpn_protocols = NULL;
self->set_sni_cb = NULL;
self->state = get_ssl_state(module);
self->psk_client_callback = NULL;
self->psk_server_callback = NULL;
/* Don't check host name by default */
if (proto_version == PY_SSL_VERSION_TLS_CLIENT) {
@ -3235,6 +3239,8 @@ context_clear(PySSLContext *self)
Py_CLEAR(self->set_sni_cb);
Py_CLEAR(self->msg_cb);
Py_CLEAR(self->keylog_filename);
Py_CLEAR(self->psk_client_callback);
Py_CLEAR(self->psk_server_callback);
if (self->keylog_bio != NULL) {
PySSL_BEGIN_ALLOW_THREADS
BIO_free_all(self->keylog_bio);
@ -4662,6 +4668,222 @@ _ssl__SSLContext_get_ca_certs_impl(PySSLContext *self, int binary_form)
return NULL;
}
static unsigned int psk_client_callback(SSL *s,
const char *hint,
char *identity,
unsigned int max_identity_len,
unsigned char *psk,
unsigned int max_psk_len)
{
PyGILState_STATE gstate = PyGILState_Ensure();
PyObject *callback = NULL;
PySSLSocket *ssl = SSL_get_app_data(s);
if (ssl == NULL || ssl->ctx == NULL) {
goto error;
}
callback = ssl->ctx->psk_client_callback;
if (callback == NULL) {
goto error;
}
PyObject *hint_str = (hint != NULL && hint[0] != '\0') ?
PyUnicode_DecodeUTF8(hint, strlen(hint), "strict") :
Py_NewRef(Py_None);
if (hint_str == NULL) {
/* The remote side has sent an invalid UTF-8 string
* (breaking the standard), drop the connection without
* raising a decode exception. */
PyErr_Clear();
goto error;
}
PyObject *result = PyObject_CallFunctionObjArgs(callback, hint_str, NULL);
Py_DECREF(hint_str);
if (result == NULL) {
goto error;
}
const char *psk_;
const char *identity_;
Py_ssize_t psk_len_;
Py_ssize_t identity_len_ = 0;
if (!PyArg_ParseTuple(result, "z#y#", &identity_, &identity_len_, &psk_, &psk_len_)) {
Py_DECREF(result);
goto error;
}
if (identity_len_ + 1 > max_identity_len || psk_len_ > max_psk_len) {
Py_DECREF(result);
goto error;
}
memcpy(psk, psk_, psk_len_);
if (identity_ != NULL) {
memcpy(identity, identity_, identity_len_);
}
identity[identity_len_] = 0;
Py_DECREF(result);
PyGILState_Release(gstate);
return (unsigned int)psk_len_;
error:
if (PyErr_Occurred()) {
PyErr_WriteUnraisable(callback);
}
PyGILState_Release(gstate);
return 0;
}
/*[clinic input]
_ssl._SSLContext.set_psk_client_callback
callback: object
[clinic start generated code]*/
static PyObject *
_ssl__SSLContext_set_psk_client_callback_impl(PySSLContext *self,
PyObject *callback)
/*[clinic end generated code: output=0aba86f6ed75119e input=7627bae0e5ee7635]*/
{
if (self->protocol == PY_SSL_VERSION_TLS_SERVER) {
_setSSLError(get_state_ctx(self),
"Cannot add PSK client callback to a "
"PROTOCOL_TLS_SERVER context", 0, __FILE__, __LINE__);
return NULL;
}
SSL_psk_client_cb_func ssl_callback;
if (callback == Py_None) {
callback = NULL;
// Delete the existing callback
ssl_callback = NULL;
} else {
if (!PyCallable_Check(callback)) {
PyErr_SetString(PyExc_TypeError, "callback must be callable");
return NULL;
}
ssl_callback = psk_client_callback;
}
Py_XDECREF(self->psk_client_callback);
Py_XINCREF(callback);
self->psk_client_callback = callback;
SSL_CTX_set_psk_client_callback(self->ctx, ssl_callback);
Py_RETURN_NONE;
}
static unsigned int psk_server_callback(SSL *s,
const char *identity,
unsigned char *psk,
unsigned int max_psk_len)
{
PyGILState_STATE gstate = PyGILState_Ensure();
PyObject *callback = NULL;
PySSLSocket *ssl = SSL_get_app_data(s);
if (ssl == NULL || ssl->ctx == NULL) {
goto error;
}
callback = ssl->ctx->psk_server_callback;
if (callback == NULL) {
goto error;
}
PyObject *identity_str = (identity != NULL && identity[0] != '\0') ?
PyUnicode_DecodeUTF8(identity, strlen(identity), "strict") :
Py_NewRef(Py_None);
if (identity_str == NULL) {
/* The remote side has sent an invalid UTF-8 string
* (breaking the standard), drop the connection without
* raising a decode exception. */
PyErr_Clear();
goto error;
}
PyObject *result = PyObject_CallFunctionObjArgs(callback, identity_str, NULL);
Py_DECREF(identity_str);
if (result == NULL) {
goto error;
}
char *psk_;
Py_ssize_t psk_len_;
if (PyBytes_AsStringAndSize(result, &psk_, &psk_len_) < 0) {
Py_DECREF(result);
goto error;
}
if (psk_len_ > max_psk_len) {
Py_DECREF(result);
goto error;
}
memcpy(psk, psk_, psk_len_);
Py_DECREF(result);
PyGILState_Release(gstate);
return (unsigned int)psk_len_;
error:
if (PyErr_Occurred()) {
PyErr_WriteUnraisable(callback);
}
PyGILState_Release(gstate);
return 0;
}
/*[clinic input]
_ssl._SSLContext.set_psk_server_callback
callback: object
identity_hint: str(accept={str, NoneType}) = None
[clinic start generated code]*/
static PyObject *
_ssl__SSLContext_set_psk_server_callback_impl(PySSLContext *self,
PyObject *callback,
const char *identity_hint)
/*[clinic end generated code: output=1f4d6a4e09a92b03 input=65d4b6022aa85ea3]*/
{
if (self->protocol == PY_SSL_VERSION_TLS_CLIENT) {
_setSSLError(get_state_ctx(self),
"Cannot add PSK server callback to a "
"PROTOCOL_TLS_CLIENT context", 0, __FILE__, __LINE__);
return NULL;
}
SSL_psk_server_cb_func ssl_callback;
if (callback == Py_None) {
callback = NULL;
// Delete the existing callback and hint
ssl_callback = NULL;
identity_hint = NULL;
} else {
if (!PyCallable_Check(callback)) {
PyErr_SetString(PyExc_TypeError, "callback must be callable");
return NULL;
}
ssl_callback = psk_server_callback;
}
if (SSL_CTX_use_psk_identity_hint(self->ctx, identity_hint) != 1) {
PyErr_SetString(PyExc_ValueError, "failed to set identity hint");
return NULL;
}
Py_XDECREF(self->psk_server_callback);
Py_XINCREF(callback);
self->psk_server_callback = callback;
SSL_CTX_set_psk_server_callback(self->ctx, ssl_callback);
Py_RETURN_NONE;
}
static PyGetSetDef context_getsetlist[] = {
{"check_hostname", (getter) get_check_hostname,
@ -4716,6 +4938,8 @@ static struct PyMethodDef context_methods[] = {
_SSL__SSLCONTEXT_CERT_STORE_STATS_METHODDEF
_SSL__SSLCONTEXT_GET_CA_CERTS_METHODDEF
_SSL__SSLCONTEXT_GET_CIPHERS_METHODDEF
_SSL__SSLCONTEXT_SET_PSK_CLIENT_CALLBACK_METHODDEF
_SSL__SSLCONTEXT_SET_PSK_SERVER_CALLBACK_METHODDEF
{NULL, NULL} /* sentinel */
};

137
Modules/clinic/_ssl.c.h generated
View file

@ -1014,6 +1014,141 @@ exit:
return return_value;
}
PyDoc_STRVAR(_ssl__SSLContext_set_psk_client_callback__doc__,
"set_psk_client_callback($self, /, callback)\n"
"--\n"
"\n");
#define _SSL__SSLCONTEXT_SET_PSK_CLIENT_CALLBACK_METHODDEF \
{"set_psk_client_callback", _PyCFunction_CAST(_ssl__SSLContext_set_psk_client_callback), METH_FASTCALL|METH_KEYWORDS, _ssl__SSLContext_set_psk_client_callback__doc__},
static PyObject *
_ssl__SSLContext_set_psk_client_callback_impl(PySSLContext *self,
PyObject *callback);
static PyObject *
_ssl__SSLContext_set_psk_client_callback(PySSLContext *self, PyObject *const *args, Py_ssize_t nargs, PyObject *kwnames)
{
PyObject *return_value = NULL;
#if defined(Py_BUILD_CORE) && !defined(Py_BUILD_CORE_MODULE)
#define NUM_KEYWORDS 1
static struct {
PyGC_Head _this_is_not_used;
PyObject_VAR_HEAD
PyObject *ob_item[NUM_KEYWORDS];
} _kwtuple = {
.ob_base = PyVarObject_HEAD_INIT(&PyTuple_Type, NUM_KEYWORDS)
.ob_item = { &_Py_ID(callback), },
};
#undef NUM_KEYWORDS
#define KWTUPLE (&_kwtuple.ob_base.ob_base)
#else // !Py_BUILD_CORE
# define KWTUPLE NULL
#endif // !Py_BUILD_CORE
static const char * const _keywords[] = {"callback", NULL};
static _PyArg_Parser _parser = {
.keywords = _keywords,
.fname = "set_psk_client_callback",
.kwtuple = KWTUPLE,
};
#undef KWTUPLE
PyObject *argsbuf[1];
PyObject *callback;
args = _PyArg_UnpackKeywords(args, nargs, NULL, kwnames, &_parser, 1, 1, 0, argsbuf);
if (!args) {
goto exit;
}
callback = args[0];
return_value = _ssl__SSLContext_set_psk_client_callback_impl(self, callback);
exit:
return return_value;
}
PyDoc_STRVAR(_ssl__SSLContext_set_psk_server_callback__doc__,
"set_psk_server_callback($self, /, callback, identity_hint=None)\n"
"--\n"
"\n");
#define _SSL__SSLCONTEXT_SET_PSK_SERVER_CALLBACK_METHODDEF \
{"set_psk_server_callback", _PyCFunction_CAST(_ssl__SSLContext_set_psk_server_callback), METH_FASTCALL|METH_KEYWORDS, _ssl__SSLContext_set_psk_server_callback__doc__},
static PyObject *
_ssl__SSLContext_set_psk_server_callback_impl(PySSLContext *self,
PyObject *callback,
const char *identity_hint);
static PyObject *
_ssl__SSLContext_set_psk_server_callback(PySSLContext *self, PyObject *const *args, Py_ssize_t nargs, PyObject *kwnames)
{
PyObject *return_value = NULL;
#if defined(Py_BUILD_CORE) && !defined(Py_BUILD_CORE_MODULE)
#define NUM_KEYWORDS 2
static struct {
PyGC_Head _this_is_not_used;
PyObject_VAR_HEAD
PyObject *ob_item[NUM_KEYWORDS];
} _kwtuple = {
.ob_base = PyVarObject_HEAD_INIT(&PyTuple_Type, NUM_KEYWORDS)
.ob_item = { &_Py_ID(callback), &_Py_ID(identity_hint), },
};
#undef NUM_KEYWORDS
#define KWTUPLE (&_kwtuple.ob_base.ob_base)
#else // !Py_BUILD_CORE
# define KWTUPLE NULL
#endif // !Py_BUILD_CORE
static const char * const _keywords[] = {"callback", "identity_hint", NULL};
static _PyArg_Parser _parser = {
.keywords = _keywords,
.fname = "set_psk_server_callback",
.kwtuple = KWTUPLE,
};
#undef KWTUPLE
PyObject *argsbuf[2];
Py_ssize_t noptargs = nargs + (kwnames ? PyTuple_GET_SIZE(kwnames) : 0) - 1;
PyObject *callback;
const char *identity_hint = NULL;
args = _PyArg_UnpackKeywords(args, nargs, NULL, kwnames, &_parser, 1, 2, 0, argsbuf);
if (!args) {
goto exit;
}
callback = args[0];
if (!noptargs) {
goto skip_optional_pos;
}
if (args[1] == Py_None) {
identity_hint = NULL;
}
else if (PyUnicode_Check(args[1])) {
Py_ssize_t identity_hint_length;
identity_hint = PyUnicode_AsUTF8AndSize(args[1], &identity_hint_length);
if (identity_hint == NULL) {
goto exit;
}
if (strlen(identity_hint) != (size_t)identity_hint_length) {
PyErr_SetString(PyExc_ValueError, "embedded null character");
goto exit;
}
}
else {
_PyArg_BadArgument("set_psk_server_callback", "argument 'identity_hint'", "str or None", args[1]);
goto exit;
}
skip_optional_pos:
return_value = _ssl__SSLContext_set_psk_server_callback_impl(self, callback, identity_hint);
exit:
return return_value;
}
static PyObject *
_ssl_MemoryBIO_impl(PyTypeObject *type);
@ -1527,4 +1662,4 @@ exit:
#ifndef _SSL_ENUM_CRLS_METHODDEF
#define _SSL_ENUM_CRLS_METHODDEF
#endif /* !defined(_SSL_ENUM_CRLS_METHODDEF) */
/*[clinic end generated code: output=aa6b0a898b6077fe input=a9049054013a1b77]*/
/*[clinic end generated code: output=6342ea0062ab16c7 input=a9049054013a1b77]*/