]> git.ipfire.org Git - people/ms/suricata.git/commitdiff
detect: remove unused arg from generic list inspect
authorVictor Julien <victor@inliniac.net>
Mon, 19 Oct 2020 13:33:36 +0000 (15:33 +0200)
committerVictor Julien <victor@inliniac.net>
Mon, 7 Dec 2020 19:47:56 +0000 (20:47 +0100)
29 files changed:
src/detect-dce-iface.c
src/detect-dnp3.c
src/detect-dns-opcode.c
src/detect-engine-dns.c
src/detect-engine.c
src/detect-engine.h
src/detect-ftpbounce.c
src/detect-ftpdata.c
src/detect-http2.c
src/detect-krb5-errcode.c
src/detect-krb5-msgtype.c
src/detect-lua.c
src/detect-mqtt-connack-sessionpresent.c
src/detect-mqtt-connect-flags.c
src/detect-mqtt-flags.c
src/detect-mqtt-protocol-version.c
src/detect-mqtt-qos.c
src/detect-mqtt-reason-code.c
src/detect-mqtt-type.c
src/detect-nfs-procedure.c
src/detect-nfs-version.c
src/detect-rfb-secresult.c
src/detect-rfb-sectype.c
src/detect-snmp-pdu_type.c
src/detect-snmp-version.c
src/detect-ssh-software-version.c
src/detect-ssl-state.c
src/detect-tls-cert-validity.c
src/detect-tls.c

