headers: include-what-you-use fixes

Usage of includes are set up the way found to be in use, with a help of
magnificent utility.

http://code.google.com/p/include-what-you-use/

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: add missing full-html flag

Missing both from help output & manual page.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: help screen & man page

The help screen has now same notation as most of the GNU/Linux
utilities, e.g. if a option requires argument it is marked as
option=ARG. That enabled me to use help2man to make small nroff
fixes to manual page to unify it's outlook as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

thanks: list update

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

shared network names to be dynamically allocated

This patch is a step a head to make data all structures
completely dynamic. After this the next obvious thing to do is to
make shared networks struct a linked list instead of a list.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

make range allocation dynamic

This is a test fix after commit

5cbe8d07fb23db630e76d18022f20368312720ec

to see what can be done. Truth is that not much. I could fix how
ranges are allocated, but the fact there is pointers to shared
networks and network names reallocating the memory spaces is not
really going to work. The only way to truly fix this issue is to
create better data structures. As you can expect that is a major
change, and will take some time to implement.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

clean_up: checking errno at exit does not make sense

It seems that at least FreeBSD is setting errno when malloc is
successful. There is a change that other operating systems will
do similar things with same, or other system calls, and this will
invalidate the idea of checking errno at exit.

See also email list thread for further information.

http://lists.freebsd.org/pipermail/freebsd-questions/2011-April/229150.html

Reported-by: Roar Pettersen <rope1968@gmail.com>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

getdata: increase max number of shared networks

Anders & Thor Eivind reported dhcpd-pools to mess console and
crash at the end of execution. Reason was missing boundary check.
The fix this time is more of less just a increase to max numbers.
That will work ot some extent.

The memory allocations has to be made fully dynamic, but that is
a little bit difficult with the way default.h and parse_config
are working. Basically problem is globals gone crazy & re-entrant
issue with parse function. The true fix to the issue will have to
wait for complete redesign of internals of the command.

Reported-by: Anders Låstad <ala@nextgentel.com>
Reported-by: Thor Eivind Brantzeg <tbr@nextgentel.com>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: git describe listed wrong tag

The tag v2.14 was probably done without -a option so it was not
recogniced by git describe. To over come this I found easiest to add
--tag to git-version-gen. This change should be reverted after a
release.

Signed-off-by: Sami Kerola <sami.kerola@tomtom.com>

build-sys: use git version as program version output

The functionality copied is from gnulib.

http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=blob;f=build-aux/git-version-gen;h=686f7031a6037445bfcb7b43fb4da3c8240d8575;hb=HEAD

Signed-off-by: Sami Kerola <sami.kerola@tomtom.com>

dhcpd-pools.h: portable program_invocation_short_name

The program_invocation_short_name portability code is copied
from util-linux project almost as is.

Reported-by: Roar Pettersen <rope1968@gmail.com>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flose files & free memory when not needed

Fixes to issues found with valgrind.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

autotools: point correct location of src

Fix to error where program src is. Also a bit of clean up and
alignment with autoscan output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

getdata.c: initialize variables to have safe values

Primarily to get rid of following warning.

getdata.c:63:26: warning: 'macstring' may be used uninitialized in this function
getdata.c:66:20: warning: 'macaddr_p' may be used uninitialized in this function

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

dhcpd-pools.h: indentation fix

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

dhcpd-pools.h: function attributes declarations

Advice compiler about functions to generate better optimization
and argument checks.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Release 2.14

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

configure.ac: autoscan updates

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ignore INSTALL file

I forgot to put the INSTALL file to .gitignore at the time of
commit fdcfcd12d3fd5a5a62ed7c326c9d7c52c6aefc94

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

fix to safe_strdup prototype.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

INSTALL file is unnecessary

Also few typos got to be fixed.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Allow xml output with leases to use output file

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Copyright notice change to GNU license 3

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

autogen.sh: typo fix in previous commit

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

autogen.sh: test bug fix & clean up to messages

The test had == used as = which is bashisms, and does not work on
system, such as Ubuntu, where /bin/sh is dash or some other
non-bash shell.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Do not analyze if sort field selector is rubbish.

Error message also had a bug, it's changed to be over all more
helpful.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

posix_fadvice advice parameter change

With a bit of manual rereading and thinking I came to conclusion
that users will most likely fetch same data over and over again.
Advicing to kernel to keep pages into cache seems to be correct
thing to do.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Impossible to reach sections removed

With a bit of gcov analysis, and logical thinking, it was obvious
that section that got to be removed cannot be reach.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Not a syscall error, warn should be warnx.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Use what libc will provide

The eprintf removed and replaced with err & warn. Option parsing
no longer tries to find missing optargs, which are getopts should
notice. Few complier warnings got to be removed as well. Finally
the commments will no longer exceed standard terminal width.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Optimization on basis of gcov

Code blocks to more common first order. Also couple checks got to
be refactored next to section where update happens, that will
reduce unnecessary checking.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

The xml output is dirty and marked as FIXME

The xml output does not use output file as all other outputs, it
has sections here and there while everything is supposed to be in
output.c, and finally the unnecessary field analysis loop made
program about 20% slower when xml was not asked.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Arithmetic comparisons to be same way around

In writing arithmetic comparisons, use "<" and "<=" rather than
">" and ">=". For some justification, read this:

http://thread.gmane.org/gmane.comp.version-control.git/3903/focus=4126

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Automake 2.64 is needed.

This is due AC_PACKAGE_URL that is used in AC_INIT, see release
announcement for details.

http://lists.gnu.org/archive/html/autoconf/2009-07/msg00079.html

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Fix to range dynamic-bootp specifier

Rely on inet_aton return value instead of some unreliable
conversion result after first syscall failed.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

After fopen in ouput ioctl does like /dev/null which cause
ENOTTY, and clean_up will see that without this reset. At least
linux does this, and possibly some other systems. There's a
report from FreeBSD 8.0 which matches quite well with the
symptom.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Proper .gitignore files

The standard stuff that is needed in ignore files to make git
status etc commands not to freak out after a build.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Oops adding empty m4 directory did not work.

The m4 has now .gitignore without content. That should work just
fine as a place holder director for automake.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

The automake missed local m4 directory and libtool init.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

README with test data wanted announcement

The file is also reintended to be more like other files.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

README to include quick start.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ChangeLog obsoleted

See git '(log|show|diff)' for all possible details you would ever
need.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Rest of the automake file deletions.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

All files that autotools will generate are removed.

Who ever will compile the sources is encouraged to use
./autogen.sh to get ./configure and other necessary files.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

README and THANKS files to support git

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Import from release candidate 2.13.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>