sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2025-02-25 17:57:42 +03:00
Code

ntlmssp: fix compilation with -O2 -fno-inline

Browse Source 
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>
This commit is contained in:
Douglas Bagnall 2016-12-21 13:24:46 +13:00 committed by Andrew Bartlett
parent 2ad69f4bd0
commit 3ee56607db
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
auth/ntlmssp/ntlmssp.c
View File

@ -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,