]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
rust: fix clippy lints for clippy::collapsible_else_if
authorJason Ish <jason.ish@oisf.net>
Wed, 5 Oct 2022 15:00:04 +0000 (09:00 -0600)
committerVictor Julien <vjulien@oisf.net>
Mon, 24 Oct 2022 09:20:08 +0000 (11:20 +0200)
rust/src/dcerpc/dcerpc_udp.rs
rust/src/http2/detect.rs
rust/src/http2/http2.rs
rust/src/quic/parser.rs
rust/src/ssh/ssh.rs

index 1b5513eeb5ff1930f41d30d3cde2f7f389d82cae..ea82074653a81fbe8a15a0bcf58163836cbe72ed 100644 (file)
@@ -312,10 +312,8 @@ pub unsafe extern "C" fn rs_dcerpc_probe_udp(_f: *const core::Flow, direction: u
             if dir != Direction::ToServer {
                 *rdir = Direction::ToServer.into();
             }
-        } else {
-            if dir != Direction::ToClient {
-                *rdir = Direction::ToClient.into();
-            }
+        } else if dir != Direction::ToClient {
+            *rdir = Direction::ToClient.into();
         };
         return ALPROTO_DCERPC;
     }
index 58f62d1c65aa512cab0e282bf00fad00565c491f..9a3a6a57a22d48fe4a4b50682a8fa1b103f89881 100644 (file)
@@ -634,12 +634,10 @@ pub unsafe extern "C" fn rs_http2_tx_get_cookie(
             *buffer_len = value.len() as u32;
             return 1;
         }
-    } else {
-        if let Ok(value) = http2_frames_get_header_value(tx, Direction::ToClient, "set-cookie") {
-            *buffer = value.as_ptr(); //unsafe
-            *buffer_len = value.len() as u32;
-            return 1;
-        }
+    } else if let Ok(value) = http2_frames_get_header_value(tx, Direction::ToClient, "set-cookie") {
+        *buffer = value.as_ptr(); //unsafe
+        *buffer_len = value.len() as u32;
+        return 1;
     }
     return 0;
 }
@@ -717,10 +715,8 @@ fn http2_header_iscookie(direction: Direction, hname: &[u8]) -> bool {
             if s.to_lowercase() == "cookie" {
                 return true;
             }
-        } else {
-            if s.to_lowercase() == "set-cookie" {
-                return true;
-            }
+        } else if s.to_lowercase() == "set-cookie" {
+            return true;
         }
     }
     return false;
index c6ce7de6834b7443ba0fedb146db60a35d2beffc..ffc4ffae9d6532a2bb1b720324a7e19555ff4018 100644 (file)
@@ -237,13 +237,11 @@ impl HTTP2Transaction {
                         }
                     }
                 }
-            } else {
-                if !self.file_range.is_null() {
-                    if over {
-                        range::http2_range_close(self, Direction::ToClient, decompressed)
-                    } else {
-                        range::http2_range_append(self.file_range, decompressed)
-                    }
+            } else if !self.file_range.is_null() {
+                if over {
+                    range::http2_range_close(self, Direction::ToClient, decompressed)
+                } else {
+                    range::http2_range_append(self.file_range, decompressed)
                 }
             }
             let (files, flags) = self.files.get(Direction::ToClient);
@@ -348,10 +346,8 @@ impl HTTP2Transaction {
                         if self.state < HTTP2TransactionState::HTTP2StateDataClient {
                             self.state = HTTP2TransactionState::HTTP2StateDataClient;
                         }
-                    } else {
-                        if self.state < HTTP2TransactionState::HTTP2StateDataServer {
-                            self.state = HTTP2TransactionState::HTTP2StateDataServer;
-                        }
+                    } else if self.state < HTTP2TransactionState::HTTP2StateDataServer {
+                        self.state = HTTP2TransactionState::HTTP2StateDataServer;
                     }
                 }
             }
index 69d93a7ddae794c0e3ddf580e47f06ca857a858e..dfd09cb00c858cd7e2d8471a3e4f4da88252f4a3 100644 (file)
@@ -357,12 +357,10 @@ impl QuicHeader {
                 } else {
                     return Err(nom7::Err::Error(QuicError::InvalidPacket));
                 }
+            } else if let Ok(length) = u16::try_from(rest.len()) {
+                (rest, length)
             } else {
-                if let Ok(length) = u16::try_from(rest.len()) {
-                    (rest, length)
-                } else {
-                    return Err(nom7::Err::Error(QuicError::InvalidPacket));
-                }
+                return Err(nom7::Err::Error(QuicError::InvalidPacket));
             };
 
             Ok((
index 912a6ba16d709330b78f28f0c2c1e47b20a1fa61..0a36f816663a398dac72ed8e23afa9eec2420d07 100644 (file)
@@ -426,10 +426,8 @@ pub unsafe extern "C" fn rs_ssh_tx_get_alstate_progress(
         if tx.cli_hdr.flags >= SSHConnectionState::SshStateBannerDone {
             return SSHConnectionState::SshStateBannerDone as i32;
         }
-    } else {
-        if tx.srv_hdr.flags >= SSHConnectionState::SshStateBannerDone {
-            return SSHConnectionState::SshStateBannerDone as i32;
-        }
+    } else if tx.srv_hdr.flags >= SSHConnectionState::SshStateBannerDone {
+        return SSHConnectionState::SshStateBannerDone as i32;
     }
 
     return SSHConnectionState::SshStateInProgress as i32;
@@ -508,11 +506,9 @@ pub unsafe extern "C" fn rs_ssh_tx_get_log_condition( tx: *mut std::os::raw::c_v
             return true; 
         }
     }
-    else {
-        if  tx.cli_hdr.flags == SSHConnectionState::SshStateBannerDone && 
-            tx.srv_hdr.flags == SSHConnectionState::SshStateBannerDone {
-            return true;
-        }
+    else if  tx.cli_hdr.flags == SSHConnectionState::SshStateBannerDone && 
+        tx.srv_hdr.flags == SSHConnectionState::SshStateBannerDone {
+        return true;
     }
     return false;
 }