config/input: validate xkb keymap before storing

This allows for an optional validation stage when storing an input
config. Currently, only the xkb keymap is validated. If storing the
delta input config will result in any invalid xkb keymaps, the input
config will not be stored and error will be populated with the first
line of the xkbcommon log.

1 files changed, 4 insertions, 7 deletions

diff --git a/sway/commands/input.c b/sway/commands/input.c
index 23a6644f..d95c4baf 100644
--- a/sway/commands/input.c
+++ b/sway/commands/input.c
@@ -88,9 +88,10 @@ struct cmd_results *cmd_input(int argc, char **argv) {
 
         if (!res || res->status == CMD_SUCCESS) {
                 char *error = NULL;
-                struct xkb_keymap *keymap = sway_keyboard_compile_keymap(
-                                config->handler_context.input_config, &error);
-                if (!keymap) {
+                struct input_config *ic =
+                        store_input_config(config->handler_context.input_config, &error);
+                if (!ic) {
+                        free_input_config(config->handler_context.input_config);
                         if (res) {
                                 free_cmd_results(res);
                         }
@@ -99,10 +100,6 @@ struct cmd_results *cmd_input(int argc, char **argv) {
                         free(error);
                         return res;
                 }
-                xkb_keymap_unref(keymap);
-
-                struct input_config *ic =
-                        store_input_config(config->handler_context.input_config);
 
                 input_manager_apply_input_config(ic);
                 retranslate_keysyms(ic);