src-test2/libexec/ftpd, branch release/5.3.0_cvs

This commit was manufactured by cvs2svn to create tag

2004-11-04T19:12:42Z

'RELENG_5_3_0_RELEASE'. This commit was manufactured to restore the state of the 5.3-RELEASE image.

ftpd(8) seems to be WARNS=2 clean now.

2004-07-31T15:07:33Z

Tested on: i386, ia64, amd64, sparc64, alpha

Change ``(foo *)0'' to ``NULL'' where it's possible

2004-07-31T15:03:17Z

(and it appears possible throughout ftpd(8) source.) It is not a mere issue of style: Null pointers in C seem to have been mistaken one way or another quite often.

Kill a small herd of casts to off_t where they were not needed.

2004-07-31T14:46:41Z

Thank Fortune, the C compiler can figure out by itself the proper conversion for assignments, comparisons, and prototyped function arguments.

Printf(3) off_t values through conversion to intmax_t since

2004-07-31T14:22:02Z

we've got et al now. (This makes ftpd(8) WARNS=2 clean.)

Kill an unused variable (heading to WARNS=2.)

2004-07-31T14:03:59Z

Convert a couple of bogus null statements to the right form.

2004-07-31T14:03:14Z

(Heading to WARNS=2.)

Ditto for (gid_t).

2004-07-30T17:30:07Z

Kill casts to (uid_t) obviously left from the K&R era.

2004-07-30T17:27:23Z

Prototyping library functions in header files has rendered them superfluous.

Add a comment to explain that the loop around the call to bind(2)

2004-07-30T17:18:57Z

is not a hack, but it has a clear purpose.