Correctly parse "0/0" as the default route when specifying
the classless-[ms-]static-routes options in dhcpd.conf.

Brings dhcpd.conf CIDR parsing in line with dhclient.conf,
dhcp-options(5) and the command lines of various utilities.

Inconsistency noted and fix tested by Freda Bundchen. Thanks!

Fix dhcpd build on gcc architectures.

OK deraadt@ jca@

Switch domain-search parsing from hand rolled hex string to
human readable list of strings.

Slightly tweaked version of a diff from William Ahern via tech@.

Long silence on all fronts leads to community based testing.

Add 'echo-client-id' statment, so that RFC 6842 behaviour can be
turned off for those clients and networks that find it impossible to
move past RFC 2131. Modelled on the same statement in recent ISC
versions, though we default to 'on' (a.k.a. RFC 6842) rather that
'off' (a.k.a. RFC 2131).

Problems reported by Bastien Durel (Xerox Phaser 6022 printer) and
Bryan Vyhmeister (Hon Hai Precision router) via misc@. Thanks!

Add missing function prototypes. Mark kw_cmp() as static like all
its friends.

branches: 1.31.4;
Eliminate pointless'%m' (a.k.a. hand rolled strerror()) by using fatal() and
log_warn(). Zap a couple of explicit 'syslog()' calls.

Adjust some long lines.

Switch from old errwarn.c logging to shiny new log.[ch].

ok benno@

KNF and clean up many comments.

Ansify function definitions.

Diff from Edgar Pettijohn.

Eliminate #include inside *.h files and include only needed headers in
each *.c file.

Inspired by mention of header silliness by Edgar Pettijohn and mmcc@
on tech@.

stdlib.h is in scope; do not cast malloc/calloc/realloc*
ok millert krw

Tweak parsing so that hostnames starting with 0-9 are accepted.

Reported long ago by matthieu@. Also Jacob Berkman via the lists.

Tests and suggestions from Jacob and Matthieu.

Fix classless-{ms-,}static-routes to comply RFC 3442. Number of
octets should be changed by corresponding to the prefix length. And 0
should be allowed for the prefix length. Also fix white spaces.

Based on diff from Yuuichi Someya.

ok krw

Add parsing for options 121 (classless-static-routes) and 249
(classless-ms-static-routes). dhcpd can now specify and serve
these options and dhclient can recognize and use supersede, etc.
statements on them.

Based on a diff from Stefan Rinke. Thanks!

Use the provided netmask on the subnet value provided in 'subnet'
statements, allowing subsequent comparisions with interface
addresses to always work properly.

Problem noted, fix tested & ok dlg@

plug memory leaks on error paths

joint work with krw@

ok krw@, stsp@

malloc/strlcpy -> strdup. Use consistent idiom.

ok (as part of larger diff) blambert@ kettenis@ stsp@ zinovik@

Eliminate all uses of dmalloc() where the returned pointer
is checked for NULL and a specific error/warning issued. Add
two such manual warning/error checks and kill those dmalloc
calls. And then there were none, so kill dmalloc(). Whew.

Two more calloc() wrapper wrappers, new_shared_network() and
new_subnet(), bite the dust.

