- Introduce static helper `_truncate_datetime_suffix()` to safely trim datetime strings, removing unwanted suffixes like timezone offsets
- Replace manual string management with auto_gchar and g_strdup for safer, clearer ownership and to prevent leaks
- Add safety checks and logging warnings for unexpected datetime string lengths or null pointers
- Refactor _mam_rsm_id_handler to use the helper function and updated string handling
- Change log_database_get_previous_chat parameters for consistent ownership semantics, avoiding double frees and mem leaks
- Overall improve stability and prevent memory leaks during log database queries

@@ -589,3 +589,2 @@
+chatwin_db_history(ProfChatWin* chatwin, const gchar* start_time, const gchar* end_time, gboolean flip)
 {
-    if (!end_time) {
-        end_time = buffer_size(((ProfWin*)chatwin)->layout->buffer) == 0 ? NULL : g_date_time_format_iso8601(buffer_get_entry(((ProfWin*)chatwin)->layout->buffer, 0)->time);

@@ -716,3 +717,1 @@
-            GDateTime* now = g_date_time_new_now_local();
-            gchar* end_date = g_date_time_format_iso8601(now);
-            g_date_time_unref(now);
+            auto_gchar gchar* end_date = g_date_time_format_iso8601(now);

@@ -2754,1 +2754,4 @@
 
+// Trim the last 3 characters from the datetime string to remove unwanted suffix (timezone offset)
+static void
+_truncate_datetime_suffix(gchar* str)

@@ -645,3 +645,3 @@
         int max_sessions = accounts_get_max_sessions(account_name);
         if (max_sessions > 0) {
-            const gchar* cur_resource = accounts_get_resource(account_name);
+            auto_gchar gchar* cur_resource = accounts_get_resource(account_name);