From d3fffca5e9ab45b43fa3f460ad6051356c9a00a9 Mon Sep 17 00:00:00 2001 From: Andreas Schneider Date: Wed, 13 Nov 2019 10:06:20 +0100 Subject: auth:pycreds: Check return code of netlogon_creds_client_authenticator() BUG: https://bugzilla.samba.org/show_bug.cgi?id=14195 Signed-off-by: Andreas Schneider Reviewed-by: Andrew Bartlett --- auth/credentials/pycredentials.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'auth') diff --git a/auth/credentials/pycredentials.c b/auth/credentials/pycredentials.c index 446f30970a2..7427e286dca 100644 --- a/auth/credentials/pycredentials.c +++ b/auth/credentials/pycredentials.c @@ -832,6 +832,7 @@ static PyObject *py_creds_new_client_authenticator(PyObject *self, struct cli_credentials *creds = NULL; struct netlogon_creds_CredentialState *nc = NULL; PyObject *ret = NULL; + NTSTATUS status; creds = PyCredentials_AsCliCredentials(self); if (creds == NULL) { @@ -848,9 +849,13 @@ static PyObject *py_creds_new_client_authenticator(PyObject *self, return NULL; } - netlogon_creds_client_authenticator( - nc, - &auth); + status = netlogon_creds_client_authenticator(nc, &auth); + if (!NT_STATUS_IS_OK(status)) { + PyErr_SetString(PyExc_ValueError, + "Failed to create client authenticator"); + return NULL; + } + ret = Py_BuildValue("{s"PYARG_BYTES_LEN"si}", "credential", (const char *) &auth.cred, sizeof(auth.cred), -- cgit v1.2.1