- From: Rick Kwan <kenobi@lightsaber.com>
- Date: Fri, 25 Jun 1999 10:46:35 -0700 (PDT)
- To: Raffaele Sena <raff@nuvomedia.com>
- cc: libwww <www-lib@w3.org>
On Fri, 25 Jun 1999, Raffaele Sena wrote: > > > Raffaele, > > > > I'm able to make the browser crash on a certain number of > > pages with poorly formed links. (nested links, extra </a> > > with no matching <a>.) > > > well, the code currently checked in doesn't have any call to > LMHText_beginAnchor > (unless I didn't see it), so that's why I commented out the call to > LMHText_endAnchor. It's not obvious. LMText_beginAnchor() is currently called as a registered callback. Registration takes place in HTBrowse.c. Take a look at the stack trace from my previous message and you'll see how it gets called. > with the two call in place I agree that you'll get a crash on pages that > starts with </A> > and that you'll get wrong information for extra </A>. > > maybe you could add a text->current_anchor that is set to > text->last_anchor for an <A> > and to NULL for a </A> and use that for anchor management. That will probably make for better bookkeeping. I will look into it. > -- Raffaele > --Rick Kwan
Received on Friday, 25 June 1999 13:20:13 UTC