Re: lynx-dev 2.8.5dev.7 uses all CPU

[Thomas Dickey]

On Thu, Feb 07, 2002 at 07:44:06PM -0500, Philip Webb wrote:
> 020207 Thomas Dickey wrote:
> > On Thu, Feb 07, 2002 at 05:31:30PM -0200, Fr?d?ric L. W. Meunier wrote:
> >> Lynx starts using all CPU doing the following:
> >> 1- http://www.ibazar.com.br/
> >> 2- Click "Cadastro" and accept all cookies
> >> 3- Click "[accepte.gif]"
> >> The last message is "Data transfer complete."
> > I added some limit checks to avoid the infinite loop:
> 
> works when hand-edited in 2-8-5dev.6 ; BTW see nitpick below (smile).
> the document is blank w Lynx, tho'  \  reveals the HTML.

w3m also shows a blank page (this morning opera can't connect, so I can't
compare).
> 
> > --- src/GridText.c.orig     Tue Jan  1 20:55:54 2002
> > +++ src/GridText.c  Thu Feb  7 18:40:01 2002
> > @@ -5731,13 +5731,16 @@
> >          && count_line >= stop_before) {
> >             LYSetHiText(anchor_ptr, NULL, 0);
> >             break;
> > +       } else if (line_ptr2 == text->last_line) {
> > +           break;
> >         }
> >  
> >         /*
> >          * Double check that we have a line pointer, and if so, copy into
> >          * highlight text.
> >          */
> > -       if (line_ptr2) {
> > +       if (line_ptr2
> > +        && line_ptr2->size) {
> >             char *hi_string = NULL;
>         ^^^^ missing tab?
> >             int hi_offset = line_ptr2->offset;
>         ^^^^ missing tab?

There are two tabs on those lines (I think it looks right).

-- 
Thomas E. Dickey <address@hidden>
http://invisible-island.net
ftp://invisible-island.net

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