devs/cproof
1
2
Fork 1
Code Issues 43 Pull Requests 9 Actions Packages Projects 1 Releases Wiki Activity

Check for sane argument in autoconnect set

Browse Source 
Check that user doesnt do `/autoconnect set   `.
Which would result in: `Account '(null)' does not exist`.
This commit is contained in:
Michael Vetter
2019-06-05 14:41:09 +02:00
parent 53ef209272
commit 4e972f8aef
1 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/command/cmd_funcs.c
View File

@@ -6555,11 +6555,15 @@ cmd_autoconnect(ProfWin *window, const char *const command, gchar **args)
prefs_set_string(PREF_CONNECT_ACCOUNT, NULL); prefs_set_string(PREF_CONNECT_ACCOUNT, NULL);
cons_show("Autoconnect account disabled."); cons_show("Autoconnect account disabled.");
} else if (strcmp(args[0], "set") == 0) { } else if (strcmp(args[0], "set") == 0) {
if (accounts_account_exists(args[1])) { if (args[1] == NULL || strlen(args[1]) == 0) {
prefs_set_string(PREF_CONNECT_ACCOUNT, args[1]); cons_bad_cmd_usage(command);
cons_show("Autoconnect account set to: %s.", args[1]);
} else { } else {
cons_show_error("Account '%s' does not exist.", args[1]); if (accounts_account_exists(args[1])) {
prefs_set_string(PREF_CONNECT_ACCOUNT, args[1]);
cons_show("Autoconnect account set to: %s.", args[1]);
} else {
cons_show_error("Account '%s' does not exist.", args[1]);
}
} }
} else { } else {
cons_bad_cmd_usage(command); cons_bad_cmd_usage(command);