]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
source/nfq: Improve retry counter limit name 8787/head
authorJeff Lucovsky <jlucovsky@oisf.net>
Mon, 3 Apr 2023 13:01:55 +0000 (09:01 -0400)
committerVictor Julien <vjulien@oisf.net>
Mon, 1 May 2023 06:25:14 +0000 (08:25 +0200)
The NFQ verdict retry is based on attempts, not time; thus, we rename
the token controlling it to reflect that.

src/source-nfq.c

index b0f0eba04ee76b1a32880763cb2905fdc47f4a74..d7222da694e2c9a2e43b573dae5541864417a11c 100644 (file)
@@ -100,7 +100,7 @@ static TmEcode NoNFQSupportExit(ThreadVars *tv, const void *initdata, void **dat
 extern intmax_t max_pending_packets;
 
 #define MAX_ALREADY_TREATED 5
-#define NFQ_VERDICT_RETRY_TIME 3
+#define NFQ_VERDICT_RETRY_COUNT 3
 static int already_seen_warning;
 static int runmode_workers;
 
@@ -315,7 +315,7 @@ static void NFQVerdictCacheFlush(NFQQueueVars *t)
             ret = nfq_set_verdict_batch(t->qh,
                                         t->verdict_cache.packet_id,
                                         t->verdict_cache.verdict);
-    } while ((ret < 0) && (iter++ < NFQ_VERDICT_RETRY_TIME));
+    } while ((ret < 0) && (iter++ < NFQ_VERDICT_RETRY_COUNT));
 
     if (ret < 0) {
         SCLogWarning("nfq_set_verdict_batch failed: %s", strerror(errno));
@@ -428,7 +428,7 @@ static int NFQSetupPkt (Packet *p, struct nfq_q_handle *qh, void *data)
             already_seen_warning++;
             do {
                 ret = nfq_set_verdict(qh, p->nfq_v.id, NF_ACCEPT, 0, NULL);
-            } while ((ret < 0) && (iter++ < NFQ_VERDICT_RETRY_TIME));
+            } while ((ret < 0) && (iter++ < NFQ_VERDICT_RETRY_COUNT));
             if (ret < 0) {
                 SCLogWarning(
                         "nfq_set_verdict of %p failed %" PRId32 ": %s", p, ret, strerror(errno));
@@ -1175,7 +1175,7 @@ TmEcode NFQSetVerdict(Packet *p)
 #endif /* HAVE_NFQ_SET_VERDICT2 */
                 break;
         }
-    } while ((ret < 0) && (iter++ < NFQ_VERDICT_RETRY_TIME));
+    } while ((ret < 0) && (iter++ < NFQ_VERDICT_RETRY_COUNT));
 
     NFQMutexUnlock(t);