]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
sdp: use rust join
authorPhilippe Antoine <pantoine@oisf.net>
Mon, 28 Apr 2025 08:44:09 +0000 (10:44 +0200)
committerVictor Julien <victor@inliniac.net>
Mon, 28 Apr 2025 18:06:06 +0000 (20:06 +0200)
It is much faster as it does not do an allocation for each element

rust/src/sdp/parser.rs

index 3ab9d932ca9f3e2039e6c9da9a9b808f383fe4b0..105bc51f6c288c9ed186ee7d44f5123226d1a561 100644 (file)
@@ -464,9 +464,9 @@ fn parse_media_description(i: &[u8]) -> IResult<&[u8], MediaDescription> {
         format!("{}", port)
     };
     let mut media_str = format!("{} {} {}", &media, &port, &proto);
-    let fmt: Vec<String> = fmt.into_iter().map(String::from).collect();
-    for f in &fmt {
-        media_str = format!("{} {}", media_str, f);
+    if !fmt.is_empty() {
+        let fmt: Vec<String> = fmt.into_iter().map(String::from).collect();
+        media_str = format!("{} {}", media_str, fmt.join(" "));
     }
     Ok((
         i,