]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
rust: fix clippy warning manual_contains 13012/head 13030/head
authorPhilippe Antoine <pantoine@oisf.net>
Thu, 3 Apr 2025 11:43:29 +0000 (13:43 +0200)
committerPhilippe Antoine <pantoine@oisf.net>
Fri, 11 Apr 2025 19:06:50 +0000 (21:06 +0200)
warning: using `contains()` instead of `iter().any()` is more efficient
   --> src/http2/http2.rs:267:20
    |
267 |                 if block.value.iter().any(|&x| x == b'@') {
    |                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `block.value.contains(&b'@')`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_contains
    = note: `#[warn(clippy::manual_contains)]` on by default

(cherry picked from commit 0f3932afb78680c8af167f2089f52e0342431b1f)

rust/src/http2/http2.rs

index eace93ccc36f1a72826c32fff28a0efdf6d590f4..20b7cd94747ae12461841e8bf143eaddc658a739 100644 (file)
@@ -212,7 +212,7 @@ impl HTTP2Transaction {
                 self.decoder.http2_encoding_fromvec(&block.value, dir);
             } else if block.name.eq_ignore_ascii_case(b":authority") {
                 authority = Some(&block.value);
-                if block.value.iter().any(|&x| x == b'@') {
+                if block.value.contains(&b'@') {
                     // it is forbidden by RFC 9113 to have userinfo in this field
                     // when in HTTP1 we can have user:password@domain.com
                     self.set_event(HTTP2Event::UserinfoInUri);