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

ok benno@

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@.

remove NULL-checks before free()

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 the

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.

show more ...

Eliminate all uses of dfree() where the pointer is either dereferenced
immediately before the use or the pointer is checked for NULL before
the call. And then there were none, so kill dfree().

Wrappers around a wrapper around free() are of no more use than
wrappers around wrappers around calloc().

The purpose of the free() wrapper is to generate an error message
when free'ing NULL. But fr

Wrappers around a wrapper around free() are of no more use than
wrappers around wrappers around calloc().

The purpose of the free() wrapper is to generate an error message
when free'ing NULL. But free_hash_bucket(), free_lease(), and
free_tree() are all called immediately after dereferencing the
pointer to be freed. Thus the error message will never be issued.

show more ...

A slightly more complex calloc() wrapper wrapper, new_hash_table(),
bites the dust.

Another calloc() wrapper wrapper, new_tree(), bites the dust.

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

Single-use wrapper functions that just call a wrapper function
around calloc() and end up producing duplicate error messages are
just confusing.

Eliminate new_hash_bucket(), new_lease(), new_leases(

Single-use wrapper functions that just call a wrapper function
around calloc() and end up producing duplicate error messages are
just confusing.

Eliminate new_hash_bucket(), new_lease(), new_leases(), new_group().

show more ...

Plug a potential leak similar to the one parfait spotted and Theo
fixed.

Eliminate the function new_class() by expanding the only use

Use strdup() and calloc() rather than manually simulating them.

Plug a potential leak similar to the one parfait spotted and Theo
fixed.

Eliminate the function new_class() by expanding the only use

Use strdup() and calloc() rather than manually simulating them.

No functional change.

show more ...

avoid aliasing with libc functions; ok henning

remove things not used, spotted by lint mostly; ok henning

cheat to KNF and ansi this: take the one from dhclient and add back the
few missing functions

may the whacking begin