From 43a99e64f249c166f9636646399429457bba5c92 Mon Sep 17 00:00:00 2001 From: Paul LeoNerd Evans Date: Wed, 3 Dec 2008 20:19:15 +0000 Subject: strdup() the terminfo keypad_local and keypad_xmit strings at construct time, in case multiple instances and they change beneath us --- driver-ti.c | 38 ++++++++++++++++++++++++++++++++++---- 1 file changed, 34 insertions(+), 4 deletions(-) diff --git a/driver-ti.c b/driver-ti.c index d598db7..fe28137 100644 --- a/driver-ti.c +++ b/driver-ti.c @@ -1,3 +1,6 @@ +// we want strdup() +#define _XOPEN_SOURCE 500 + #include "termkey.h" #include "termkey-internal.h" @@ -39,6 +42,9 @@ typedef struct { termkey_t *tk; struct trie_node *root; + + char *start_string; + char *stop_string; } termkey_ti; static int funcname2keysym(const char *funcname, termkey_type *typep, termkey_keysym *symp, int *modmask, int *modsetp); @@ -189,6 +195,20 @@ static void *new_driver(termkey_t *tk, const char *term) ti->root = compress_trie(ti->root); + /* Take copies of these terminfo strings, in case we build multiple termkey + * instances for multiple different termtypes, and it's different by the + * time we want to use it + */ + if(keypad_xmit) + ti->start_string = strdup(keypad_xmit); + else + ti->start_string = NULL; + + if(keypad_local) + ti->stop_string = strdup(keypad_local); + else + ti->stop_string = NULL; + return ti; abort_free_trie: @@ -202,20 +222,24 @@ abort_free_ti: static void start_driver(termkey_t *tk, void *info) { + termkey_ti *ti = info; + /* The terminfo database will contain keys in application cursor key mode. * We may need to enable that mode */ - if(keypad_xmit) { + if(ti->start_string) { // Can't call putp or tputs because they suck and don't give us fd control - write(tk->fd, keypad_xmit, strlen(keypad_xmit)); + write(tk->fd, ti->start_string, strlen(ti->start_string)); } } static void stop_driver(termkey_t *tk, void *info) { - if(keypad_local) { + termkey_ti *ti = info; + + if(ti->stop_string) { // Can't call putp or tputs because they suck and don't give us fd control - write(tk->fd, keypad_local, strlen(keypad_local)); + write(tk->fd, ti->stop_string, strlen(ti->stop_string)); } } @@ -225,6 +249,12 @@ static void free_driver(void *info) free_trie(ti->root); + if(ti->start_string) + free(ti->start_string); + + if(ti->stop_string) + free(ti->stop_string); + free(ti); } -- cgit v1.2.3-70-g09d2