remove one-lease-per-client parameter which was never implemented (it
doesn't do anything and is not documented); ok henning@

typo; from sstull
ok ckuethe

cleanup keyword lookup for the lexer, and remove keywords not valid in
either dhclient or dhcpd context; tested by many

tiny bit of delinting; ok krw@

avoid aliasing with libc functions; ok henning

minor KNF, Moritz Jodeit <moritz@jodeit.org>

Avoid following a NULL pointer if a hostname lookup fails and
remove an extraneous "exiting" when there is a config file error.
OK henning@ and mcbride@

resolve hostnames at parse time, solves PR3771, ok theo

spacing

$OpenBSD$

more KNF

more knf

various knf; henning ok

spaces

time_t and kill a few dumb defines

may the whacking begin

Fix dhcpd build on gcc architectures.

OK deraadt@ jca@

Switch domain-search parsing from hand rolled hex string to
human readable list of strings.

Slightly tweaked version of a diff from William Ahern via tech@.

Long silence on all fronts leads to community based testing.

Add 'echo-client-id' statment, so that RFC 6842 behaviour can be
turned off for those clients and networks that find it impossible to
move past RFC 2131. Modelled on the same statement in recent ISC
versions, though we default to 'on' (a.k.a. RFC 6842) rather that
'off' (a.k.a. RFC 2131).

Problems reported by Bastien Durel (Xerox Phaser 6022 printer) and
Bryan Vyhmeister (Hon Hai Precision router) via misc@. Thanks!

Add missing function prototypes. Mark kw_cmp() as static like all
its friends.

branches: 1.31.4;
Eliminate pointless'%m' (a.k.a. hand rolled strerror()) by using fatal() and
log_warn(). Zap a couple of explicit 'syslog()' calls.

Adjust some long lines.

Switch from old errwarn.c logging to shiny new log.[ch].

ok benno@

KNF and clean up many comments.

Ansify function definitions.

Diff from Edgar Pettijohn.

Eliminate #include inside *.h files and include only needed headers in
each *.c file.

Inspired by mention of header silliness by Edgar Pettijohn and mmcc@
on tech@.

stdlib.h is in scope; do not cast malloc/calloc/realloc*
ok millert krw

Tweak parsing so that hostnames starting with 0-9 are accepted.

Reported long ago by matthieu@. Also Jacob Berkman via the lists.

Tests and suggestions from Jacob and Matthieu.

Fix classless-{ms-,}static-routes to comply RFC 3442. Number of
octets should be changed by corresponding to the prefix length. And 0
should be allowed for the prefix length. Also fix white spaces.

Based on diff from Yuuichi Someya.

ok krw

Add parsing for options 121 (classless-static-routes) and 249
(classless-ms-static-routes). dhcpd can now specify and serve
these options and dhclient can recognize and use supersede, etc.
statements on them.

Based on a diff from Stefan Rinke. Thanks!

Use the provided netmask on the subnet value provided in 'subnet'
statements, allowing subsequent comparisions with interface
addresses to always work properly.

Problem noted, fix tested & ok dlg@

plug memory leaks on error paths

joint work with krw@

ok krw@, stsp@

malloc/strlcpy -> strdup. Use consistent idiom.

ok (as part of larger diff) blambert@ kettenis@ stsp@ zinovik@

Eliminate all uses of dmalloc() where the returned pointer
is checked for NULL and a specific error/warning issued. Add
two such manual warning/error checks and kill those dmalloc
calls. And then there were none, so kill dmalloc(). Whew.

Two more calloc() wrapper wrappers, new_shared_network() and
new_subnet(), bite the dust.

remove one-lease-per-client parameter which was never implemented (it
doesn't do anything and is not documented); ok henning@

typo; from sstull
ok ckuethe

cleanup keyword lookup for the lexer, and remove keywords not valid in
either dhclient or dhcpd context; tested by many

tiny bit of delinting; ok krw@

avoid aliasing with libc functions; ok henning

minor KNF, Moritz Jodeit <moritz@jodeit.org>

Avoid following a NULL pointer if a hostname lookup fails and
remove an extraneous "exiting" when there is a config file error.
OK henning@ and mcbride@

resolve hostnames at parse time, solves PR3771, ok theo

spacing

$OpenBSD$

more KNF

more knf

various knf; henning ok

spaces

time_t and kill a few dumb defines

may the whacking begin

Switch domain-search parsing from hand rolled hex string to
human readable list of strings.

Slightly tweaked version of a diff from William Ahern via tech@.

Long silence on all fronts leads to community based testing.

Add 'echo-client-id' statment, so that RFC 6842 behaviour can be
turned off for those clients and networks that find it impossible to
move past RFC 2131. Modelled on the same statement in recent ISC
versions, though we default to 'on' (a.k.a. RFC 6842) rather that
'off' (a.k.a. RFC 2131).

Problems reported by Bastien Durel (Xerox Phaser 6022 printer) and
Bryan Vyhmeister (Hon Hai Precision router) via misc@. Thanks!

Add missing function prototypes. Mark kw_cmp() as static like all
its friends.

branches: 1.31.4;
Eliminate pointless'%m' (a.k.a. hand rolled strerror()) by using fatal() and
log_warn(). Zap a couple of explicit 'syslog()' calls.

Adjust some long lines.

Switch from old errwarn.c logging to shiny new log.[ch].

ok benno@

KNF and clean up many comments.

Ansify function definitions.

Diff from Edgar Pettijohn.

Eliminate #include inside *.h files and include only needed headers in
each *.c file.

Inspired by mention of header silliness by Edgar Pettijohn and mmcc@
on tech@.

stdlib.h is in scope; do not cast malloc/calloc/realloc*
ok millert krw

Tweak parsing so that hostnames starting with 0-9 are accepted.

Reported long ago by matthieu@. Also Jacob Berkman via the lists.

Tests and suggestions from Jacob and Matthieu.

Fix classless-{ms-,}static-routes to comply RFC 3442. Number of
octets should be changed by corresponding to the prefix length. And 0
should be allowed for the prefix length. Also fix white spaces.

Based on diff from Yuuichi Someya.

ok krw

Add parsing for options 121 (classless-static-routes) and 249
(classless-ms-static-routes). dhcpd can now specify and serve
these options and dhclient can recognize and use supersede, etc.
statements on them.

Based on a diff from Stefan Rinke. Thanks!

Use the provided netmask on the subnet value provided in 'subnet'
statements, allowing subsequent comparisions with interface
addresses to always work properly.

Problem noted, fix tested & ok dlg@

plug memory leaks on error paths

joint work with krw@

ok krw@, stsp@

malloc/strlcpy -> strdup. Use consistent idiom.

ok (as part of larger diff) blambert@ kettenis@ stsp@ zinovik@

Eliminate all uses of dmalloc() where the returned pointer
is checked for NULL and a specific error/warning issued. Add
two such manual warning/error checks and kill those dmalloc
calls. And then there were none, so kill dmalloc(). Whew.

Two more calloc() wrapper wrappers, new_shared_network() and
new_subnet(), bite the dust.

remove one-lease-per-client parameter which was never implemented (it
doesn't do anything and is not documented); ok henning@

typo; from sstull
ok ckuethe

cleanup keyword lookup for the lexer, and remove keywords not valid in
either dhclient or dhcpd context; tested by many

tiny bit of delinting; ok krw@

avoid aliasing with libc functions; ok henning

minor KNF, Moritz Jodeit <moritz@jodeit.org>

Avoid following a NULL pointer if a hostname lookup fails and
remove an extraneous "exiting" when there is a config file error.
OK henning@ and mcbride@

resolve hostnames at parse time, solves PR3771, ok theo

spacing

$OpenBSD$

more KNF

more knf

various knf; henning ok

spaces

time_t and kill a few dumb defines

may the whacking begin

Add 'echo-client-id' statment, so that RFC 6842 behaviour can be
turned off for those clients and networks that find it impossible to
move past RFC 2131. Modelled on the same statement in recent ISC
versions, though we default to 'on' (a.k.a. RFC 6842) rather that
'off' (a.k.a. RFC 2131).

Problems reported by Bastien Durel (Xerox Phaser 6022 printer) and
Bryan Vyhmeister (Hon Hai Precision router) via misc@. Thanks!

Add missing function prototypes. Mark kw_cmp() as static like all
its friends.

branches: 1.31.4;
Eliminate pointless'%m' (a.k.a. hand rolled strerror()) by using fatal() and
log_warn(). Zap a couple of explicit 'syslog()' calls.

Adjust some long lines.

Switch from old errwarn.c logging to shiny new log.[ch].

ok benno@

KNF and clean up many comments.

Ansify function definitions.

Diff from Edgar Pettijohn.

Eliminate #include inside *.h files and include only needed headers in
each *.c file.

Inspired by mention of header silliness by Edgar Pettijohn and mmcc@
on tech@.

stdlib.h is in scope; do not cast malloc/calloc/realloc*
ok millert krw

Tweak parsing so that hostnames starting with 0-9 are accepted.

Reported long ago by matthieu@. Also Jacob Berkman via the lists.

Tests and suggestions from Jacob and Matthieu.

Fix classless-{ms-,}static-routes to comply RFC 3442. Number of
octets should be changed by corresponding to the prefix length. And 0
should be allowed for the prefix length. Also fix white spaces.

Based on diff from Yuuichi Someya.

ok krw

Add parsing for options 121 (classless-static-routes) and 249
(classless-ms-static-routes). dhcpd can now specify and serve
these options and dhclient can recognize and use supersede, etc.
statements on them.

Based on a diff from Stefan Rinke. Thanks!

Use the provided netmask on the subnet value provided in 'subnet'
statements, allowing subsequent comparisions with interface
addresses to always work properly.

Problem noted, fix tested & ok dlg@

plug memory leaks on error paths

joint work with krw@

ok krw@, stsp@

malloc/strlcpy -> strdup. Use consistent idiom.

ok (as part of larger diff) blambert@ kettenis@ stsp@ zinovik@

Eliminate all uses of dmalloc() where the returned pointer
is checked for NULL and a specific error/warning issued. Add
two such manual warning/error checks and kill those dmalloc
calls. And then there were none, so kill dmalloc(). Whew.

Two more calloc() wrapper wrappers, new_shared_network() and
new_subnet(), bite the dust.

remove one-lease-per-client parameter which was never implemented (it
doesn't do anything and is not documented); ok henning@

typo; from sstull
ok ckuethe

cleanup keyword lookup for the lexer, and remove keywords not valid in
either dhclient or dhcpd context; tested by many

tiny bit of delinting; ok krw@

avoid aliasing with libc functions; ok henning

minor KNF, Moritz Jodeit <moritz@jodeit.org>

Avoid following a NULL pointer if a hostname lookup fails and
remove an extraneous "exiting" when there is a config file error.
OK henning@ and mcbride@

resolve hostnames at parse time, solves PR3771, ok theo

spacing

$OpenBSD$

more KNF

more knf

various knf; henning ok

spaces

time_t and kill a few dumb defines

may the whacking begin