lynx-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

lynx-dev rowspan core dump should be solved (was: another largish patch)


From: Klaus Weide
Subject: lynx-dev rowspan core dump should be solved (was: another largish patch)
Date: Wed, 27 Oct 1999 19:34:24 -0500 (CDT)

On Wed, 27 Oct 1999, Kim DeVaughn wrote:

> On Tue, Oct 26, 1999, Klaus Weide (address@hidden) said:
> |
> | ... is available temporarily at
> |    <http://enteract.com/~kweide/lynx/>
> 
> Klaus ... I tried the above out (along with the followup patch for
> compiling w/o the prettysrc stuff), and got a core dump when I tried
> going to:
> 
>    Linkname: ESPN.com - NFL - National Football League
>         URL: http://espn.go.com/nfl/
> 
> I'm attaching the gdb backtrace, and the tail end of the Lynx.trace
> (if you need more of the Lynx.trace, I can email it to you).

It looks just like the problem already fixed in a later patch.
Please try the  col-tbody-etc.diff  from the same place, or
just add the two lines in TRSTable.c by hand:

@@ -1097,6 +1122,8 @@

     if (rowspan > 1) {
        Stbl_reserveCellsInTable(me, icell, colspan, rowspan);
+       /* me->rows may now have been realloced, make lastrow valid pointer */
+       lastrow = me->rows + (me->nrows - 1);
     }

     {


I loaded the ESPN url with no problem with the patches applied.


> #2  0x2f5da in FatalProblem (sig=11) at ./LYMain.c:3905
> #3  0xefbfdfdc in ?? ()
> #4  0x152e3 in HText_startStblTD (me=0x212400, colspan=1, rowspan=3, 
>     alignment=-1, isheader=0 '\000') at ./GridText.c:4573

Yep, looks like the some problem.  I think it only appeared with
a rowspan > 2, which is why I didn't run into it immediately.

I have no immediate plans for more TRST changes, so the situation
should stabilize a bit now...

   Klaus


reply via email to

[Prev in Thread] Current Thread [Next in Thread]