tests/integration: sleep a bit more

With address sanitizer, things are now slower.

travis: retry non-parallel tests on failure

On failure, reports are incomplete with parallel tests. In this case,
rerun failed tests with full verbosity.

build: let configure tell us if we have address sanitizer

We cannot really rely on __has_feature or __ADDRESS_SANITIZER__ in code
since we are mostly interested in the leak sanitizer and there is
neither a feature nor a macro for that. Early version of GCC have the
address sanitizer, but not the leak sanitizer. We don't support this
configuration, but we need to build correctly either way. So, the user
is expected to enable address sanitizers only on configuration
supporting also the leak sanitizer.

travis: use gcc-5 if possible

Better support for address and leak sanitizer

client: fix a memory leak in kv writer

tests/integration: make use of libtool to execute lldpd/lldpcli

tests/integration: mount /proc in namespaces

Because of the use of a PID namespace, we must mount /proc into the
appropriate namespace. We don't do that directly when creating
namespaces as clone() doesn't account for the namespace change with
setns() when we are still in the same process. We also fork a process to
do the mount as it seems mount() doesn't get that we are in a different
mount namespace either. Obviously, setns() has some drawbacks we need to
workaround.

We also mount /proc in the chroot. It's absolutely not safe to do so,
but that's only for address sanitizer to work as expected.

lldpcli: silence expected memory leaks

The suppression file doesn't have the same syntax as valgrind. Instead,
use code annotations.

fixedpoint: fix buffer overflow in fixed point computations

This was detected by address sanitizer. This was harmless as we use
exclusively 5-byte buffers and we know the next byte is always unused
due to alignment.

tests: don't check for leaks in fixed point tests

tests: correctly encode civic address for SNMP tests

For some reason "\x02F" was interpreted as "\x2f", not as "\x02"
"F". Use separate strings to ensure correctness.

tests: disable leak sanitizer for unittests

Unittests will leak because it's not easy to cleanup after them. We
still want other sanitizer checks.

travis: enable address sanitizer

build: add an option to enable sanitizers

A user cannot just use CFLAGS=-fsanitize=address because autoconf is too
fragile in some of its test (notably AC_FUNC_MALLOC and
AC_FUNC_REALLOC).

build: display compiler version info after configure

This is useful to get more insight on what is supported (notably Asan
support).

tests/integration: when several neighbors, be safe and sleep a lot

travis: use a separate test for oldies

We want a complete coverage for this case.

tests/integration: down then up with vlan require dot1 support

travis: run integration tests on Linux

After some tweaks, they seem to work as expected.

tests/integration: sleep more after anything that create an interface

See previous commit for rationale.

tests/integration: wait more when down/up interface

With IPv6, DAD may interfer and trigger more changes and lldpd will wait
those changes to settle.

tests/integration: reduce the number of emulated neighbor

This is costly when we are running on low-memory containers.

tests/integration: fix _lldpd group creation

tests/integration: use ctypes for mount instead of util-linux

Some versions of util-linux had a bug with respect to private/slave
mounts. The kernel doesn't expect the option to be passed during the
initial mount but only during subsequent calls (you mount the FS, you
make it private and slave). Directly use mount(2) to avoid the problem.

travis: move install/run steps into tests/ci

They are not specific to Travis.

tests: report complete Linux version as well

tests: replace integration test by py.test+namespace tests

Relying on namespaces enable us to quickly run isolated instances of
lldpd without the need of virtual machines. Since the startup time is
quite fast (despite having to wait for lldpd to be "ready"), we can use
a classic unittest framework like py.test to run tests and get
appropriate reports. Tests can be run in parallel to overcome the
slowness induced by all those `time.sleep(2)`.

build: ensure "make distcheck" work for any value of sysconfdir

When using `--sysconfdir=/etc`, `make distcheck` was failing because it
did use `--prefix=...` to force a different installation path (and not
`DESTDIR`). During `make distcheck`, we force the use the original
value.

