From 7f3c92059f89ea796ff403b45458621d2d5a7213 Mon Sep 17 00:00:00 2001 From: Bart Schaefer Date: Tue, 4 Oct 2005 14:27:12 +0000 Subject: 21821, 21822: fix two infinite loops --- ChangeLog | 9 +++++++++ Src/Zle/complist.c | 10 ++++++++-- 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 82c53e098..d8482d70a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2005-10-04 Bart Schaefer + + * 21822 (annotated): Src/Zle/complist.c: prevent a long-standing + infinite loop when redrawing the listing during menu-selection + with a very narrow window. + + * 21821: Src/Zle/complist.c: fix infinite loop when multibyte + support is disabled, introduced by 21784. + 2005-10-03 Peter Stephenson * 21809: Src/utils.c: need to export nicedup() for diff --git a/Src/Zle/complist.c b/Src/Zle/complist.c index a9d57ae43..b13932e24 100644 --- a/Src/Zle/complist.c +++ b/Src/Zle/complist.c @@ -672,7 +672,7 @@ clnicezputs(Listcols colors, char *s, int ml) if (colors) initiscol(colors); - while ((cc = *s)) { + while ((cc = *s++)) { if (colors) doiscol(colors, i++); if (itok(cc)) { @@ -2153,9 +2153,15 @@ domenuselect(Hookdef dummy, Chdata dat) if (y < mlines) mline = y; } + DPUTS(mline < 0, + "BUG: mline < 0 after re-scanning mtab in domenuselect()"); while (mline < mlbeg) - if ((mlbeg -= step) < 0) + if ((mlbeg -= step) < 0) { mlbeg = 0; + /* Crude workaround for BUG above */ + if (mline < 0) + break; + } if (mlbeg && lbeg != mlbeg) { Cmatch **p = mtab + ((mlbeg - 1) * columns), **q; -- cgit 1.4.1