Re: LYNX-DEV FIX: debugging that URL...

[Jason Baker]

In a previous missive, Foteos Macrides wrote:
> 
> Filip M Gieszczykiewicz <address@hidden> wrote:
> >Greetings. Add the following line right before the for (..)
> >on line 37 of the WWW/Library/Implementation/HTString.c:
> >
> >    if (!p) return -1;
> >
> >Now it won't dump when it hits URL:
> >
> >http://www.safetycare.com.au/mig.html
> >
> >and the likes of it.
> 
>       That function is called from a zillion places, and should never
> be called without valid pointers.  If you block the crash there, you'll
> just mask the real programming error.

Just a thought - what about putting something like (pardon possible
C errors, I've been doing Oracle PL/SQL too long):

# Catch function calls without valid pointer
if (!p)
{
   # Display error so the programming bug doesn't go unseen
   /* insert debug message call here */

   # Prevent ourselves from crashing
   return -1;
}

?

>                               Fote

Jason

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
 address@hidden                    | To err is human; to really
 Systems Administrator, Information Systems    | bugger things up requires
 BC Family Maintenance Enforcement Program     | the root password.

 print unpack("u","92G5S\=\"!A;F]T:&5R(\'!E<FP\@:&%C:V5R\"address@hidden");
                 PGP key available from keyservers.
;
; To UNSUBSCRIBE:  Send a mail message to address@hidden
;                  with "unsubscribe lynx-dev" (without the
;                  quotation marks) on a line by itself.
;