From 45c74c8084d2db14fef6a79cd98068be2ab73f30 Mon Sep 17 00:00:00 2001 From: Stefan Metzmacher Date: Sat, 3 Aug 2013 11:26:13 +0200 Subject: auth/gensec: first check GENSEC_FEATURE_SESSION_KEY before returning NOT_IMPLEMENTED Preferr NT_STATUS_NO_USER_SESSION_KEY as return value of gensec_session_key(). Signed-off-by: Stefan Metzmacher Reviewed-by: Andrew Bartlett --- auth/gensec/gensec.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'auth') diff --git a/auth/gensec/gensec.c b/auth/gensec/gensec.c index ea62861797..9a8f0ef15a 100644 --- a/auth/gensec/gensec.c +++ b/auth/gensec/gensec.c @@ -155,13 +155,14 @@ _PUBLIC_ NTSTATUS gensec_session_key(struct gensec_security *gensec_security, TALLOC_CTX *mem_ctx, DATA_BLOB *session_key) { - if (!gensec_security->ops->session_key) { - return NT_STATUS_NOT_IMPLEMENTED; - } if (!gensec_have_feature(gensec_security, GENSEC_FEATURE_SESSION_KEY)) { return NT_STATUS_NO_USER_SESSION_KEY; } + if (!gensec_security->ops->session_key) { + return NT_STATUS_NOT_IMPLEMENTED; + } + return gensec_security->ops->session_key(gensec_security, mem_ctx, session_key); } -- cgit