dragonfly.git
16 years agoWhile removing a memory leak, rev 1.32 introduced a
Matthew Dillon [Fri, 7 Jan 2005 20:30:21 +0000 (20:30 +0000)]
While removing a memory leak, rev 1.32 introduced a
free-memory-and-reuse-it-after. Correct both problems and
make rcp -r work again under /etc/malloc.conf -> AJ.

Submitted-by: Xin LI <delphij@frontfree.net>
Taken-From: FreeBSD/1.37 charnier
PR: FreeBSD:bin/54969

16 years agoCorrect two bugs that may result in incorrect CBCP response for
Matthew Dillon [Fri, 7 Jan 2005 20:19:54 +0000 (20:19 +0000)]
Correct two bugs that may result in incorrect CBCP response for
administrator specified number.

The implementation in our version transmits extra zero byte that is a
violation of the Microsoft PPP Callback Control Protocol section 3.2[1].
This can lead to incorrect behavior on certain devices, as reported
in bin/50613.

Confirmed with: cbcp.c in ppp's cvs, rev. 1.4 and 1.15 [2]

Submitted-by: Xin LI <delphij@frontfree.net>
Taken-From: FreeBSD/rev1.7
Submitted-to-FreeBSD-by: Dmitry Pryanishnikov <dmitry at atlantis.dp.ua>
PR: FreeBSD:bin/50613

16 years agoMerge of FreeBSD rev. 1.36+1.37 of ip_nat.c. Conditionalize declarations
Matthew Dillon [Fri, 7 Jan 2005 20:15:47 +0000 (20:15 +0000)]
Merge ... FreeBSD rev. 1.36+1.37 of ip_nat.c.  Conditionalize declarations
properly when LARGE_NAT is not defined.

Submitted-by: Xin LI <delphij@frontfree.net>
16 years agoChange variable name from retstr to ret. Reduce diffs between FreeBSD
Max Okumoto [Fri, 7 Jan 2005 11:46:31 +0000 (11:46 +0000)]
Change variable name from retstr to ret. Reduce diffs between FreeBSD

16 years agoCatch up to recent rtlookup() changes.
Jeffrey Hsu [Fri, 7 Jan 2005 05:42:59 +0000 (05:42 +0000)]
Catch up to recent rtlookup() changes.

Submitted by: Tim Wickberg <me@k9mach3.org>

16 years agoRaise WARNS to 6:
Chris Pressey [Fri, 7 Jan 2005 02:43:41 +0000 (02:43 +0000)]
Raise WARNS to 6:
- Constify a string variable;
- Parentheses around an assignment in a test.

And style(9):
- Remove casts to void when ignoring return values;
- Remove `register' keywords;
- Make check against '\0' explicit.

16 years agoRaise WARNS to 6:
Chris Pressey [Fri, 7 Jan 2005 02:32:50 +0000 (02:32 +0000)]
Raise WARNS to 6:
- Make an extern declaration global.

And style(9):
- Remove `register' keywords;
- Braces around multiline blocks;
- Remove casts to void when ignoring return values;
- Make tests against NULL and 0 explicit;
- Untangle a particularly convoluted conditional printf().

16 years agoAdd missing prototype that I missed with spliting up nonints.h
Max Okumoto [Thu, 6 Jan 2005 23:29:09 +0000 (23:29 +0000)]
Add missing prototype that I missed with spliting up nonints.h

16 years agoClarify/clean up code, make fewer assumptions about types:
Chris Pressey [Thu, 6 Jan 2005 22:37:46 +0000 (22:37 +0000)]
Clarify/clean up code, make fewer assumptions about types:
- Store pids/gids/uids/devs in a union instead of casting to a long;
- Don't use the nonspecific LT_GENERIC list type;
- Add a more specific LT_PPID list type for parent pids;
- Use NODEV to indicate the lack of a specified device;
- Make the makelist() function more straightforward:
  - use only one switch statement instead of two cascading ones;
  - introduce a helper function parse_pid() to parse pids;
- No functional changes.

