lib: keep space for new values in `lldpctl_key_t`

By inserting new values in the middle `lldpctl_key_t`, we break the
ABI of the library. To ensure future backward compatibility, we
introduce some space between each block. Unfortunately, this breaks
the current backward compatibility.

debian: synchronize packaging with Debian

release: prepare 0.7.3

NEWS: add a word about Solaris support

solaris: less intrusive way to get CMSG_* macros

The use of _XOPEN_SOURCE breaks most BSD.

solaris: assume that IP forwarding is disabled

We don't know how to detect that while in the chroot.

solaris: preliminary support

Basic functionalities are present. However, the interface support is
very poor. There is no way to detect bridges, VLAN and
aggregates. There is no MAC/PHY support. There is no detection of
wireless devices.

The code to detect IP forwarding is here but does not work inside the
chroot.

NEWS: add an entry about LLDP-MED fast start.

lldpcli: allow to enable/disable fast-start and set delay

lldpd: enable fast start by default

lldpd: make fast start work only when receiving LLDP-MED LLDPDU

lldpd: fast start support

This patch adds support for lldpd fast start.

The feature is currently disabled by default.
A cli command to enable the feature is in the
works.

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>

configure: use `AC_PROG_EGREP` to find a suitable `grep -E`

On some systems, this is `ggrep -E`.

lib: use uint8_t instead of u_int8_t

`uint8_t` is part of C99 and comes from `stdint.h`. `u_int8_t` works
on many OS but not on some. There are many other occurrences but we
fix those here because this is the only public header.

configure: display configured OS

lldpcli: complete the manual page with the syntax of "show statistics"

lldpcli: summary of statistics is done on provided ports only

lldpcli: fix two warnings introduced by the previous commit

NEWS: add an entry for "show statistics summary" command

lldpcli: global stats command

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>

os: add support for DragonFly BSD

This is similar to FreeBSD except that DragonFly BSD does not support
the following:

 - link aggregation
 - interface description
 - DMI

netlink: fix IP address discovery

Since the switch to Netlink, the IP address was not advertised
correctly. Fix `netlink_send()`. Patch courtesy of Pradosh Mohapatra
from Cumulus Networks.

lldpcli: fix typo in the man page for setting altitude

bsd: don't report a warning when no media information is present

On NetBSD, xennet interfaces do not support reporting media
information. This is expected. Just log this at debug level.
Closes: #34.

build: both client and lib may need shipped headers in include/

For example, when sys/queue.h is incomplete.

fgetln: getline may need _GNU_SOURCE

install: install ChangeLog

bsd: reorder headers to fix compilation of various BSD

Prepare for 0.7.2 release.

doc: update NEWS entry for "show statistics" command

lib: remove unneeded struct for stats in private.h

lldpcli: use human labels for statistics

Also fix display of "Ageout" and "Inserted".

lldpcli: reuse cmd_restrict_ports() to handle port restriction in "show stats"

event: add missing <time.h> include

lldpcli: Add support to display lldp stats

This patch adds a new lldpcli command and required
server code to display per interface lldp stats

Sample output:

#lldpcli show statistics ports eth0
---------------------------------------------------
LLDP statistics:
---------------------------------------------------
Interface:    eth0
  tx_cnt:       2
  rx_cnt:       2
  rx_discarded_cnt: 0
  rx_unrecognized_cnt: 0
  ageout_cnt:
  insert_cnt:
  delete_cnt:   0
---------------------------------------------------

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>

Revert "debian: on Debian, don't ship upstart script"

This reverts commit d5e9611c1cc903410f42a8272898b31404559c6f.

We can't request dh-autoreconf and have a buggy dh_installinit at the
same time. Let's remove this code about upstart. This only needs to be
fixed in backports.

debian: on Debian, don't ship upstart script

debian: require dh-autoreconf for autoreconf.mk to be present

cdp: force CDPv2 protocol with `-ccc` argument

lldpd: really accepts only 4 neighbors

lldpd: enforce TTL on remote chassis

Remote chassis were not expired in a timely manner. They were only
expired when a change happened on an interface. To fix this, we
maintain a timer that will be triggered when an interface is about to
expire. The timer is updated when a change happens and rescheduled
once it is run.

lldpd: limit the maximum of neighbors per port and per protocol

lldpd memory can be exhausted by adding too many neighbors. Moreover,
the communication with lldpcli is limited to a fixed number of
neighbors (packets may not exceed 1 << 19 in size). Therefore, we put
a hard limit to the number of neighbors accepted for a given port, for
a given protocol.

The limit will be tunable in the future.

Feature suggested by Chris Yang. Closes #32.

README: add a note about executing lldpd without installing it

lib: fix sync_recv to handle segmentation

While it should not happen with Unix sockets, a read() can return only
partial data. Therefore, we continue to poll until we get the
appropriate length.

This fix was proposed by Chris Yang. Closes #31.

test: add an helper to generate many neighbors

Using scapy, we generate a pcap with as many neighbors we need.

lldpd: correctly parse /etc/os-release

The previous parsing was just bogus and eated latest characters.

lib: `ctlname` in `lldpctl_conn_t` is not const

It is allocated on the heap by `strdup()`

lib: bump version-info of liblldpctl for `lldpctl_new_name()` function

Add a argument to specify the unix control socket

Add autoreconf to call autogen.sh while build deb

build: also ship ./autogen.sh

build: use the same way as libbsd for version and changelog

See:
 http://cgit.freedesktop.org/libbsd/tree/

build: try to save current version in `VERSION`

install: remove liblldpctl previous symbolic link

Otherwise, we get an error on "make install".

configure: only match tags starting with a digit for version number

build: check if libevent as provided by pkg-config is working

