sway/commands: Handle incorrect resize unit

problem: an invalid usage of the command resize set will cause sway to crash because it doesn't check for an invalid height.

solution: validate height along with width.

fixes #8619

collect errors of all cmd_resize variations and bubble them up

Log invalid command usage

without this, the invalid command usages from the config criteria do not
get logged or printed to the user anywhere.

Author: sahinf

Diff for: sway/commands.c

@@ -295,6 +295,7 @@ list_t *execute_command(char *_exec, struct sway_seat *seat,
 			struct cmd_results *res = handler->handle(argc-1, argv+1);
 			list_add(res_list, res);
 			if (res->status == CMD_INVALID) {
+				sway_log(SWAY_ERROR, "%s", res->error);
 				free_argv(argc, argv);
 				goto cleanup;
 			}

Diff for: sway/commands/resize.c

@@ -457,7 +457,7 @@ static struct cmd_results *cmd_resize_set(int argc, char **argv) {
 		if (argc > num_consumed_args) {
 			return cmd_results_new(CMD_INVALID, "%s", usage);
 		}
-		if (width.unit == MOVEMENT_UNIT_INVALID) {
+		if (height.unit == MOVEMENT_UNIT_INVALID) {
 			return cmd_results_new(CMD_INVALID, "%s", usage);
 		}
 	}
@@ -582,17 +582,13 @@ struct cmd_results *cmd_resize(int argc, char **argv) {
 	}
 
 	if (strcasecmp(argv[0], "set") == 0) {
-		return cmd_resize_set(argc - 1, &argv[1]);
+		error = cmd_resize_set(argc - 1, &argv[1]);
 	}
-	if (strcasecmp(argv[0], "grow") == 0) {
-		return cmd_resize_adjust(argc - 1, &argv[1], 1);
+	else if (strcasecmp(argv[0], "grow") == 0) {
+		error = cmd_resize_adjust(argc - 1, &argv[1], 1);
 	}
-	if (strcasecmp(argv[0], "shrink") == 0) {
-		return cmd_resize_adjust(argc - 1, &argv[1], -1);
+	else if (strcasecmp(argv[0], "shrink") == 0) {
+		error = cmd_resize_adjust(argc - 1, &argv[1], -1);
 	}
-
-	const char usage[] = "Expected 'resize <shrink|grow> "
-		"<width|height|up|down|left|right> [<amount>] [px|ppt]'";
-
-	return cmd_results_new(CMD_INVALID, "%s", usage);
+	return error;
 }