Browse code
forward: Make sure pip flags are treated as unsigned
process_ip_header already expects them to be unsigned,
make sure the flags are to avoid spurious conversion warnings.
Change-Id: I6d42c67b8dc5512933bed482bd9c2be80c63e993
Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <20250901194531.13683-1-gert@greenie.muc.de>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg32740.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>
Frank Lichtenheld authored on 2025/09/02 04:45:25Showing 1 changed files
| ... | ... |
@@ -314,13 +314,13 @@ bool send_control_channel_string_dowork(struct tls_session *session, const char |
| 314 | 314 |
*/ |
| 315 | 315 |
void reschedule_multi_process(struct context *c); |
| 316 | 316 |
|
| 317 |
-#define PIPV4_PASSTOS (1 << 0) |
|
| 318 |
-#define PIP_MSSFIX (1 << 1) /* v4 and v6 */ |
|
| 319 |
-#define PIP_OUTGOING (1 << 2) |
|
| 320 |
-#define PIPV4_EXTRACT_DHCP_ROUTER (1 << 3) |
|
| 321 |
-#define PIPV4_CLIENT_NAT (1 << 4) |
|
| 322 |
-#define PIPV6_ICMP_NOHOST_CLIENT (1 << 5) |
|
| 323 |
-#define PIPV6_ICMP_NOHOST_SERVER (1 << 6) |
|
| 317 |
+#define PIPV4_PASSTOS (1u << 0) |
|
| 318 |
+#define PIP_MSSFIX (1u << 1) /* v4 and v6 */ |
|
| 319 |
+#define PIP_OUTGOING (1u << 2) |
|
| 320 |
+#define PIPV4_EXTRACT_DHCP_ROUTER (1u << 3) |
|
| 321 |
+#define PIPV4_CLIENT_NAT (1u << 4) |
|
| 322 |
+#define PIPV6_ICMP_NOHOST_CLIENT (1u << 5) |
|
| 323 |
+#define PIPV6_ICMP_NOHOST_SERVER (1u << 6) |
|
| 324 | 324 |
|
| 325 | 325 |
|
| 326 | 326 |
void process_ip_header(struct context *c, unsigned int flags, struct buffer *buf, |