]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
app-layer: make dns,smb,tls parsers less noisy w/o config
authorVictor Julien <victor@inliniac.net>
Thu, 17 Oct 2019 05:38:46 +0000 (07:38 +0200)
committerVictor Julien <victor@inliniac.net>
Thu, 17 Oct 2019 05:38:46 +0000 (07:38 +0200)
src/app-layer-dns-tcp.c
src/app-layer-dns-udp.c
src/app-layer-smb.c
src/app-layer-ssl.c

index ccc13c767280baca25bd44e4d1b0983fd5a8549f..ac0f4b5de7c533e331ffc5574e35336216af9050 100644 (file)
@@ -131,16 +131,15 @@ void RegisterDNSTCPParsers(void)
                     sizeof(DNSHeader) + 2, RustDNSTCPProbe, RustDNSTCPProbe);
             /* if we have no config, we enable the default port 53 */
             if (!have_cfg) {
-                SCLogWarning(SC_ERR_DNS_CONFIG, "no DNS TCP config found, "
-                                                "enabling DNS detection on "
-                                                "port 53.");
+                SCLogConfig("no DNS TCP config found, enabling DNS detection "
+                            "on port 53.");
                 AppLayerProtoDetectPPRegister(IPPROTO_TCP, "53", ALPROTO_DNS, 0,
                         sizeof(DNSHeader) + 2, STREAM_TOSERVER, RustDNSTCPProbe,
                         RustDNSTCPProbe);
             }
         }
     } else {
-        SCLogInfo("Protocol detection and parser disabled for %s protocol.",
+        SCLogConfig("Protocol detection and parser disabled for %s protocol.",
                   proto_name);
         return;
     }
@@ -175,7 +174,7 @@ void RegisterDNSTCPParsers(void)
                 APP_LAYER_PARSER_OPT_ACCEPT_GAPS);
 
     } else {
-        SCLogInfo("Parsed disabled for %s protocol. Protocol detection"
+        SCLogConfig("Parsed disabled for %s protocol. Protocol detection"
                   "still on.", proto_name);
     }
 
index 955f2b82d4ffc72a5c0cd48cb7726083cf39819c..f2fab7fc2345d85c104e49bd8c719ba503223e00 100644 (file)
@@ -139,7 +139,7 @@ void RegisterDNSUDPParsers(void)
             /* If no config, enable on port 53. */
             if (!have_cfg) {
 #ifndef AFLFUZZ_APPLAYER
-                SCLogWarning(SC_ERR_DNS_CONFIG, "no DNS UDP config found, "
+                SCLogConfig("no DNS UDP config found, "
                         "enabling DNS detection on port 53.");
 #endif
                 AppLayerProtoDetectPPRegister(IPPROTO_UDP, "53", ALPROTO_DNS,
@@ -148,7 +148,7 @@ void RegisterDNSUDPParsers(void)
             }
         }
     } else {
-        SCLogInfo("Protocol detection and parser disabled for %s protocol.",
+        SCLogConfig("Protocol detection and parser disabled for %s protocol.",
                   proto_name);
         return;
     }
@@ -187,7 +187,7 @@ void RegisterDNSUDPParsers(void)
         DNSUDPConfigure();
 #endif
     } else {
-        SCLogInfo("Parsed disabled for %s protocol. Protocol detection"
+        SCLogConfig("Parsed disabled for %s protocol. Protocol detection"
                 "still on.", proto_name);
     }
 #ifdef UNITTESTS
index b23f20b38d163bf3be30675dcc3827745e1a94f6..0b7f3bcab3ec5a4075fceac6dc0490e08b9963fd 100644 (file)
@@ -280,16 +280,15 @@ void RegisterSMBParsers(void)
                     MIN_REC_SIZE, SMBTCPProbe, SMBTCPProbe);
             /* if we have no config, we enable the default port 445 */
             if (!have_cfg) {
-                SCLogWarning(SC_ERR_SMB_CONFIG, "no SMB TCP config found, "
-                                                "enabling SMB detection on "
-                                                "port 445.");
+                SCLogConfig("no SMB TCP config found, enabling SMB detection "
+                            "on port 445.");
                 AppLayerProtoDetectPPRegister(IPPROTO_TCP, "445", ALPROTO_SMB, 0,
                         MIN_REC_SIZE, STREAM_TOSERVER, SMBTCPProbe,
                         SMBTCPProbe);
             }
         }
     } else {
-        SCLogInfo("Protocol detection and parser disabled for %s protocol.",
+        SCLogConfig("Protocol detection and parser disabled for %s protocol.",
                   proto_name);
         return;
     }
@@ -346,7 +345,7 @@ void RegisterSMBParsers(void)
 
         AppLayerParserSetStreamDepth(IPPROTO_TCP, ALPROTO_SMB, stream_depth);
     } else {
-        SCLogInfo("Parsed disabled for %s protocol. Protocol detection"
+        SCLogConfig("Parsed disabled for %s protocol. Protocol detection"
                   "still on.", proto_name);
     }
 #ifdef UNITTESTS
index a6573d568006b533890ae36459e65853d1bd1e1e..599774e196baa910e0d0136c26c6b44b0696f0ad 100644 (file)
@@ -2865,9 +2865,8 @@ void RegisterSSLParsers(void)
                                                     proto_name, ALPROTO_TLS,
                                                     0, 3,
                                                     SSLProbingParser, NULL) == 0) {
-                SCLogWarning(SC_ERR_MISSING_CONFIG_PARAM,
-                             "no TLS config found, "
-                             "enabling TLS detection on port 443.");
+                SCLogConfig("no TLS config found, "
+                            "enabling TLS detection on port 443.");
                 AppLayerProtoDetectPPRegister(IPPROTO_TCP,
                                               "443",
                                               ALPROTO_TLS,
@@ -2877,7 +2876,7 @@ void RegisterSSLParsers(void)
             }
         }
     } else {
-        SCLogInfo("Protocol detection and parser disabled for %s protocol",
+        SCLogConfig("Protocol detection and parser disabled for %s protocol",
                   proto_name);
         return;
     }
@@ -2970,7 +2969,7 @@ void RegisterSSLParsers(void)
 #endif
 
     } else {
-        SCLogInfo("Parsed disabled for %s protocol. Protocol detection"
+        SCLogConfig("Parsed disabled for %s protocol. Protocol detection"
                   "still on.", proto_name);
     }