lldpcli: fix output with json-c to be the same as with jansson

lldpcli: acknowledge memory leak in another map conversion

doc: document last commit in NEWS

interfaces: handle correctly operation conversation of a port

When a port was regular and become an enslaved member of a bond, we
created a different port. Since we now keep the old ports around, the
old port may still attract a lot of things, like specific configuration
or VLAN. Therefore, we handle the conversion of a port from one kind to
another.

Another idea would be to not do special handling for bonds. Only old
kernels need that. We could remove that later.

interfaces: replace "bonded" by "enslaved"

It is more in par with the appropriate terminology for bonds in Linux.

netlink: ensure lower link doesn't change for an interface

The lower link of an interface is defined at its creation. It's not
possible for it to change. This is important to not try to change it
because the kernel won't send IFLA_LINK_NETNSID each time it sends
IFLA_LINK.

interfaces: issue a warning if we get a loop

We really want to correct that.

interfaces: limit the maximum search depth when applying a VLAN

It's now quite easy to hit a bug where we loop over interfaces when
trying to find the physical interface associated to a VLAN. Put a
maximum depth of 5.

custom: fix a capitalization error in man page

Also correct some indentation.

include: fix definition of IFLA_RTA to please clang

In `linux/if_link.h`, clang assume we have an alignment problem while we
know this is not the case. Add a cast to void* to avoid this warning.

netlink: don't consider a lower interface when in another namespace

The index of an interface is specific to a namespace, don't try to
interpret anything about interfaces belonging to another namespace. This
change unbreak some scenario, like in an appropriate loop detection
because an interface from another namespace is detected (vlan100 ->
veth1 -> veth0 with same index as vlan100). However, it is not possible
to exactly detect a physical interface anymore since we don't really
know what can be on the other side of the interface (in the other
namespace). However, bridged, bonded and VLAN interfaces should be safe.

lldpcli: with key/value display, remove any newline

They make hard to parse the result. Just remove them. This happens for
example with CDP.

lldpcli: fix display of LLDP-MED POE TLV

lldpcli: display LLDP-MED caps like LLDP caps

While LLDP caps can be available and/or enabled, LLDP-MED caps are only
available. However, the way they were declared made them invisible in
some formats, like the key/value pair. Try to fix that. Warn about the
change in NEWS file.

solaris: remove unused headers

version: display configuration directory as well

version: display more build information when using "-vv"

Display supported features as well as compilation flags used. This will
be useful to let integration tests autodetect compiled features.

doc: document PR #163 in NEWS

Merge pull request #163 from kanrajag/feature/enhance-customtlv

enhance custom tlv support

enhance custom tlv with add/replace commands
allow for specific custom tlv to be deleted

osx: update Homebrew formula to 0.9.1

osx: reduce differences with official homebrew formula

osx: ensure _lldpd user is hidden

We use /usr/bin/false. Otherwise, with OSX >= 10.8, having an UID < 500
is not sufficient. Another way would be to set `IsHidden` to TRUE but no
other users have this property.

build: keep user-specified CFLAGS

When user provides a CFLAGS at configure-time, keep it.

osx: explain how to compile for older versions with recent SDK

It's still quite unclear how we could know which versions a given SDK
supports. Let's assume that we can support all still supported versions
with El Capitan.

redhat: don't try to guess systemd unit directory

On SuSE, this doesn't work.

redhat: enable some CFLAGS

redhat: make systemd stuff also works with SuSE

redhat: fix parentheses on conditions

For some reason, the regular precedence for "&&" and "||" doesn't work
in a .spec. Use parentheses to overcome that.

Also, RHEL doesn't ship with libevent-devel at all. Don't use it.

debian/redhat: prepare 0.9.1

netlink: log error code when unable to receive netlink

netlink: add an entry in NEWS for previous commit

Merge pull request #161 from 6WIND/fix-large-netlink-messages

