s3-ntlmssp void function cannot return value
authorAndrew Bartlett <abartlet@samba.org>
Wed, 3 Aug 2011 22:38:21 +0000 (08:38 +1000)
committerAndrew Bartlett <abartlet@samba.org>
Thu, 4 Aug 2011 00:19:46 +0000 (02:19 +0200)
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

source3/libsmb/ntlmssp_wrap.c

index 31ba8f5306aae5c8ccc7aadb6690efe7dd827d7a..ad8ad27d51ec1e5794e3a2b0c56c09270a4368d1 100644 (file)
@@ -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);