16 years agoFix a kernel crash that occurs when the SMB protocol stack is used. The
Matthew Dillon [Thu, 6 Jan 2005 22:31:16 +0000 (22:31 +0000)]
Fix a kernel crash that occurs when the SMB protocol stack is used.  The
SMB protocol stack is still hacked to create a kernel thread with a
process structure attached to it (when we really ought to be using a pure
kernel thread... this is the only subsystem that still doesn't).  However,
the SMB code was not properly cleaning the process structure up on exit
resulting in a corrupt process a list.  A 'ps' equivalent occuring after
this situation occurs can crash the kernel.

Reported-by: Rumko <rumcic@gmail.com>
16 years agoenomem() is only used in one file, make it static.
Max Okumoto [Thu, 6 Jan 2005 21:06:25 +0000 (21:06 +0000)]
enomem() is only used in one file, make it static.

16 years agoRemoved unused routines:
Max Okumoto [Thu, 6 Jan 2005 21:04:20 +0000 (21:04 +0000)]
Removed unused routines:
Buf_GetByte()
Buf_GetBytes()
Buf_UngetByte()
Buf_UngetBytes()

16 years agoStyle(9):
Chris Pressey [Thu, 6 Jan 2005 21:00:33 +0000 (21:00 +0000)]
Style(9):
- braces around multiline blocks
- no blank line at start of function
- remove extraneous semicolon.

16 years agoNow that I understand the poorly written BSD routing code and what
Jeffrey Hsu [Thu, 6 Jan 2005 17:59:32 +0000 (17:59 +0000)]
Now that I understand the poorly written BSD routing code and what
it was trying to do, rewrite it in a clear and concise manner.

The old rtalloc1() code written by CSRG had a number of problems:
  1. it was not clear which route was being returned
  2. it was not clear what was being reported
  3. it hid the essential radix tree lookup operation inside a series of
      conditional tests and inline assignments
  4. it had multiple gotos to the inside of if statements
  5. it intermixed reporting code with the operational logic of lookup
      and cloning
  6. it assigns multiple times to key variables
  7. it has unnecessary assignments to key variables
  8. it overloaded the "report" argument parameter, to have two
      different semantics
  9. it misnamed the key route lookup function "rtalloc1", obscuring all uses
      of route lookup.

In contrast to the rtalloc1 code in FreeBSD 4 or the even more convoluted
rtalloc1 code in FreeBSD 5, the DragonFlyBSD version
  A. has a clear control flow that makes the common case obvious
      by highlighting the core call to the radix tree look up function,
      eliminating gotos into if statements,
      and completely separating out the special-case cloning logic
  B. makes it clear which route is being returned
      by only assigning once to the key "rt" variable
      and by expliciting returning "rt" or "clonedroute"
  C. abstracts out the reporting code into its own reporting API
  D. cleans up the semantics of the "report" argument parameter to
      only indicate whether to report a miss and not whether to clone
  E. introduces a simple single-argument API for caller that want to clone
      and those that do not.

16 years agoFix compile error.
Jeffrey Hsu [Thu, 6 Jan 2005 17:54:35 +0000 (17:54 +0000)]
Fix compile error.

16 years agoMove a global variable into local scope for MP safety.
Jeffrey Hsu [Thu, 6 Jan 2005 17:38:26 +0000 (17:38 +0000)]
Move a global variable into local scope for MP safety.

16 years agostyle(9) cleanup.
Joerg Sonnenberger [Thu, 6 Jan 2005 17:34:25 +0000 (17:34 +0000)]
style(9) cleanup.

16 years agoAdd _DIAGASSERT macro for library internal usage. This is not active by
Joerg Sonnenberger [Thu, 6 Jan 2005 17:32:44 +0000 (17:32 +0000)]
Add _DIAGASSERT macro for library internal usage. This is not active by
default.

Obtained-from: NetBSD

16 years ago- Fix a comment typo.
Sascha Wildner [Thu, 6 Jan 2005 14:54:57 +0000 (14:54 +0000)]
- Fix a comment typo.

- Fix bogus ${FOO:Mbar} tests where the actual intent is to check
  if the result set is empty.  While here, replace non-bogus empty
  string comparisons with equivalent empty() checks.
  [FreeBSD rev. 1.68]

16 years agoHiding pointer in typedefs is evil. 'Buffer *' -> 'Buffer'
Max Okumoto [Thu, 6 Jan 2005 13:18:58 +0000 (13:18 +0000)]
Hiding pointer in typedefs is evil. 'Buffer *' -> 'Buffer'

16 years agoAdd missing header files to CVS, for check in on 2005/01/06 10:52:59.
Max Okumoto [Thu, 6 Jan 2005 11:41:47 +0000 (11:41 +0000)]
Add missing header files to CVS, for check in on 2005/01/06 10:52:59.

16 years agoMerge the remaining lst routines into a single file, and remove
Max Okumoto [Thu, 6 Jan 2005 11:35:27 +0000 (11:35 +0000)]
Merge the remaining lst routines into a single file, and remove
the single function files.

16 years agoClean up include header mess. Split nonints into separate header files.
Max Okumoto [Thu, 6 Jan 2005 10:53:00 +0000 (10:53 +0000)]
Clean up include header mess. Split nonints into separate header files.
Headers now only include other headers so they can compile.  Source
files include all required headers.

16 years agoCosmetic cleanups.
Jeffrey Hsu [Thu, 6 Jan 2005 09:14:13 +0000 (09:14 +0000)]
Cosmetic cleanups.

16 years agoAdd icmp6 alias.
Jeroen Ruigrok/asmodai [Thu, 6 Jan 2005 08:43:35 +0000 (08:43 +0000)]
Add icmp6 alias.

Submitted by: Jake Maciejewski

16 years agoGet rid of the register keyword.
Jeroen Ruigrok/asmodai [Thu, 6 Jan 2005 08:33:11 +0000 (08:33 +0000)]
Get rid of the register keyword.

Submitted by: Steve Mynott

16 years agoMake sure the temporary .c file generated from the .y file is properly
Matthew Dillon [Thu, 6 Jan 2005 05:23:45 +0000 (05:23 +0000)]
Make sure the temporary .c file generated from the .y file is properly
removed by the make clean target.

16 years agoRaise WARNS of newfs to 6:
Chris Pressey [Thu, 6 Jan 2005 03:21:00 +0000 (03:21 +0000)]
Raise WARNS of newfs to 6:
- Use static char buffer instead of string constant;
- Move and add some function prototypes;
- Don't reuse a scratch variable, use a real pid_t to hold the
  result of a fork();
- Cast variables as needed (mainly to u_long and u_int32) to avoid
  signed-unsigned comparison warnings;
- Rename some variables to avoid name clashes;
- Mark some parameters as unused;
- Add MOPT_NULL as a struct mntopt constant for terminating a list.

And apply some style(9):
- Move out some loop invariant code, for clarity;
- Remove `register' keywords;
- Make explicit some checks against NULL.

Based-on-patch-by: Peter Schuller
                   <peter_dot_schuller_at_infidyne_dot_com>
Tested-by: typical usage of installer
16 years agoReplace perl code with a sed script.
Sascha Wildner [Thu, 6 Jan 2005 01:20:01 +0000 (01:20 +0000)]
Replace perl code with a sed script.

16 years agoRemoved unintentional added macro.
Max Okumoto [Thu, 6 Jan 2005 00:24:37 +0000 (00:24 +0000)]
Removed unintentional added macro.

16 years agoFix compilation for the USE_KQUEUE case.
Max Okumoto [Wed, 5 Jan 2005 23:32:25 +0000 (23:32 +0000)]
Fix compilation for the USE_KQUEUE case.
Submitted by:   Emil Mikulic <emikulic@dmr.ath.cx>

Taken-from: FreeBSD
Author: harti

16 years agomaxJobs is declared extern in job.h so it cannot be static in job.c.
Max Okumoto [Wed, 5 Jan 2005 23:30:35 +0000 (23:30 +0000)]
maxJobs is declared extern in job.h so it cannot be static in job.c.
Submitted by: Andreas Jochens; Robert Millan <rmh@debian.org>
PR: bin/75210

Taken-from: FreeBSD
Author: harti

16 years agoRemove cleanup functions. There is no reason to free memory
Max Okumoto [Wed, 5 Jan 2005 23:28:20 +0000 (23:28 +0000)]
Remove cleanup functions. There is no reason to free memory
just before exiting (especially given the number of memory leaks) -
it just costs time.

Taken-from: FreeBSD
Author: harti

16 years agoReduce diff between FreeBSD. kqueue code doesn't crash the kernel.
Max Okumoto [Wed, 5 Jan 2005 22:21:46 +0000 (22:21 +0000)]
Reduce diff between FreeBSD.  kqueue code doesn't crash the kernel.

16 years agoUnhook Perl from build.
Joerg Sonnenberger [Wed, 5 Jan 2005 22:03:51 +0000 (22:03 +0000)]
Unhook Perl from build.

16 years agoMove VarREError() to different position in var.c to reduce diffs from FReeBSD.
Max Okumoto [Wed, 5 Jan 2005 20:10:40 +0000 (20:10 +0000)]
Move VarREError() to different position in var.c to reduce diffs from FReeBSD.

16 years ago- Remove both sys/wait.h & sys/types.h - unnecessary
Liam J. Foy [Wed, 5 Jan 2005 18:42:33 +0000 (18:42 +0000)]
- Remove both sys/wait.h & sys/types.h - unnecessary
- Remove registers
- Static functions & Constify arguments

16 years agoMove sa_X macros under _KERNEL protection for now.
Joerg Sonnenberger [Wed, 5 Jan 2005 18:21:20 +0000 (18:21 +0000)]
Move sa_X macros under _KERNEL protection for now.

16 years ago- UNIX conformance: If -r -f on non-existent directory, don't emit error.
Liam J. Foy [Wed, 5 Jan 2005 16:24:19 +0000 (16:24 +0000)]
- UNIX conformance: If -r -f on non-existent directory, don't emit error.
  (FreeBSD, 1.52)
- Constify

Ok'ed by: Joerg@

16 years agoMark binutils-2.14 as dead.
Joerg Sonnenberger [Wed, 5 Jan 2005 15:33:11 +0000 (15:33 +0000)]
Mark binutils-2.14 as dead.

16 years agoWARNS=6 safeness:
Joerg Sonnenberger [Wed, 5 Jan 2005 15:26:05 +0000 (15:26 +0000)]
WARNS=6 safeness:
- GC myname, not used
- constify a lot of string references
- remove done(), use atexit instead
- fix the signal handler, it must not call fclose and can't use exit therefore
- rename local variables which collidate with globals

16 years agoMerge FreeBSD rev 1.70:
Joerg Sonnenberger [Wed, 5 Jan 2005 15:14:04 +0000 (15:14 +0000)]
Merge FreeBSD rev 1.70:
When specifying an interface to ifconfig, first look up the interface
index, then retrieve statistics for that index, rather than retrieving
all interfaces and then looking for a matching name.  This allows the
user to refer to an interface via an alias name.

16 years agoAdd ISO C99's _Exit, which is identical to _exit.
Joerg Sonnenberger [Wed, 5 Jan 2005 11:51:18 +0000 (11:51 +0000)]
Add ISO C99's _Exit, which is identical to _exit.

16 years agoRaise WARNS to 6:
Chris Pressey [Wed, 5 Jan 2005 02:40:23 +0000 (02:40 +0000)]
Raise WARNS to 6:
- Constify some strings;
- Remove `register' and/or add `int' to variable declarations
  as appropriate.

16 years agoRaise WARNS to 6.
Chris Pressey [Wed, 5 Jan 2005 02:33:25 +0000 (02:33 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6.
Chris Pressey [Wed, 5 Jan 2005 02:30:23 +0000 (02:30 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6:
Chris Pressey [Wed, 5 Jan 2005 00:46:52 +0000 (00:46 +0000)]
Raise WARNS to 6:
- Mark a parameter as unused.

16 years agoRaise WARNS to 6.
Chris Pressey [Wed, 5 Jan 2005 00:45:31 +0000 (00:45 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6.
Chris Pressey [Wed, 5 Jan 2005 00:38:38 +0000 (00:38 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6:
Chris Pressey [Wed, 5 Jan 2005 00:34:36 +0000 (00:34 +0000)]
Raise WARNS to 6:
- Constify a string.

16 years agoRaise WARNS to 6:
Chris Pressey [Wed, 5 Jan 2005 00:33:50 +0000 (00:33 +0000)]
Raise WARNS to 6:
- Constify the typecast in a macro.

16 years agoRaise WARNS to 6:
Chris Pressey [Wed, 5 Jan 2005 00:32:50 +0000 (00:32 +0000)]
Raise WARNS to 6:
- Initialize a variable before use.

16 years agoWARNS 2->6 and a minor code readability cleanup.
Matthew Dillon [Tue, 4 Jan 2005 23:08:13 +0000 (23:08 +0000)]
WARNS 2->6 and a minor code readability cleanup.

Submitted-by: Jeandre du Toit <jeandre.dutoit@gmail.com>
16 years agoDon't install old GDB documentation.
Joerg Sonnenberger [Tue, 4 Jan 2005 22:25:26 +0000 (22:25 +0000)]
Don't install old GDB documentation.

16 years agoRIP
Joerg Sonnenberger [Tue, 4 Jan 2005 20:00:34 +0000 (20:00 +0000)]
RIP

16 years agoMerge from vendor branch DHCP:
Joerg Sonnenberger [Tue, 4 Jan 2005 19:58:53 +0000 (19:58 +0000)]
Merge from vendor branch DHCP:
Update isc-dhcp to 3.0.2rc3 using patch infrastructure.
Also include the server and relay, but currently without rcNG script.

Submitted-by: Simon 'corecode' Schubert <corecode@fs.ei.tum.de>
16 years agoUpdate isc-dhcp to 3.0.2rc3 using patch infrastructure.
Joerg Sonnenberger [Tue, 4 Jan 2005 19:58:54 +0000 (19:58 +0000)]
Update isc-dhcp to 3.0.2rc3 using patch infrastructure.
Also include the server and relay, but currently without rcNG script.

Submitted-by: Simon 'corecode' Schubert <corecode@fs.ei.tum.de>
16 years agoUpdate isc-dhcp to 3.0.2rc3 using patch infrastructure. vendor/DHCP
Joerg Sonnenberger [Tue, 4 Jan 2005 19:58:53 +0000 (19:58 +0000)]
Update isc-dhcp to 3.0.2rc3 using patch infrastructure.
Also include the server and relay, but currently without rcNG script.

Submitted-by: Simon 'corecode' Schubert <corecode@fs.ei.tum.de>
16 years agoSwitch from binutils 2.14 to binutils 2.15.
Joerg Sonnenberger [Tue, 4 Jan 2005 19:14:18 +0000 (19:14 +0000)]
Switch from binutils 2.14 to binutils 2.15.

16 years agoReplace sockstat(1) Perl script with the C version from FreeBSD 5.
Joerg Sonnenberger [Tue, 4 Jan 2005 18:56:58 +0000 (18:56 +0000)]
Replace sockstat(1) Perl script with the C version from FreeBSD 5.
This doesn't use the binary scan to determine the buffer size, but
calls the sysctl directly twice.

More work is needed to separate the lists again, but this can be
done in tree.

16 years agoMark old file as dead.
Joerg Sonnenberger [Tue, 4 Jan 2005 18:46:32 +0000 (18:46 +0000)]
Mark old file as dead.

16 years agoDon't build the gdb-related documents in binutils-2.15.
Joerg Sonnenberger [Tue, 4 Jan 2005 14:28:06 +0000 (14:28 +0000)]
Don't build the gdb-related documents in binutils-2.15.

16 years agoStyle(9) cleanup:
Chris Pressey [Tue, 4 Jan 2005 05:45:02 +0000 (05:45 +0000)]
Style(9) cleanup:
- Add braces to multi-line blocks;
- Remove casts to void when ignoring return values;
- Explicitly compare a pointer against NULL;
- Remove a blank line at the start of a function;
- Raise WARNS to 6.

Based-on-patch-submitted-by: Dion Blazakis
                             <dion_dot_blazakis_at_gmail_dot_com>

16 years agoStyle(9): remove `register' keywords.
Chris Pressey [Tue, 4 Jan 2005 05:27:47 +0000 (05:27 +0000)]
Style(9): remove `register' keywords.

Submitted-by: Steve Mynott <steve_dot_mynott_at_gmail_dot_com>
16 years agoClarify a run-on sentence by splitting it into two. snprintf() and
Chris Pressey [Tue, 4 Jan 2005 00:00:52 +0000 (00:00 +0000)]
Clarify a run-on sentence by splitting it into two.  snprintf() and
vsnprintf() can return negative values, at least in theory, so don't
make it sound like they can't.

16 years agoIgnore ENOENT when fetching the interface MIB. This can happen for
Joerg Sonnenberger [Mon, 3 Jan 2005 23:47:20 +0000 (23:47 +0000)]
Ignore ENOENT when fetching the interface MIB. This can happen for
sparse lists.

16 years agoCheck that the ifnet_addrs entry is actually used before dereferencing it.
Joerg Sonnenberger [Mon, 3 Jan 2005 23:25:25 +0000 (23:25 +0000)]
Check that the ifnet_addrs entry is actually used before dereferencing it.
Also protect the check with splnet() to avoid further race conditions.

16 years agoSet ip6_v6only to true by default. The administrators who want to use
Jeffrey Hsu [Mon, 3 Jan 2005 22:11:51 +0000 (22:11 +0000)]
Set ip6_v6only to true by default.  The administrators who want to use
IPv4-mapped IPv6 address should tweak the sysctl manually with knowledge
of the security concerns.

Obtained from KAME (itojun).
Reference:
    ftp://ftp.itojun.org/pub/paper/draft-itojun-v6ops-v4mapped-harmful-01.txt
Same as NetBSD rev 1.55 by briggs, FreeBSD rev 1.18 by ume.
Submitted by Hiroki Sato <hrs@allbsd.org>.

16 years agoCorrect a byte-order bug with fragment header scanning.
Jeffrey Hsu [Mon, 3 Jan 2005 22:03:26 +0000 (22:03 +0000)]
Correct a byte-order bug with fragment header scanning.

Obtained from KAME rev 1.331 by suz.
Same as FreeBSD rev 1.54 by ume.
Submitted by Hiroki Sato <hrs@allbsd.org>

16 years ago- Static functions/variables
Liam J. Foy [Mon, 3 Jan 2005 17:35:57 +0000 (17:35 +0000)]
- Static functions/variables
- Remove unused prototype
- Remove unused variable
- int -> pid_t
- Make clear what the 'outfile' is
- Silence GCC/Make WARNS 6 clean
- Constify

Makefile:
- Add WARNS 6
- Add $DragonFly$ tag

16 years agoAdd matching line for ATI Radeon RV280 9200SE.
Jeroen Ruigrok/asmodai [Mon, 3 Jan 2005 06:45:23 +0000 (06:45 +0000)]
Add matching line for ATI Radeon RV280 9200SE.

Submitted by: Gary Tivey [back in November]

16 years agoRaise WARNS to 6.
Chris Pressey [Mon, 3 Jan 2005 01:44:56 +0000 (01:44 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6.
Chris Pressey [Mon, 3 Jan 2005 01:11:53 +0000 (01:11 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6:
Chris Pressey [Mon, 3 Jan 2005 01:06:24 +0000 (01:06 +0000)]
Raise WARNS to 6:
- Constify a parameter.

While here, remove register keywords & make local functions static.

16 years agoRaise WARNS to 6.
Chris Pressey [Sun, 2 Jan 2005 22:47:48 +0000 (22:47 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6.
Chris Pressey [Sun, 2 Jan 2005 22:14:13 +0000 (22:14 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6:
Chris Pressey [Sun, 2 Jan 2005 02:40:20 +0000 (02:40 +0000)]
Raise WARNS to 6:
- Remove unused variables.
- Mark a parameter as unused.

Based-on-patch-submitted-by: Peter Schuller
                             <peter_dot_schuller_at_infidyne_dot_com>

16 years agoRaise WARNS to 6.
Chris Pressey [Sun, 2 Jan 2005 01:45:20 +0000 (01:45 +0000)]
Raise WARNS to 6.

Submitted-by: Peter Schuller <peter_dot_schuller_at_infidyne_dot_com>
16 years agoRaise WARNS to 6.
Chris Pressey [Sun, 2 Jan 2005 01:40:45 +0000 (01:40 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6:
Chris Pressey [Sun, 2 Jan 2005 01:35:13 +0000 (01:35 +0000)]
Raise WARNS to 6:
- Initialize an uninitialized variable.

16 years agoRaise WARNS to 6:
Chris Pressey [Sun, 2 Jan 2005 01:22:49 +0000 (01:22 +0000)]
Raise WARNS to 6:
- Un-nest a nested extern declaration.

16 years agoRaise WARNS to 6.
Chris Pressey [Sun, 2 Jan 2005 01:18:57 +0000 (01:18 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6.
Chris Pressey [Sun, 2 Jan 2005 01:13:20 +0000 (01:13 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6:
Chris Pressey [Sat, 1 Jan 2005 23:09:57 +0000 (23:09 +0000)]
Raise WARNS to 6:
- Constify a parameter;
- Remove unnecessary casts to u_int.

16 years agoRaise WARNS to 6:
Chris Pressey [Sat, 1 Jan 2005 23:02:42 +0000 (23:02 +0000)]
Raise WARNS to 6:
- Use a static character buffer instead of a string constant.

16 years agoRaise WARNS to 6:
Chris Pressey [Sat, 1 Jan 2005 22:06:25 +0000 (22:06 +0000)]
Raise WARNS to 6:
- Constify some strings;
- Make a temporary copy of a const string for parsing purposes.

16 years agoApply style(9):
Chris Pressey [Sat, 1 Jan 2005 22:00:15 +0000 (22:00 +0000)]
Apply style(9):
- Remove casts to void when ignoring return values;
- Use ANSI style function definitions;
- Use STD{IN,OUT,ERR}_FILENO constants for readability;
- Consistently use parentheses after sizeof.

Also raise WARNS to 6:
- Use a static string instead of a string constant;
- Mark a parameter as unused.

16 years agoThere is no such thing as WARNS=7 (yet,) so use WARNS=6.
Chris Pressey [Sat, 1 Jan 2005 21:50:23 +0000 (21:50 +0000)]
There is no such thing as WARNS=7 (yet,) so use WARNS=6.

Pointed-out-by: liamfoy
16 years agoRaise WARNS to 6.
Chris Pressey [Sat, 1 Jan 2005 21:28:27 +0000 (21:28 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6.
Chris Pressey [Sat, 1 Jan 2005 21:24:07 +0000 (21:24 +0000)]
Raise WARNS to 6.

16 years agoRaise WARNS to 6:
Chris Pressey [Sat, 1 Jan 2005 21:10:43 +0000 (21:10 +0000)]
Raise WARNS to 6:
- Constify some strings.

16 years agoRaise WARNS to 6.
Chris Pressey [Sat, 1 Jan 2005 21:07:39 +0000 (21:07 +0000)]
Raise WARNS to 6.

16 years ago- Remove duplicate headers
Liam J. Foy [Sat, 1 Jan 2005 15:56:56 +0000 (15:56 +0000)]
- Remove duplicate headers

16 years agoSlap 2005 into the copyright. Happy New Year!
Hiten Pandya [Sat, 1 Jan 2005 11:15:44 +0000 (11:15 +0000)]
Slap 2005 into the copyright.  Happy New Year!

16 years agoSlap 2005 into the COPYRIGHT. Happy New Year!
Hiten Pandya [Sat, 1 Jan 2005 11:08:57 +0000 (11:08 +0000)]
Slap 2005 into the COPYRIGHT.  Happy New Year!

16 years agoFix double-free problem when sysctl net.inet.ip.rtexpire=0.
Jeffrey Hsu [Sat, 1 Jan 2005 09:20:05 +0000 (09:20 +0000)]
Fix double-free problem when sysctl net.inet.ip.rtexpire=0.

16 years agoFix double-free problem when sysctl net.inet.ip.rtexpire=0.
Jeffrey Hsu [Sat, 1 Jan 2005 09:19:40 +0000 (09:19 +0000)]
Fix double-free problem when sysctl net.inet.ip.rtexpire=0.

Obtained from:  KAME rev 1.24 by SUZUKI Shinsuke <suz@kame.net>
Equivalent to:  FreeBSD rev 1.1.2.4 by SUZUKI Shinsuke <suz@kame.net>
Submitted by:  Hiroki Sato <hrs@allbsd.org>

16 years agoStyle(9): the return type of strcmp() is not a boolean, so don't
Chris Pressey [Sat, 1 Jan 2005 01:36:02 +0000 (01:36 +0000)]
Style(9): the return type of strcmp() is not a boolean, so don't
treat it as one, explcitly compare it against zero instead.

16 years agoRaise WARNS to 6:
Chris Pressey [Sat, 1 Jan 2005 00:22:33 +0000 (00:22 +0000)]
Raise WARNS to 6:
- Include missing header files.

16 years agoUse real getopt() handling instead of the hand-rolled and IOCCC-worthy
Chris Pressey [Sat, 1 Jan 2005 00:13:49 +0000 (00:13 +0000)]
Use real getopt() handling instead of the hand-rolled and IOCCC-worthy
"Micro getopt()" macros.  In the process, clean up the option-handling
code:
- Sort the options in the switch statement;
- Plug piddling memory leaks when processing repeated options by
  freeing strings before allocating them for a second time;
- Die with a fatal error if the requested report file cannot be
  opened for appending;
- Don't call init() before usage() (to prevent the usage message
  being mangled by changes to the terminal settings;)
- Raise WARNS to 6.