netlink: fix reception of large netlink messages

netlink: fix reception of large netlink messages

If received netlink message is bigger than the buffer provided by the
application, netlink message flag will contain the MSG_TRUNC flag. In
this case it is up to the application to allocate a buffer large enough
to receive the full message and to call recv() again.

interfaces: when no alias available, use interface name

Even if the interface name was already used as port ID.

travis: disable integration tests

Without acceleration, this is too slow.

travis: enable travis to access kernel file

travis: don't run distcheck on integration tests

travis: order environment variables to be easier to read

doc: add an entry about netlink-related fixes

travis: compatibility with Ubuntu Precise

travis: try to run integration tests on travis

It's likely to fail...

interfaces: fix alias handling

And try to add some simple tests for them. The manual page says that if
an alias is provided, the default is to use the alias as description and
port name as port ID. We try to respect that.

The tests are not as complete as they could be because the MSAP changes
due to the port ID change and this makes tests more complex.

tests: speed up integration tests by sleeping less

netlink: merge old attributes with new ones

It seems that netlink won't advertise all attributes each time. At least
type can be not advertised in a netlink message. Copy the appropriate
attributes from the old ones when they are missing and we know they
can't change or be absent.

tests: use generated libtool

libtool can be absent on the system.

tests: require presence of "ip" and "brctl"

tests: execute rtmon to keep a trace of netlink messages

tests: rework integration tests to not rely on tmux

They should be runable by a CI.

travis: also try to build OSX package

osx: workaround issues with SIP on El Capitan

We cannot install in /usr anymore. Install in /usr/local instead (hoping
we won't conflict with HomeBrew).

osx: simplify build

We don't need to support multiple archs anymore. Therefore, no need for
lipo. Also, no need for rebuild. We can use the current build.

doc: replace "Mac OS X" by "OS X"

build: only test libbsd linking if we have found it

lib: only define ntohll if not already defined

interfaces/linux: remove interface blacklisting

Once there is a candidate to be blacklisted, we can restore the
code. Meantime, less code, less bugs.

interfaces/linux: blacklist some drivers instead of whitelisting

Currently, there is no driver to blacklist. In the past, the virtualbox
pseudo-interface was a good candidate but it is not used anymore.

interfaces/linux: don't check transmit queue size

Its size can change while no netlink messages will be sent to update it.

man: fix a spelling error in manual page

client: fix code style

client: ensure JSON is written to the provided file handle

client: ensure XML is written to the provided file handle

Currently, this is just stdout but the interface provides a file handle
and we should use it.

doc: fix NEWS file about netlink change

There is no libnl3 API-compatible library at all. It's the previous code
with small modifications.

lib: rework a bit documentation around reference counting

build: add some integration tests

build: don't use libbsd if we can't link it

netlink: handle veth loops correctly

Since Linux 4.1, a pair of veth are referenced as IFLINK for each
other. We previously detected this loop easily and did break the loop by
removing this bogus information. However, when one of the interface is
updated, only the modified interface has IFLINK pointing to the other
interface. Therefore, no more loop but the information is incorrect. To
avoid that, don't reset indexes, just pointers. We only use indexes to
build correctly those pointers.

lldpd: fix change detection

When a port was set down then up, no change was detected because the
flags were not effectively compared. Fix in 196757 was incomplete.

debian/redhat: prepare 0.9.0

No scheduled for release yet, but since the packaging was updated as
part of 0fa225, put the right version number.

travis: don't require libnl-3-dev

netlink: remove use of libnl3

Use netlink implementation from 0.7.19 instead but manage a cache
ourselves. The changes are quite minimal compared to the implementation
in 0.7.19. We handle deletion and updates. The use of linked list may be
problematic performance-wise.

When an interface goes down then up, no PDU is scheduled to be sent
again. This bug was already present in the previous implementation and
should be a regression of 36080c.

lldpd: cleanup local chassis on exit

Still for valgrind happiness.