diff options
| author | Ivan Nardi <12729895+IvanNardi@users.noreply.github.com> | 2025-01-19 21:02:03 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-01-19 21:02:03 +0100 |
| commit | 45066c4a0d7c82b2fa6ee42136d5fd8974ad3fc2 (patch) | |
| tree | eb7a1ee282f69949165054a9f9ca090ff1cb6cf8 | |
| parent | 0d18746e09ac79459fbfe81e4ee67fd960463353 (diff) | |
OpenVPN: fix a warning (#2686)
```
protocols/openvpn.c:378:11: error: variable 'iter' set but not used [-Werror,-Wunused-but-set-variable]
378 | int rc, iter, offset;
```
| -rw-r--r-- | src/lib/protocols/openvpn.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/lib/protocols/openvpn.c b/src/lib/protocols/openvpn.c index dab6a77ab..43076590a 100644 --- a/src/lib/protocols/openvpn.c +++ b/src/lib/protocols/openvpn.c @@ -375,7 +375,10 @@ static int search_heur_opcode(struct ndpi_detection_module_struct* ndpi_struct, u_int16_t ovpn_payload_len = packet->payload_packet_len; int dir = packet->packet_direction; u_int16_t pdu_len; - int rc, iter, offset; + int rc, offset; +#ifdef NDPI_ENABLE_DEBUG_MESSAGES + int iter; +#endif /* To reduce false positives number, trigger the heuristic only for flows to suspicious/unknown addresses */ @@ -409,7 +412,9 @@ static int search_heur_opcode(struct ndpi_detection_module_struct* ndpi_struct, offset = 0; } +#ifdef NDPI_ENABLE_DEBUG_MESSAGES iter = 0; +#endif rc = 1; /* Exclude */ while(offset + 2 + 1 /* The first byte is the opcode */ <= ovpn_payload_len) { pdu_len = ntohs((*(u_int16_t *)(ovpn_payload + offset))); @@ -434,7 +439,9 @@ static int search_heur_opcode(struct ndpi_detection_module_struct* ndpi_struct, flow->ovpn_heur_opcode__missing_bytes[dir]); return 0; /* Continue */ } +#ifdef NDPI_ENABLE_DEBUG_MESSAGES iter++; +#endif } return rc; } else { |