diff options
| author | Douglas Bagnall <douglas.bagnall@catalyst.net.nz> | 2016-12-21 13:24:46 +1300 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2017-02-10 05:09:09 +0100 |
| commit | 3ee56607db8b4fedd5d04f74de548ae7324686cd (patch) | |
| tree | 0ee0a219c3ea511a87af4cf881603f540ee56ed2 /auth/ntlmssp | |
| parent | 2ad69f4bd0f724296a40347a140a031f6ab341e9 (diff) | |
| download | samba-3ee56607db8b4fedd5d04f74de548ae7324686cd.tar.gz | |
ntlmssp: fix compilation with -O2 -fno-inline
Without inlining the function, GCC doesn't know that
gensec_ntlmssp->ntlmssp_state->role always has a valid value.
With inlining, this is obviously redundant but GCC clearly knows
enough to detect this and elide the default case.
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Diffstat (limited to 'auth/ntlmssp')
| -rw-r--r-- | auth/ntlmssp/ntlmssp.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/auth/ntlmssp/ntlmssp.c b/auth/ntlmssp/ntlmssp.c index 4abab88627a..3283c964470 100644 --- a/auth/ntlmssp/ntlmssp.c +++ b/auth/ntlmssp/ntlmssp.c @@ -107,6 +107,10 @@ static NTSTATUS gensec_ntlmssp_update_find(struct gensec_security *gensec_securi return NT_STATUS_INVALID_PARAMETER; } break; + default: + DEBUG(1, ("NTLMSSP state has invalid role %d\n", + gensec_ntlmssp->ntlmssp_state->role)); + return NT_STATUS_INVALID_PARAMETER; } } else { if (!msrpc_parse(gensec_ntlmssp->ntlmssp_state, |