From: Victor Julien <victor@inliniac.net>
Date: Wed, 14 Mar 2018 12:16:38 +0000 (+0100)
Subject: smb2: don't log/track each READ/WRITE/etc
X-Git-Tag: suricata-4.1.0-beta1~58
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=32b19fac99d4602b392c03118e98a67ad3cf98e3;p=thirdparty%2Fsuricata.git

smb2: don't log/track each READ/WRITE/etc
---

diff --git a/rust/src/smb/smb1.rs b/rust/src/smb/smb1.rs
index df704551cd..6b355511f4 100644
--- a/rust/src/smb/smb1.rs
+++ b/rust/src/smb/smb1.rs
@@ -121,8 +121,8 @@ pub fn smb1_command_string(c: u8) -> String {
 
 // later we'll use this to determine if we need to
 // track a ssn per type
-pub fn smb1_create_new_tx(_cmd: u8) -> bool {
-    match _cmd {
+pub fn smb1_create_new_tx(cmd: u8) -> bool {
+    match cmd {
         SMB1_COMMAND_READ_ANDX |
         SMB1_COMMAND_WRITE_ANDX |
         SMB1_COMMAND_TRANS |
diff --git a/rust/src/smb/smb2.rs b/rust/src/smb/smb2.rs
index 9da07ba57c..86a575fa39 100644
--- a/rust/src/smb/smb2.rs
+++ b/rust/src/smb/smb2.rs
@@ -90,8 +90,14 @@ pub fn smb2_dialect_string(d: u16) -> String {
 
 // later we'll use this to determine if we need to
 // track a ssn per type
-fn smb2_create_new_tx(_cmd: u16) -> bool {
-    true
+fn smb2_create_new_tx(cmd: u16) -> bool {
+    match cmd {
+        SMB2_COMMAND_READ |
+        SMB2_COMMAND_WRITE |
+        SMB2_COMMAND_GET_INFO |
+        SMB2_COMMAND_SET_INFO => { false },
+        _ => { true },
+    }
 }
 
 fn smb2_read_response_record_generic<'b>(state: &mut SMBState, r: &Smb2Record<'b>)