index 2383efba8dca71a20e4f9792796a52023ec6778e..3b134ec7bfb47c6b9965c93f78865ae2b09226fd 100644 (file)
@@ -100,7 +100,7 @@ static int InspectDceGeneric(DetectEngineCtx *de_ctx, DetectEngineThreadCtx *det
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 6ffa42f25af2d43bf0361204ca08563f531aa899..d0f78e7ac754a30c6af03e7a620bc1c862547d4b 100644 (file)
@@ -181,7 +181,7 @@ static int DetectEngineInspectDNP3(DetectEngineCtx *de_ctx, DetectEngineThreadCt
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index bbb58d2ccaa78eef2fd98a3792db1a13c67b260f..e4356c47bf189822815f9695c0ab06386e7f54b0 100644 (file)
@@ -79,7 +79,7 @@ static int DetectEngineInspectRequestGenericDnsOpcode(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 void DetectDnsOpcodeRegister(void)
index 77721b3d4193d80522fe5c80b0ea5b26ec454581..c3d6873b3f30fc76c9629b37ea638891c1222efe 100644 (file)
@@ -52,7 +52,7 @@ int DetectEngineInspectDnsRequest(DetectEngineCtx *de_ctx, DetectEngineThreadCtx
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 int DetectEngineInspectDnsResponse(DetectEngineCtx *de_ctx, DetectEngineThreadCtx *det_ctx,
@@ -60,5 +60,5 @@ int DetectEngineInspectDnsResponse(DetectEngineCtx *de_ctx, DetectEngineThreadCt
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
index 7e6633483dbd6fb1e7706a0ff76775354cf2996e..cbb681963ba7da0724c0794a61180b7f93c294f2 100644 (file)
@@ -1529,12 +1529,9 @@ int DetectEngineReloadIsIdle(void)
  *  \retval 0 no match
  *  \retval 1 match
  */
-int DetectEngineInspectGenericList(ThreadVars *tv,
-                                   const DetectEngineCtx *de_ctx,
-                                   DetectEngineThreadCtx *det_ctx,
-                                   const Signature *s, const SigMatchData *smd,
-                                   Flow *f, const uint8_t flags,
-                                   void *alstate, void *txv, uint64_t tx_id)
+int DetectEngineInspectGenericList(const DetectEngineCtx *de_ctx, DetectEngineThreadCtx *det_ctx,
+        const Signature *s, const SigMatchData *smd, Flow *f, const uint8_t flags, void *alstate,
+        void *txv, uint64_t tx_id)
 {
     SCLogDebug("running match functions, sm %p", smd);
     if (smd != NULL) {
index 7d61ff91e71a5822c283b5deff1fb3e6a92f427d..dc72b2f670955db9d018132f9b258cbcc95db8ee 100644 (file)
@@ -114,11 +114,8 @@ int DetectEngineTentantUnregisterVlanId(uint32_t tenant_id, uint16_t vlan_id);
 int DetectEngineTentantRegisterPcapFile(uint32_t tenant_id);
 int DetectEngineTentantUnregisterPcapFile(uint32_t tenant_id);
 
-int DetectEngineInspectGenericList(ThreadVars *, const DetectEngineCtx *,
-                                   DetectEngineThreadCtx *,
-                                   const Signature *, const SigMatchData *,
-                                   Flow *, const uint8_t, void *, void *,
-                                   uint64_t);
+int DetectEngineInspectGenericList(const DetectEngineCtx *, DetectEngineThreadCtx *,
+        const Signature *, const SigMatchData *, Flow *, const uint8_t, void *, void *, uint64_t);
 
 int DetectEngineInspectBufferGeneric(
         DetectEngineCtx *de_ctx, DetectEngineThreadCtx *det_ctx,
index 5e55676b276b663db502d1cbb9b77c50b8522dbf..244b6387ddf087d1df9e1d112b3d21970333eec9 100644 (file)
@@ -88,7 +88,7 @@ static int InspectFtpRequest(DetectEngineCtx *de_ctx, DetectEngineThreadCtx *det
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 0c08e2f383e0b80786f40fded5bf4ff300095134..1d38e17a554d05a3b83163cf729c14149d30cd5a 100644 (file)
@@ -92,7 +92,7 @@ static int DetectEngineInspectFtpdataGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 3a986d99cb0db8d8992d179b46c0e46704fad140..af51cd1eb0f7b3498985edaec09e2b0acbf90374 100644 (file)
@@ -116,7 +116,7 @@ static int DetectEngineInspectHTTP2(DetectEngineCtx *de_ctx, DetectEngineThreadC
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 44012f4b1cfaf1cdd13a177dd108ddfef8aa73f1..c6c6e32d9809bab4ddc4dca6f87204f56acfdace 100644 (file)
@@ -91,7 +91,7 @@ static int DetectEngineInspectKRB5Generic(DetectEngineCtx *de_ctx, DetectEngineT
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index fe0a6355c2488144003a97a46ab2eabe804d7ce7..5c291d7464e52381f6295f3b64c62c9de4f78e63 100644 (file)
@@ -91,7 +91,7 @@ static int DetectEngineInspectKRB5Generic(DetectEngineCtx *de_ctx, DetectEngineT
         void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 1ff14226dc18b984e4ed7f3129a1fb60fb33ffe4..3ce903a8f6f8bfb33eb0c066d15aec6f069337a2 100644 (file)
@@ -139,7 +139,7 @@ static int InspectSmtpGeneric(DetectEngineCtx *de_ctx, DetectEngineThreadCtx *de
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 #define DATATYPE_PACKET                     (1<<0)
index 4e195e150b7d30b7e0dbf2abf8141d463a27a7a9..cb3321b2e97865170e44635b50b4129444979f13 100644 (file)
@@ -77,7 +77,7 @@ static int DetectEngineInspectMQTTConnackSessionPresentGeneric(DetectEngineCtx *
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 8447d31a2b9470d4de2a7fdb45919478d88f7ac9..6054bd4acf6fd455389cf2ed3001d94baf38c69a 100644 (file)
@@ -85,7 +85,7 @@ static int DetectEngineInspectMQTTConnectFlagsGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 59c8fd730359fb9ed8ed49a9adb8964663b03c9c..a2cde067b40906f6363a1c194eaa48c9e790dfd5 100644 (file)
@@ -81,7 +81,7 @@ static int DetectEngineInspectMQTTFlagsGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index bc0f9e05ac108402911721960c13e370e9dfa43a..3ad43b43a9c6e746ae76b8190575c72f8ab63b31 100644 (file)
@@ -74,7 +74,7 @@ static int DetectEngineInspectMQTTProtocolVersionGeneric(DetectEngineCtx *de_ctx
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 588f5d504a3275bcdd9a64fbfb8ca19dcb6b9a01..7166dd13d03dfc6611cd5c7bd0d17d5c3e9f494f 100644 (file)
@@ -73,7 +73,7 @@ static int DetectEngineInspectMQTTQosGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 8a1adba954d00d4d454a26d71ea99124a2925e5f..1fa1b7e93463568ecf777a31174042bba3bc09f7 100644 (file)
@@ -79,7 +79,7 @@ static int DetectEngineInspectMQTTReasonCodeGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index c75c84d833fba5defd137a86e85e50aa07d2efb7..493be73f8a971e26e5944b207409d678bb8e7ea7 100644 (file)
@@ -72,7 +72,7 @@ static int DetectEngineInspectMQTTTypeGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index fc51af7208c7b45b19773db7f2d5a1ea9e271215..b944d0507ed05c7c4178e65678d04f3d323debed 100644 (file)
@@ -115,7 +115,7 @@ static int DetectEngineInspectNfsRequestGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 static inline int
index f65753b2aec9f00fc5d033a3ddd941168a620cbf..74ac4cd6625b33ac1390cba24f1822dc3ab32b84 100644 (file)
@@ -114,7 +114,7 @@ static int DetectEngineInspectNfsRequestGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 static inline int
index 92435a01d4ae71a5c25c4fd96db7a699da056e07..6f9915ca653bfba1b360f3a746c0456c2db1aba5 100644 (file)
@@ -82,7 +82,7 @@ static int DetectEngineInspectRfbSecresultGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 enum {
index 39e840e8e1a636f1c4b57e4901c18f36db09b9b6..3f254c6da6957f503be62b4126778a6109081a26 100644 (file)
@@ -89,7 +89,7 @@ static int DetectEngineInspectRfbSectypeGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 static inline int SectypeMatch(const uint32_t version,
index ad198c1d37a15c3b0f29d19d26b18924a6e02887..27620798314c24e4d8a7a4ec2bfff37bd3f509c6 100644 (file)
@@ -86,7 +86,7 @@ static int DetectEngineInspectSNMPRequestGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 76b166d78c156fc6c0055f4227548ba361016418..8013202d15dbc7d50912b9930fe14ac11769ea34 100644 (file)
@@ -98,7 +98,7 @@ static int DetectEngineInspectSNMPRequestGeneric(DetectEngineCtx *de_ctx,
         const Signature *s, Flow *f, uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 static inline int
index 4a603f24aaa98f844e1e5eb02ad9d51473c67b4a..0db75aa6536b7e0d447e135e81638d04e31884ac 100644 (file)
@@ -81,7 +81,7 @@ static int InspectSshBanner(DetectEngineCtx *de_ctx, DetectEngineThreadCtx *det_
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 2af5db1a4b9cd43ceeb094a8b910605652e42796..958a7116fa37867af086db56cf97743b397d6d5f 100644 (file)
@@ -105,7 +105,7 @@ static int InspectTlsGeneric(DetectEngineCtx *de_ctx, DetectEngineThreadCtx *det
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**
index 8d4fb446607fc8485f9a3a1e095f54d37a878242..aca8a697979697fa3140e562df2510ef5a160f81 100644 (file)
@@ -139,8 +139,9 @@ static int DetectEngineInspectTlsValidity(DetectEngineCtx *de_ctx, DetectEngineT
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
+
 /**
  * \internal
  * \brief Function to match validity field in a tls certificate.
index 4585d9f4fae7a9e74fe80db17f62444c16d7c694..626bd2de9ea5be086b9a78d87833f0a4535892e1 100644 (file)
@@ -100,7 +100,7 @@ static int InspectTlsCert(DetectEngineCtx *de_ctx, DetectEngineThreadCtx *det_ct
         uint8_t flags, void *alstate, void *txv, uint64_t tx_id)
 {
     return DetectEngineInspectGenericList(
-            NULL, de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
+            de_ctx, det_ctx, s, engine->smd, f, flags, alstate, txv, tx_id);
 }
 
 /**