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

[Philip Webb]

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.

> --- 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?

-- 
========================,,============================================
SUPPORT     ___________//___,  Philip Webb : address@hidden
ELECTRIC   /] [] [] [] [] []|  Centre for Urban & Community Studies
TRANSIT    `-O----------O---'  University of Toronto

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