]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
rust/mqtt: fix clippy 1.77 warning 10721/head 10724/head
authorPhilippe Antoine <pantoine@oisf.net>
Thu, 21 Mar 2024 21:45:41 +0000 (22:45 +0100)
committerPhilippe Antoine <pantoine@oisf.net>
Mon, 25 Mar 2024 20:02:21 +0000 (21:02 +0100)
error: creating a mutable reference to mutable static is discouraged
   --> src/mqtt/mqtt.rs:752:23
    |
752 |     let max_msg_len = &mut MAX_MSG_LEN;
    |                       ^^^^^^^^^^^^^^^^ mutable reference to mutable static
    |
    = note: for more information, see issue #114447 <https://github.com/rust-lang/rust/issues/114447>
    = note: this will be a hard error in the 2024 edition
    = note: this mutable reference has lifetime `'static`, but if the static gets accessed (read or written) by any other means, or any other reference is created, then any further use of this mutable reference is Undefined Behavior

rust/src/mqtt/mqtt.rs

index 8260251218bd0f22a2502b1192b0fb20a56428fe..35f3c6d459a8a9d7d6d7d4f75a589aadc0598924 100644 (file)
@@ -749,8 +749,7 @@ export_state_data_get!(rs_mqtt_get_state_data, MQTTState);
 #[no_mangle]
 pub unsafe extern "C" fn rs_mqtt_register_parser(cfg_max_msg_len: u32) {
     let default_port = CString::new("[1883]").unwrap();
-    let max_msg_len = &mut MAX_MSG_LEN;
-    *max_msg_len = cfg_max_msg_len;
+    MAX_MSG_LEN = cfg_max_msg_len;
     let parser = RustParser {
         name: PARSER_NAME.as_ptr() as *const std::os::raw::c_char,
         default_port: default_port.as_ptr(),