Skip to content

Commit 9492bfa

Browse files
committed
Mark defined numbers as unsigned
Oss-Fuzz complains that shifting an "int" left by 31 is problematic This is fine in practice at least on the compiler used in common distributions. But let's try to address this undefined bheavior by making it clear to the compiler that we want to treat these numbers as unsigned ints. Signed-off-by: Simo Sorce <simo@redhat.com>
1 parent b133254 commit 9492bfa

File tree

1 file changed

+32
-32
lines changed

1 file changed

+32
-32
lines changed

src/ntlm.h

Lines changed: 32 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -8,38 +8,38 @@
88
#include "ntlm_common.h"
99

1010
/* Negotiate Flags */
11-
#define NTLMSSP_NEGOTIATE_56 (1 << 31)
12-
#define NTLMSSP_NEGOTIATE_KEY_EXCH (1 << 30)
13-
#define NTLMSSP_NEGOTIATE_128 (1 << 29)
14-
#define UNUSED_R1 (1 << 28)
15-
#define UNUSED_R2 (1 << 27)
16-
#define UNUSED_R3 (1 << 26)
17-
#define NTLMSSP_NEGOTIATE_VERSION (1 << 25)
18-
#define UNUSED_R4 (1 << 24)
19-
#define NTLMSSP_NEGOTIATE_TARGET_INFO (1 << 23)
20-
#define NTLMSSP_REQUEST_NON_NT_SESSION_KEY (1 << 22)
21-
#define UNUSED_R5 /* Davenport: NEGOTIATE_ACCEPT */ (1 << 21)
22-
#define NTLMSSP_NEGOTIATE_IDENTIFY (1 << 20)
23-
#define NTLMSSP_NEGOTIATE_EXTENDED_SESSIONSECURITY (1 << 19)
24-
#define UNUSED_R6 /* Davenport:TARGET_TYPE_SHARE */ (1 << 18)
25-
#define NTLMSSP_TARGET_TYPE_SERVER (1 << 17)
26-
#define NTLMSSP_TARGET_TYPE_DOMAIN (1 << 16)
27-
#define NTLMSSP_NEGOTIATE_ALWAYS_SIGN (1 << 15)
28-
#define UNUSED_R7 /* Davenport:LOCAL_CALL */ (1 << 14)
29-
#define NTLMSSP_NEGOTIATE_OEM_WORKSTATION_SUPPLIED (1 << 13)
30-
#define NTLMSSP_NEGOTIATE_OEM_DOMAIN_SUPPLIED (1 << 12)
31-
#define NTLMSSP_ANONYMOUS (1 << 11)
32-
#define UNUSED_R8 (1 << 10)
33-
#define NTLMSSP_NEGOTIATE_NTLM (1 << 9)
34-
#define UNUSED_R9 (1 << 8)
35-
#define NTLMSSP_NEGOTIATE_LM_KEY (1 << 7)
36-
#define NTLMSSP_NEGOTIATE_DATAGRAM (1 << 6)
37-
#define NTLMSSP_NEGOTIATE_SEAL (1 << 5)
38-
#define NTLMSSP_NEGOTIATE_SIGN (1 << 4)
39-
#define UNUSED_R10 (1 << 3)
40-
#define NTLMSSP_REQUEST_TARGET (1 << 2)
41-
#define NTLMSSP_NEGOTIATE_OEM (1 << 1)
42-
#define NTLMSSP_NEGOTIATE_UNICODE (1 << 0)
11+
#define NTLMSSP_NEGOTIATE_56 (1U << 31)
12+
#define NTLMSSP_NEGOTIATE_KEY_EXCH (1U << 30)
13+
#define NTLMSSP_NEGOTIATE_128 (1U << 29)
14+
#define UNUSED_R1 (1U << 28)
15+
#define UNUSED_R2 (1U << 27)
16+
#define UNUSED_R3 (1U << 26)
17+
#define NTLMSSP_NEGOTIATE_VERSION (1U << 25)
18+
#define UNUSED_R4 (1U << 24)
19+
#define NTLMSSP_NEGOTIATE_TARGET_INFO (1U << 23)
20+
#define NTLMSSP_REQUEST_NON_NT_SESSION_KEY (1U << 22)
21+
#define UNUSED_R5 /* Davenport: NEGOTIATE_ACCEPT */ (1U << 21)
22+
#define NTLMSSP_NEGOTIATE_IDENTIFY (1U << 20)
23+
#define NTLMSSP_NEGOTIATE_EXTENDED_SESSIONSECURITY (1U << 19)
24+
#define UNUSED_R6 /* Davenport:TARGET_TYPE_SHARE */ (1U << 18)
25+
#define NTLMSSP_TARGET_TYPE_SERVER (1U << 17)
26+
#define NTLMSSP_TARGET_TYPE_DOMAIN (1U << 16)
27+
#define NTLMSSP_NEGOTIATE_ALWAYS_SIGN (1U << 15)
28+
#define UNUSED_R7 /* Davenport:LOCAL_CALL */ (1U << 14)
29+
#define NTLMSSP_NEGOTIATE_OEM_WORKSTATION_SUPPLIED (1U << 13)
30+
#define NTLMSSP_NEGOTIATE_OEM_DOMAIN_SUPPLIED (1U << 12)
31+
#define NTLMSSP_ANONYMOUS (1U << 11)
32+
#define UNUSED_R8 (1U << 10)
33+
#define NTLMSSP_NEGOTIATE_NTLM (1U << 9)
34+
#define UNUSED_R9 (1U << 8)
35+
#define NTLMSSP_NEGOTIATE_LM_KEY (1U << 7)
36+
#define NTLMSSP_NEGOTIATE_DATAGRAM (1U << 6)
37+
#define NTLMSSP_NEGOTIATE_SEAL (1U << 5)
38+
#define NTLMSSP_NEGOTIATE_SIGN (1U << 4)
39+
#define UNUSED_R10 (1U << 3)
40+
#define NTLMSSP_REQUEST_TARGET (1U << 2)
41+
#define NTLMSSP_NEGOTIATE_OEM (1U << 1)
42+
#define NTLMSSP_NEGOTIATE_UNICODE (1U << 0)
4343

4444
/* (2.2.2.10 VERSION) */
4545
#define WINDOWS_MAJOR_VERSION_5 0x05

0 commit comments

Comments
 (0)