From: Victor Julien <vjulien@oisf.net>
Date: Wed, 9 Feb 2022 07:31:01 +0000 (+0100)
Subject: app-layer: fix counter setup logic
X-Git-Tag: suricata-7.0.0-beta1~914
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=84d91e2e0d6f46448e441707fdee0d473380aee2;p=thirdparty%2Fsuricata.git

app-layer: fix counter setup logic

Completes 0ccf5b914766335425d262de7f69b959c527329c
---

diff --git a/src/app-layer.c b/src/app-layer.c
index 327abe68a1..bf51a3375f 100644
--- a/src/app-layer.c
+++ b/src/app-layer.c
@@ -978,27 +978,25 @@ void AppLayerRegisterGlobalCounters(void)
 void AppLayerSetupCounters()
 {
     uint8_t ipprotos[] = { IPPROTO_TCP, IPPROTO_UDP };
-    uint8_t ipproto;
-    AppProto alproto;
     AppProto alprotos[ALPROTO_MAX];
     const char *str = "app_layer.flow.";
     const char *estr = "app_layer.error.";
 
     AppLayerProtoDetectSupportedAppProtocols(alprotos);
 
-    for (ipproto = 0; ipproto < IPPROTOS_MAX; ipproto++) {
-        uint8_t ipproto_map = FlowGetProtoMapping(ipprotos[ipproto]);
-        uint8_t other_ipproto = (ipprotos[ipproto] == IPPROTO_TCP) ? IPPROTO_UDP : IPPROTO_TCP;
-        const char *ipproto_suffix = (ipprotos[ipproto] == IPPROTO_TCP) ? "_tcp" : "_udp";
+    for (uint8_t p = 0; p < IPPROTOS_MAX; p++) {
+        const uint8_t ipproto = ipprotos[p];
+        const uint8_t ipproto_map = FlowGetProtoMapping(ipproto);
+        const uint8_t other_ipproto = ipproto == IPPROTO_TCP ? IPPROTO_UDP : IPPROTO_TCP;
+        const char *ipproto_suffix = (ipproto == IPPROTO_TCP) ? "_tcp" : "_udp";
 
-        for (alproto = 0; alproto < ALPROTO_MAX; alproto++) {
+        for (AppProto alproto = 0; alproto < ALPROTO_MAX; alproto++) {
             if (alprotos[alproto] == 1) {
                 const char *tx_str = "app_layer.tx.";
                 const char *alproto_str = AppLayerGetProtoName(alproto);
 
-                if (AppLayerParserProtoIsRegistered(ipprotos[ipproto], alproto) &&
-                    AppLayerParserProtoIsRegistered(other_ipproto, alproto))
-                {
+                if (AppLayerParserProtoIsRegistered(ipproto, alproto) &&
+                        AppLayerParserProtoIsRegistered(other_ipproto, alproto)) {
                     snprintf(applayer_counter_names[ipproto_map][alproto].name,
                             sizeof(applayer_counter_names[ipproto_map][alproto].name),
                             "%s%s%s", str, alproto_str, ipproto_suffix);