If a user installs both libevent 2.0.x and libevent 1.4.x, pkg-config
may report libevent 2.0.x where it really is 1.4.x. The system is
broken in this case but we work-around this by using the shipped
libevent.

doc: add documentation about reading /etc/lldpd.conf and /etc/lldpd.d/*

interfaces: whitelist an interface before being checking lower

Some whitelisted interfaces, like dsa, have a lower interface in
managed mode but we still want to advertise on it. Move the check for
lower interface after whitelisting.

libevent: produce smaller binaries when shipping libevent

When shipping libevent, we try to build smaller binaries by disabling
unneeded stuff (openssl, threading, malloc replacement, debug stuff)
and enabling `--gc-sections`. On various tests, this remove 80K from
lldpd size (stripping excluded).

lldpcli: possibility to configure tx-delay

lldpcli: fix exit command

The exit command did not exit lldpcli anymore.

lldpcli: split actions.c in several files

We need `configure` to be more modulable.

lldpcli: put port restriction into a commands.c for use in other files

lldpd: discard output of lldpcli

We keep stderr since we may have logs here.

lldpd: pause only if lldpcli has not been disabled

lldpd: pause at start, unpause through lldpcli

lldpd will be paused at start. Nothing will run. lldpcli job is to
unpause it once the configuration has been read.

lldpd: invoke lldpcli at start

To configure itself, lldpd now invokes lldpcli when
starting. Currently, it does not really check if everything works as
expected but in the future, lldpcli will ask lldpd to "start".

The configuration files are:
 - ${sysconfdir}/lldpd.conf
 - ${sysconfdir}/lldpd.d/*.conf

ctl: close control socket on error when connecting

build: lib and client don't need shipped OS specific headers

lldpcli: add an option to read commands from files

Fix some spelling mistakes.

debian: don't remove inexistent CHANGELOG file

debian: provide a development package for liblldpctl

priv: add ability to lock filter on Linux

This is a new feature which should be available in Linux 3.9. See:
 http://git.kernel.org/?p=linux/kernel/git/davem/net-next.git;a=commit;h=d59577b6ffd313d0ab3be39cb1ab47e29bdc9182

priv: make inability to set write filter as fatal

On BSD, if the userland supports an ioctl, the kernel is in sync and
also supports the ioctl. So, if BIOCSETWF exists, it should succeed.

priv: create chroot if it does not exist

This is not the ideal situation since the chroot should maybe contain
some additional stuff (like `/etc/localtime` with GNU libc) to make it
work properly, but this is more convenient that leaving the user does
exactly the same things. Packagers are still expected to populate the
chroot correctly before starting lldpd.

priv: lock BPF interface on BSD

We move all BPF setup in the monitor process. When the socket is
configured, we lock it. This works for both OpenBSD and FreeBSD. This
feature does not seem to exist on Linux. However, for consistency, we
also move BPF filter setup in priv.c for Linux.

lldpcli: add "help" command in manual page

osx: don't include "head" bits in homebrew formula

We will add them back when they work.

osx: update Homebrew formula with JSON and SNMP support

snmp: workaround unknown value for RONLY

This happens when Net-SNMP has been compiled with
NETSNMP_NO_LEGACY_DEFINITIONS defined. For example, on Mac OS X. This
seems a good way to deprecate an API still widely used just because it
is not used internally... <grin>

marshal: serialization may return 0 and target untouched

If serialization finds a structure already serialized, it skips it and
return 0. `target` should not be freed.

snmp: frame.h is not needed anymore

The `frame_checksum()` function is declared in `lldpd.h`.

clang: additional fixes detected by clang analyzer

Small fixes detected by clang analyzer.

 - log: mark `fatal*()` function as noreturn
 - event: insert client in the list after its creation
 - lldpcli: avoid confusion by initializing cargc to 0
 - lldpd: avoid ambiguous use of strlen() in initialization
 - lldp/edp: fix memory leaks
 - tokenizer: fix a memory leak in low memory condition
 - cdp: don't accept too short TLV for port description

packaging: update packaging stuff for release

build: more changelog drama

We need ChangeLog to be listed in `dist_doc_DATA` or
`doc_DATA`. Therefore, we list it in the former and if we happen to
install from git, we just ship an empty ChangeLog.

systemd: don't use SOCK_CLOEXEC

It is unknown on RHEL5. Moreover, we promise to not exec anything in
this function.

event: replace obsolete `sigignore()` by `signal()`

doc: new homepage for lldpd

compat: add `strnlen()` which may be absent on some older systems

bsd: compatibility with systems that do not have interface descriptions

build: fix changelog generation

We really only want to generate changelog when building the dist
tarball. We force its generation using a fake PHONY target. We don't
list it as a dist_* something since it does not exist yet. Therefore,
we don't have to clean it.

osx: add an homebrew formula

This formula is currently incorrect because of the bad MD5 sum. We
will correct it at the first release supporting Mac OS X.

configure: compatibility with automake 1.12

AM_PROG_AR should be used. The reason seems a bit obscure. This should
be done in libtool.

autogen: don't use readlink -f

This is not portable. We are only interested in the basename, let's
just use $PWD instead of `.`.

autogen: allow to override commands used

For example:

    LIBTOOLIZE=glibtoolize ./autogen.sh

dmi: implementation for Mac OS X.

We use the IOKit framework.

Mac OS X support.

This includes bond, VLAN and bridge support. Mac OS X is pretty
similar to FreeBSD except for bonding which is different. Bonding code
is stolen from ifconfig.c. Header files from XNU are
shipped because they are missing from Mac OS X developer tools.

Still missing:
 - integration into launchd
 - homebrew formula
 - DMI part (through ioreg)

build: add tag date in changelog