]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
rust: simply matches with unwrap_or_default
authorJason Ish <jason.ish@oisf.net>
Thu, 20 Jun 2024 16:17:27 +0000 (10:17 -0600)
committerVictor Julien <victor@inliniac.net>
Fri, 21 Jun 2024 08:21:24 +0000 (10:21 +0200)
New default clippy warning:
https://rust-lang.github.io/rust-clippy/master/index.html#manual_unwrap_or_default

rust/src/smb/dcerpc.rs
rust/src/smb/smb1.rs

index b4c5749eac46bd8542aa84dd44280ed30d3f28bc..de6b8def73b3742f769fa2bd167428b26cf33ce9 100644 (file)
@@ -455,10 +455,7 @@ pub fn smb_read_dcerpc_record(state: &mut SMBState,
     // msg_id 0 as this data crosses cmd/reply pairs
     let ehdr = SMBHashKeyHdrGuid::new(SMBCommonHdr::new(SMBHDR_TYPE_TRANS_FRAG,
             hdr.ssn_id, hdr.tree_id, 0_u64), guid.to_vec());
-    let mut prevdata = match state.ssnguid2vec_map.remove(&ehdr) {
-        Some(s) => s,
-        None => Vec::new(),
-    };
+    let mut prevdata = state.ssnguid2vec_map.remove(&ehdr).unwrap_or_default();
     SCLogDebug!("indata {} prevdata {}", indata.len(), prevdata.len());
     prevdata.extend_from_slice(indata);
     let data = prevdata;
index 3a04c72bd42afa415507204dfa74eb7bbcc6b901..eb2712391150a113effaaca51232928d27ef74c8 100644 (file)
@@ -894,11 +894,8 @@ pub fn smb1_trans_response_record(state: &mut SMBState, r: &SmbRecord)
             SCLogDebug!("TRANS response {:?}", rd);
 
             // see if we have a stored fid
-            let fid = match state.ssn2vec_map.remove(
-                    &SMBCommonHdr::from1(r, SMBHDR_TYPE_GUID)) {
-                Some(f) => f,
-                None => Vec::new(),
-            };
+            let fid = state.ssn2vec_map.remove(
+                    &SMBCommonHdr::from1(r, SMBHDR_TYPE_GUID)).unwrap_or_default();
             SCLogDebug!("FID {:?}", fid);
 
             let mut frankenfid = fid.to_vec();