When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.

Having a 'case 256:' in a switch (<uchar>) {} is bad on principle
and makes gcc unhappy.

In-line the single use #define, eliminate the 256: case and remove
a now unneeded local variable.

ok guenther@

Accommodate gcc's suggestion that assignments used as truth values
should be enclosed in parentheses.

ok deraadt@

Add some {}'s to eliminate "ambiguous else" warnings from gcc.

ok tb@ millert@

Remove NULL-check before free().

since stdlib.h is in scope, don't cast.... you know the drill.
no sneakiness detected by krw

use arc4random_uniform(); ok djm
plus some other markings about signal races...

deregister. no binary change.

remove SNMP code from mrouted(8) because it is broken and it is
higly unlike that anyone will use this.

ok otto@ millert@ henning@

Replace log() with logit() since ISO C reserves log() for the math
library and gcc 3.x will complain. Replacement name taken from NetBSD.
espie@ OK.

sprintf killing; millert ok

spaces

ansi, from cloder

Manual cleanup of remaining userland __P use (excluding packages maintained outside the tree)

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

recieve -> receive

Wall cleanup

from netbsd; update to mrouted 3.8

branches: 1.1.1;
Initial revision

Having a 'case 256:' in a switch (<uchar>) {} is bad on principle
and makes gcc unhappy.

In-line the single use #define, eliminate the 256: case and remove
a now unneeded local variable.

ok guenther@

Accommodate gcc's suggestion that assignments used as truth values
should be enclosed in parentheses.

ok deraadt@

Add some {}'s to eliminate "ambiguous else" warnings from gcc.

ok tb@ millert@

Remove NULL-check before free().

since stdlib.h is in scope, don't cast.... you know the drill.
no sneakiness detected by krw

use arc4random_uniform(); ok djm
plus some other markings about signal races...

deregister. no binary change.

remove SNMP code from mrouted(8) because it is broken and it is
higly unlike that anyone will use this.

ok otto@ millert@ henning@

Replace log() with logit() since ISO C reserves log() for the math
library and gcc 3.x will complain. Replacement name taken from NetBSD.
espie@ OK.

sprintf killing; millert ok

spaces

ansi, from cloder

Manual cleanup of remaining userland __P use (excluding packages maintained outside the tree)

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

recieve -> receive

Wall cleanup

from netbsd; update to mrouted 3.8

branches: 1.1.1;
Initial revision