]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
dcerpc/tcp: fix compile warning
authorVictor Julien <victor@inliniac.net>
Fri, 23 Oct 2020 05:45:50 +0000 (07:45 +0200)
committerVictor Julien <victor@inliniac.net>
Wed, 4 Nov 2020 15:30:30 +0000 (16:30 +0100)
warning: variable does not need to be mutable
    --> src/dcerpc/dcerpc.rs:1036:42
     |
1036 |                     let tx = if let Some(mut tx) = self.get_tx_by_call_id(current_call_id, core::STREAM_TOCLIENT) {
     |                                          ----^^
     |                                          |
     |                                          help: remove this `mut`
     |
     = note: `#[warn(unused_mut)]` on by default

warning: variable does not need to be mutable
    --> src/dcerpc/dcerpc.rs:1061:30
     |
1061 |                         Some(mut tx) => {
     |                              ----^^
     |                              |
     |                              help: remove this `mut`

rust/src/dcerpc/dcerpc.rs

index a14a5de2ac59c00dc695c1f4ceae608df3f6f2e5..fac3a53397d9a05dcf0baf40d3741ec3e8e0206b 100644 (file)
@@ -1033,7 +1033,7 @@ impl DCERPCState {
                     if retval == -1 {
                         return AppLayerResult::err();
                     }
-                    let tx = if let Some(mut tx) = self.get_tx_by_call_id(current_call_id, core::STREAM_TOCLIENT) {
+                    let tx = if let Some(tx) = self.get_tx_by_call_id(current_call_id, core::STREAM_TOCLIENT) {
                         tx.resp_cmd = x;
                         tx
                     } else {
@@ -1058,7 +1058,7 @@ impl DCERPCState {
                 DCERPC_TYPE_RESPONSE => {
                     let transaction = self.get_tx_by_call_id(current_call_id, core::STREAM_TOCLIENT);
                     match transaction {
-                        Some(mut tx) => {
+                        Some(tx) => {
                             tx.resp_cmd = x;
                         }
                         None => {