Re: lynx-dev Lynx or ncurses bug ?

[Thomas Dickey]

On Sun, Jul 08, 2001 at 02:13:53PM -0300, Frédéric L. W. Meunier wrote:
> Hi. I'm not really sure, but while ncurses-5.2-20010630
> apparently fixed the random segfaults I was having with LE (a
> text editor) and Lynx - sometimes, they'd segfault by moving
> the screen with arrow keys - , I noticed that with both
> 2.8.4pre.2 (compiled with ncurses 5.0) and pre.3 a simple /
> (search) for "news" in my LYHelp.h would lock up Lynx and
> it'd start using all my CPU and RAM.
> 
> I noticed that it only happens when I'm on the first or last
> page. No 'news' not found! is returned.

( it's only obvious once someone points out the bug ;-)

--- GridText.c.orig     Sat Jul  7 21:41:23 2001
+++ GridText.c  Sun Jul  8 14:04:22 2001
@@ -7756,12 +7756,12 @@
        if (LYno_attr_strstr(line->data, target)) {
            tentative_result = count;
            break;
-       } else if (line == HTMainText->last_line) {
-           count = 0;
-           wrapped = TRUE;
        } else if (count == start_line && wrapped) {
            HTUserMsg2(STRING_NOT_FOUND, target);
            return -1;
+       } else if (line == HTMainText->last_line) {
+           count = 0;
+           wrapped = TRUE;
        }
        line = line->next;
        count++;
@@ -7799,12 +7799,12 @@
        if (LYno_attr_strstr(line->data, target)) {
            tentative_result = count;
            break;
-       } else if (line == FirstHTLine(HTMainText)) {
-           count = line_num_in_text(HTMainText, LastHTLine(HTMainText)) + 1;
-           wrapped = TRUE;
        } else if (count == start_line && wrapped) {
            HTUserMsg2(STRING_NOT_FOUND, target);
            return -1;
+       } else if (line == FirstHTLine(HTMainText)) {
+           count = line_num_in_text(HTMainText, LastHTLine(HTMainText)) + 1;
+           wrapped = TRUE;
        }
        line = line->prev;
        count--;

-- 
Thomas E. Dickey <address@hidden>
http://dickey.his.com
ftp://dickey.his.com

; To UNSUBSCRIBE: Send "unsubscribe lynx-dev" to address@hidden