From: Andreas Schneider Date: Wed, 13 Nov 2019 09:06:20 +0000 (+0100) Subject: auth:pycreds: Check return code of netlogon_creds_client_authenticator() X-Git-Tag: talloc-2.3.1~30 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=d3fffca5e9ab45b43fa3f460ad6051356c9a00a9;p=thirdparty%2Fsamba.git 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 --- 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),