diff options
| author | Tobias Brunner <tobias@strongswan.org> | 2017-09-14 19:11:10 +0200 |
|---|---|---|
| committer | Tobias Brunner <tobias@strongswan.org> | 2017-09-18 12:16:54 +0200 |
| commit | 787cc3a4e59726e68765dbeb6a3f282c7b00f463 (patch) | |
| tree | d769737783387e81a6a25b456e783f240a064dd4 /src/libcharon/plugins/dhcp | |
| parent | d04deff4dfd9ef496d19a17742fa3baa25d83544 (diff) | |
| download | strongswan-787cc3a4e59726e68765dbeb6a3f282c7b00f463.tar.bz2 strongswan-787cc3a4e59726e68765dbeb6a3f282c7b00f463.tar.xz | |
dhcp: Don't use signed char for DHCP options
The value of DHCP_OPTEND is 255. When it is assigned this result in a
sign change as the positive int constant is cast to a signed char and -1
results. Clang 4.0 complains about this.
Diffstat (limited to 'src/libcharon/plugins/dhcp')
| -rw-r--r-- | src/libcharon/plugins/dhcp/dhcp_socket.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libcharon/plugins/dhcp/dhcp_socket.c b/src/libcharon/plugins/dhcp/dhcp_socket.c index 42f8f1ef9..4f4e1ffe0 100644 --- a/src/libcharon/plugins/dhcp/dhcp_socket.c +++ b/src/libcharon/plugins/dhcp/dhcp_socket.c @@ -176,7 +176,7 @@ typedef struct __attribute__((packed)) { char server_hostname[64]; char boot_filename[128]; uint32_t magic_cookie; - char options[252]; + u_char options[252]; } dhcp_t; /** |