Fix crashes

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "reoserv"
-version = "1.15.0"
+version = "1.16.0"
 authors = ["Richard Leek <richard@richardleek.com>"]
 edition = "2024"
 description = "The rust powered endless online server emulator"
@@ -47,9 +47,10 @@ evalexpr = "13.1"
 duration-str = "0.21"
 version-compare = "0.2"
 mail-send = "0.5"
-tokio-tungstenite = "0.28"
+tokio-tungstenite = "0.29"
 hex = "0.4"
-rusqlite = { version = "0.38.0", features = ["bundled"] }
+rusqlite = { version = "0.39", features = ["bundled"] }
+regex = "1.12.3"
 
 [target.'cfg(windows)'.build-dependencies]
 winres = "0.1"

config/Config.toml

@@ -232,6 +232,8 @@ max_items = 250
 
 [character]
 
+min_name_length = 3
+
 # Should be between 12 and 14
 max_name_length = 12
 

src/map/command.rs

@@ -407,7 +407,6 @@ pub enum Command {
         player_id: i32,
         npc_index: i32,
     },
-    Save,
     SaveAsync {
         respond_to: oneshot::Sender<()>,
     },

src/map/map.rs

@@ -574,8 +574,6 @@ impl Map {
                 npc_index,
             } => self.reset_character(player_id, npc_index),
 
-            Command::Save => self.save().await,
-
             Command::SaveAsync { respond_to } => {
                 self.save().await;
                 let _ = respond_to.send(());

src/map/map/board/create_board_post.rs

@@ -1,7 +1,7 @@
 use crate::{
     SETTINGS,
     db::{DbHandle, insert_params},
-    utils::get_board_tile_spec,
+    utils::{get_board_tile_spec, truncate_to_chars},
 };
 use chrono::{Duration, Utc};
 
@@ -34,14 +34,14 @@ impl Map {
             return self.open_board(player_id, board_id);
         }
 
-        let subject = if subject.len() > SETTINGS.board.max_subject_length as usize {
-            String::from(&subject[..SETTINGS.board.max_subject_length as usize])
+        let subject = if subject.chars().count() > SETTINGS.board.max_subject_length as usize {
+            truncate_to_chars(&subject, SETTINGS.board.max_subject_length as usize)
         } else {
             subject
         };
 
-        let body = if body.len() > SETTINGS.board.max_post_length as usize {
-            String::from(&body[..SETTINGS.board.max_post_length as usize])
+        let body = if body.chars().count() > SETTINGS.board.max_post_length as usize {
+            truncate_to_chars(&body, SETTINGS.board.max_post_length as usize)
         } else {
             body
         };

src/map/map/character/set_character_property.rs

@@ -6,7 +6,7 @@ use eolib::protocol::{
     },
 };
 
-use crate::{CLASS_DB, SETTINGS};
+use crate::{CLASS_DB, SETTINGS, utils::validate_character_name};
 
 use super::super::Map;
 
@@ -20,24 +20,27 @@ impl Map {
 
             match property.as_str() {
                 "title" => {
-                    if (1..={ SETTINGS.character.max_title_length }).contains(&value.len()) {
+                    let title_length = value.chars().count();
+                    if title_length > 0 && title_length <= SETTINGS.character.max_title_length {
                         character.title = Some(value);
                     } else {
                         character.title = None;
                     }
                     false
                 }
                 "fiance" => {
-                    if (1..={ SETTINGS.character.max_name_length }).contains(&value.len()) {
-                        character.fiance = Some(value);
+                    let name = value.to_lowercase();
+                    if validate_character_name(&name) {
+                        character.fiance = Some(name);
                     } else {
                         character.fiance = None;
                     }
                     false
                 }
                 "partner" => {
-                    if (1..={ SETTINGS.character.max_name_length }).contains(&value.len()) {
-                        character.partner = Some(value);
+                    let name = value.to_lowercase();
+                    if validate_character_name(&name) {
+                        character.partner = Some(name);
                     } else {
                         character.partner = None;
                     }