diff options
author | Jérémie Courrèges-Anglas <jca@wxcvbn.org> | 2014-03-04 03:03:13 +0100 |
---|---|---|
committer | Jérémie Courrèges-Anglas <jca@wxcvbn.org> | 2014-03-04 03:03:13 +0100 |
commit | 77176d78b15a702d47725738df01eace86ebfa07 (patch) | |
tree | 4dced8947d52a07fd0e5051adac1e261e1a2e51f | |
parent | d4884d7643e30ceaf7e9093d2fc41972971786f2 (diff) | |
download | ratpoison-77176d78b15a702d47725738df01eace86ebfa07.zip |
Kill last strcpy occurrence.
* replace_keybinding is seldom used, no need to half-optimize it
-rw-r--r-- | src/actions.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/actions.c b/src/actions.c index e2e0044..fa35ae9 100644 --- a/src/actions.c +++ b/src/actions.c @@ -596,10 +596,8 @@ add_keybinding (KeySym keysym, int state, char *cmd, rp_keymap *map) static void replace_keybinding (rp_action *key_action, char *newcmd) { - if (strlen (key_action->data) < strlen (newcmd)) - key_action->data = xrealloc (key_action->data, strlen (newcmd) + 1); - - strcpy (key_action->data, newcmd); + free (key_action->data); + key_action->data = xstrdup (newcmd); } static int |