diff options
author | Jason Nader <jason@kayoway.com> | 2020-01-18 18:33:43 +0900 |
---|---|---|
committer | Simon Ser <contact@emersion.fr> | 2020-01-18 10:36:03 +0100 |
commit | a576bc27bf29e91ccf6659cafffeb6cd38a8d9f1 (patch) | |
tree | d6fcf025fd61e909855f84b1567cd7fc79d73aa5 /sway/commands | |
parent | commands/move: Fix crash when required args not provided (diff) | |
download | sway-a576bc27bf29e91ccf6659cafffeb6cd38a8d9f1.tar.gz sway-a576bc27bf29e91ccf6659cafffeb6cd38a8d9f1.tar.zst sway-a576bc27bf29e91ccf6659cafffeb6cd38a8d9f1.zip |
Check argc>0 before strcasecmp. Should shortcircuit
Diffstat (limited to 'sway/commands')
-rw-r--r-- | sway/commands/move.c | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/sway/commands/move.c b/sway/commands/move.c index ceeab07c..cdbad13e 100644 --- a/sway/commands/move.c +++ b/sway/commands/move.c | |||
@@ -921,19 +921,13 @@ struct cmd_results *cmd_move(int argc, char **argv) { | |||
921 | --argc; ++argv; | 921 | --argc; ++argv; |
922 | } | 922 | } |
923 | 923 | ||
924 | if (strcasecmp(argv[0], "window") == 0 || | 924 | if (argc > 0 && (strcasecmp(argv[0], "window") == 0 || |
925 | strcasecmp(argv[0], "container") == 0) { | 925 | strcasecmp(argv[0], "container") == 0)) { |
926 | --argc; | 926 | --argc; ++argv; |
927 | if (argc > 0) { | ||
928 | ++argv; | ||
929 | } | ||
930 | } | 927 | } |
931 | 928 | ||
932 | if (strcasecmp(argv[0], "to") == 0) { | 929 | if (argc > 0 && strcasecmp(argv[0], "to") == 0) { |
933 | --argc; | 930 | --argc; ++argv; |
934 | if (argc > 0) { | ||
935 | ++argv; | ||
936 | } | ||
937 | } | 931 | } |
938 | 932 | ||
939 | if (!argc) { | 933 | if (!argc) { |