zhangyiss/editline
1
0
Fork 0
mirror of https://github.com/troglobit/editline.git synced 2025-05-05 20:11:12 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix negative array index triggered by i-search

Browse Source 
This is a fix to a bug reported by Claus Fischer.  When doing a reversed
i-search on the history, clear_line() sets rl_point to a negative value,
which is later used by ceol().  The intention is to clear the whole line
for the new Search: prompt, but indexing an array with a negative value
is strongly discouraged.

To prevent this negative indexing from happening we check in ceol() for
rl_point < 0, but we quickly notice another side effect: one lingering
character at the end of line when returning from i-search.  The problem
manifests itself more cleary when having a looooo000ooonong prompt.

The fix is to rub out the extra space, insert the i-search string, and
*then* call ceol() again.

Thanks to Claus for pointing this out!

Signed-off-by: Joachim Nilsson <troglobit@gmail.com>
This commit is contained in:
Joachim Nilsson 2017-11-29 14:53:14 +01:00
parent 91937d970d
commit 2a3b422137
1 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/editline.c
View File

@ -431,11 +431,17 @@ static el_status_t case_cap_word(void)
static void ceol(void)
{
int extras;
int extras = 0;
int i;
char *p;
for (extras = 0, i = rl_point, p = &rl_line_buffer[i]; i <= rl_end; i++, p++) {
while (rl_point < 0) {
tty_put(' ');
rl_point++;
extras++;
}
for (i = rl_point, p = &rl_line_buffer[i]; i <= rl_end; i++, p++) {
tty_put(' ');
if (ISMETA(*p)) {
if (rl_meta_chars) {
@ -443,7 +449,7 @@ static void ceol(void)
tty_put(' ');
extras += 2;
}
} if (ISCTL(*p)) {
} else if (ISCTL(*p)) {
tty_put(' ');
extras++;
}
@ -525,15 +531,19 @@ static const char *prev_hist(void)
static el_status_t do_insert_hist(const char *p)
{
el_status_t ret;
if (p == NULL)
return el_ring_bell();
rl_point = 0;
reposition();
ceol();
rl_end = 0;
return insert_string(p);
ret = insert_string(p);
ceol();
return ret;
}
static el_status_t do_hist(const char *(*move)(void))