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:
|
500 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 (24 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
|
|
|
|