From: Andrew Bartlett <abartlet@samba.org>
Date: Wed, 3 Aug 2011 22:38:21 +0000 (+1000)
Subject: s3-ntlmssp void function cannot return value
X-Git-Tag: talloc-2.0.6~64
X-Git-Url: http://git.samba.org/?p=amitay%2Fsamba.git;a=commitdiff_plain;h=ed59f21f65bf9c16422a01068535bf4a862372d7

s3-ntlmssp void function cannot return value

Removing the return is reasonable here because while no callers
currently specify more than one flag at a time, the
ntlmssp_want_feature code allows it.

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Aug  4 02:19:46 CEST 2011 on sn-devel-104
---

diff --git a/source3/libsmb/ntlmssp_wrap.c b/source3/libsmb/ntlmssp_wrap.c
index 31ba8f5306a..ad8ad27d51e 100644
--- a/source3/libsmb/ntlmssp_wrap.c
+++ b/source3/libsmb/ntlmssp_wrap.c
@@ -133,13 +133,13 @@ void auth_ntlmssp_want_feature(struct auth_ntlmssp_state *ans, uint32_t feature)
 {
 	if (ans->gensec_security) {
 		if (feature & NTLMSSP_FEATURE_SESSION_KEY) {
-			return gensec_want_feature(ans->gensec_security, GENSEC_FEATURE_SESSION_KEY);
+			gensec_want_feature(ans->gensec_security, GENSEC_FEATURE_SESSION_KEY);
 		}
 		if (feature & NTLMSSP_FEATURE_SIGN) {
-			return gensec_want_feature(ans->gensec_security, GENSEC_FEATURE_SIGN);
+			gensec_want_feature(ans->gensec_security, GENSEC_FEATURE_SIGN);
 		}
 		if (feature & NTLMSSP_FEATURE_SEAL) {
-			return gensec_want_feature(ans->gensec_security, GENSEC_FEATURE_SEAL);
+			gensec_want_feature(ans->gensec_security, GENSEC_FEATURE_SEAL);
 		}
 	} else {
 		ntlmssp_want_feature(ans->ntlmssp_state, feature);