Re: lynx-dev valgrind

[Thomas Dickey]

On Thu, Feb 06, 2003 at 02:47:18PM +0300, Leonid Pauzner wrote:
> 4-Feb-2003 01:26 I wrote:
> > 2-Feb-2003 18:08 Thomas Dickey wrote:
> >> After adding LP's fixes, I see this problem when testing with valgrind
> >> and the no-leaks configuration.  This is if I do something like visit the
> >> help page, then another, then the help page again, so it is linked more 
> >> than
> >> once in memory.
> 
> >> ==19625== Invalid read of size 4
> >> ==19625==    at 0x8110DB6: deleteLinks 
> >> (../../../WWW/Library/Implementation/HTAnchor.c:526)
> 
> > Yes, I saw this either (crash under gdb).
> > Apparently, this is due to
> >>>++ HTAnchor_link now links only foreign sources -LP
> > - destinations and sources became unbalanced
> > (I haven't thought it is so critical).
> 
> Oh!  This was an error in HTAnchor_delete vs deleteLinks mutual recursion
> logic (from early lynx days, objects were freed at wrong time - too late).
> 
> Now fixed.

ok (will test)
 
> > I made a patch against your dev13b snapshot,
> > it finishes my play around HTAnchor.c/h.
> 
> 
>  htanchor.c |  158 
> +++++++++++++++++++++++++++----------------------------------
>  htlist.c   |    8 ++-
>  2 files changed, 77 insertions, 89 deletions
> 



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

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