Merge pull request #1270 from tobiasblass/bindsym_use_after_free

fix use-after-free bug
This commit is contained in:
Drew DeVault 2017-07-10 19:00:12 -04:00 committed by GitHub
commit 8fad213f34
1 changed files with 3 additions and 2 deletions

View File

@ -61,10 +61,11 @@ struct cmd_results *cmd_bindsym(int argc, char **argv) {
sym = ((char *)split->items[i])[strlen("button")] - '1' + M_LEFT_CLICK;
}
if (!sym) {
struct cmd_results *ret = cmd_results_new(CMD_INVALID, "bindsym",
"Unknown key '%s'", (char *)split->items[i]);
free_sway_binding(binding);
free_flat_list(split);
return cmd_results_new(CMD_INVALID, "bindsym", "Unknown key '%s'",
(char *)split->items[i]);
return ret;
}
xkb_keysym_t *key = malloc(sizeof(xkb_keysym_t));
if (!key) {