To LUGNET HomepageTo LUGNET News HomepageTo LUGNET Guide Homepage
 Help on Searching
 
Post new message to lugnet.off-topic.debateOpen lugnet.off-topic.debate in your NNTP NewsreaderTo LUGNET News Traffic PageSign In (Members)
 Off-Topic / Debate / 9148
9147  |  9149
Subject: 
Re: Not at all a pact with the devil...
Newsgroups: 
lugnet.off-topic.debate
Date: 
Tue, 30 Jan 2001 01:47:32 GMT
Viewed: 
1921 times
  
In lugnet.off-topic.debate, Todd Lehman writes:
> In lugnet.off-topic.debate, Kevin Loch writes:
> > Of course that should be:
> > 
> > fixURL(char *buf) {
> >   int urlsize;
> > 
> > urlsize=strlen(buf);
> > if ((buf[urlsize]-1] != '/') && (urlsize < BUFSIZE-1)) {
> >   buf[urlsize]='/';
> >   buf[urlsize+1]=0;
> > }
> 
> Might wanna throw in a comment at the top stating that behavior is undefined
> (and may even result in a segmentation fault) if fixURL(buf) is called when
> *buf == "\0".  As written above, code assumes that strlen(buf) >= 1.

d'Oh.  Shame on me.  I mean when *buf == '\0'.  :-)

--Todd



Message is in Reply To:
  Re: Not at all a pact with the devil...
 
(...) Might wanna throw in a comment at the top stating that behavior is undefined (and may even result in a segmentation fault) if fixURL(buf) is called when *buf == "\0". As written above, code assumes that strlen(buf) >= 1. --Todd (26 years ago, 29-Jan-01, to lugnet.off-topic.debate)

45 Messages in This Thread:













Entire Thread on One Page:
Nested:  All | Brief | Compact | Dots
Linear:  All | Brief | Compact
    

©2005 LUGNET. All rights reserved. - hosted by steinbruch.info GbR