[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
- Re: lynx-dev another largish patch, (continued)
- Re: lynx-dev another largish patch, Leonid Pauzner, 1999/10/26
- Re: lynx-dev another largish patch, Klaus Weide, 1999/10/26
- more TRST support (was: Re: lynx-dev another largish patch), Leonid Pauzner, 1999/10/26
- Re: more TRST support (was: Re: lynx-dev another largish patch), Klaus Weide, 1999/10/26
- Re: more TRST support (was: Re: lynx-dev another largish patch), Leonid Pauzner, 1999/10/26
- Re: more TRST support (was: Re: lynx-dev another largish patch), Klaus Weide, 1999/10/27
- Re: more TRST support (was: Re: lynx-dev another largish patch), Leonid Pauzner, 1999/10/27
Re: lynx-dev another largish patch, rjp, 1999/10/27
Re: lynx-dev another largish patch, Kim DeVaughn, 1999/10/27
- lynx-dev rowspan core dump should be solved (was: another largish patch),
Klaus Weide <=
Re: lynx-dev another largish patch, Kim DeVaughn, 1999/10/28
Re: lynx-dev another largish patch, Henry Nelson, 1999/10/27