aboutsummaryrefslogtreecommitdiff
path: root/main/src/ui
diff options
context:
space:
mode:
authorAearil <45526730+aearil@users.noreply.github.com>2019-04-29 14:24:29 +0000
committerfiaxh <fiaxh@users.noreply.github.com>2019-04-29 08:24:29 -0600
commitf4778ef3e66bda41831251c486e497075cb82d66 (patch)
treea513072185031bdb436265542f587fda5ba92370 /main/src/ui
parent97c59cb8cda49335ce464dd1bb1c508bc6cc7e2a (diff)
downloaddino-f4778ef3e66bda41831251c486e497075cb82d66.tar.gz
dino-f4778ef3e66bda41831251c486e497075cb82d66.zip
Fix when /affiliate cmd doesn't get enough args (#557)
Added array bounds checking to /affiliate command parsing. Maybe some user feedback when using the different commands would be useful.
Diffstat (limited to 'main/src/ui')
-rw-r--r--main/src/ui/chat_input/view.vala8
1 files changed, 6 insertions, 2 deletions
diff --git a/main/src/ui/chat_input/view.vala b/main/src/ui/chat_input/view.vala
index a33e8b53..ddf1b3c0 100644
--- a/main/src/ui/chat_input/view.vala
+++ b/main/src/ui/chat_input/view.vala
@@ -112,8 +112,12 @@ public class View : Box {
stream_interactor.get_module(MucManager.IDENTITY).kick(conversation.account, conversation.counterpart, token[1]);
return;
case "/affiliate":
- string[] user_role = token[1].split(" ", 2);
- stream_interactor.get_module(MucManager.IDENTITY).change_affiliation(conversation.account, conversation.counterpart, user_role[0].strip(), user_role[1].strip());
+ if (token.length > 1) {
+ string[] user_role = token[1].split(" ", 2);
+ if (user_role.length == 2) {
+ stream_interactor.get_module(MucManager.IDENTITY).change_affiliation(conversation.account, conversation.counterpart, user_role[0].strip(), user_role[1].strip());
+ }
+ }
return;
case "/nick":
stream_interactor.get_module(MucManager.IDENTITY).change_nick(conversation.account, conversation.counterpart, token[1]);