about summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorMichael Vetter <jubalh@iodoru.org>2020-02-25 14:55:46 +0100
committerMichael Vetter <jubalh@iodoru.org>2020-02-25 14:55:46 +0100
commit240aeac4d369df97d0e4ee54bd2924afe75258c8 (patch)
tree6abc9b49b59f84ebaec6250a00bafe4abfeedcd8
parent1a87d7be3e2265734edee9909501ed36ebd8dd42 (diff)
downloadprofani-tty-240aeac4d369df97d0e4ee54bd2924afe75258c8.tar.gz
Make cmd_logging() safer
Could be that args[1] is not set.
-rw-r--r--src/command/cmd_funcs.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/src/command/cmd_funcs.c b/src/command/cmd_funcs.c
index 93b37c30..5395a53e 100644
--- a/src/command/cmd_funcs.c
+++ b/src/command/cmd_funcs.c
@@ -6714,7 +6714,8 @@ cmd_logging(ProfWin *window, const char *const command, gchar **args)
         return TRUE;
     }
 
-    if (strcmp(args[0], "chat") == 0) {
+
+    if (strcmp(args[0], "chat") == 0 && args[1] != NULL) {
         _cmd_set_boolean_preference(args[1], command, "Chat logging", PREF_CHLOG);
 
         // if set to off, disable history
@@ -6723,7 +6724,7 @@ cmd_logging(ProfWin *window, const char *const command, gchar **args)
         }
 
         return TRUE;
-    } else if (g_strcmp0(args[0], "group") == 0) {
+    } else if (g_strcmp0(args[0], "group") == 0 && args[1] != NULL) {
         if (g_strcmp0(args[1], "on") == 0 || g_strcmp0(args[1], "off") == 0) {
             _cmd_set_boolean_preference(args[1], command, "Groupchat logging", PREF_GRLOG);
             return TRUE;