release: prepare 0.9.6

redhat: add support for JSON in CentOS >= 6

This seems supported in RHEL now, but doesn't work on OSC:
 https://rhn.redhat.com/errata/RHEA-2014-1530.html

Fix #220

netlink: clear IFF_SLAVE when no information on upper interface

This means that we have just been enslaved, but don't know the
master. On Linux, enslaved device are handled differently of regular
device. When we are half-enslaved (no upper interface yet), we are still
a regular interface. When we are fully enslaved, we may do nothing
because there is no local change (since flags are the same). Therefore,
clear the flag to avoid this issue.

Merge pull request #218 from vincentbernat/feature/netlink-autoadapt

Auto-adapt Netlink socket size

Merge pull request #216 from vincentbernat/fix/epollerrr

interfaces: on Linux, clear socket error queue on error

client: fix typo in manual page

Merge pull request #219 from alagoutte/fix

Fix Expression '!iface' is always true

Fix Expression '!iface' is always true

netlink: minimize code duplication when handling buffer sizes

netlink: by default, don't change netlink buffer size

We can't really guess what the current maximum is. On my system, the
default max is quite low (~200k). We don't want everybody to get a
warning. Also, only warn people if we got a smaller size that we
requested.

netlink: automatically increase netlink receive buffer on errors

netlink: set size to the correct value for getsockopt()

Also, don't check if the returned size is correct. This is not done in
other parts of the code and it seems a bit useless to do it here.

netlink: make change of buffer size optional

We don't really need to change the send buffer size. Put it to 0 by
default.

netlink: make the values for buffer size configurable through ./configure

doc: fix name of new ethtool command

daemon: config netlink socket buf sizes at build time

I admit it would be nicer to have this configurable at run-time
or better yet, introduce a mechanism for recovering from
overrun situations via polling.

We're starting to see in certain situations that on devices
with many ports (i.e. switches with around ~50 ports) lldpd
info is not always consistent.

I'm not entirely sure that the issue is with netlink
sock buffer sizes.

But we've also recentl addressed some issues for the mstpd
daemon with netlink sock buffer sizes.
So, I'm tempted to think that it's the same issue.

We [especially I] prefer to bump sock buffer sizes where needed
and not via sysctl `net.core.rmem_default` & `net.core.wmem_default`.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

tests: avoid race condition when linking two namespaces

When creating a link between two namespaces, one lldpd will see the link
and send a LLDPDU before the other one. There is a slight delay for
lldpd to wait before sending the LLDPDU, but when the load is heavy, it
can be expected that the other side didn't configure the interface when
the LLDPDU is received. To solve this, just wait a bit before putting up
the second interface.

build: use @mkdir_p@ instead of @MKDIR_P@

Uppercase "variables" are otherwise only used for lldpd-specific stuff.

daemon: suggests additional sandboxing with systemd

This is only suggested because they are not supported by all systemd
versions (and may therefore emit warnings).

See:
 - https://lwn.net/Articles/709764/
 - https://lwn.net/Articles/709350/

interfaces: on Linux, clear socket error queue on error

With AF_PACKET, it is possible to get an error in the error queue even
when we didn't ask to receive them (in contrast with AF_INET where we
have to set IP_RECVERR socket option). Currently, the only error we can
receive is SOL_PACKET/PACKET_TX_TIMESTAMP, but we just want to clear any
error in the socket queue to be able to read packets (otherwise, poll()
will always return POLLERR).

Fix #215.

interfaces: refactor generic ethernet/bond reception for Linux

Revert "tests: mark tests spawning a new port as flaky"

This reverts commit 0efde559064fc3b842d06b56bafa1c457b1a8fb4. This
isn't compatible with xdist...

tests: mark tests spawning a new port as flaky

They will be retried once.

tests: fix when Dot3 is not enabled

client: display current MAU type even when autoneg is disabled

lldpd: don't make lldpcli complain on missing conf files

Only complain if the user has directly provided additional configuration
files. This is done through an hidden option (-C) for lldpcli.

tests: minimize file descriptor leaks

This is not perfect as we still rely on garbage collection to do its job
and it seems it is not running often enough.

lib: include MAU type map only when Dot3 support is enabled

Merge pull request #212 from vincentbernat/feature/ethtool-linksettings

linux: add support for ethtool GLINKSETTINGS

tests: wait even more when spawning a new interface while lldpd is running

linux: make ethtool stuff run as root again

This is needed for ETHTOOL_GLINKSETTINGS.

lib: add more MAU types

linux: add support for ethtool GLINKSETTINGS

With a recent enough kernel, one can use GLINKSETTINGS instead of
GSET. This is needed to overcome the limitation on the number of
supported/advertising modes supported by GSET (which is a fixed-length
bitset). We use the same kind of compatibility structure than for
ethtool.

Also, update the list of supported MAU types to support higher speed
interfaces. Unfortunately, while Linux reports the exact
advertised/supported modes, the operational mode of the interface is
only reported with its speed and its kind (copper, fiber, ...). When
this is fiber, we don't know if this is MMF or SMF or the supported
distance. Therefore, we use our best guess (this was already the case,
nothing new here).

seccomp: add more syscall (sendmmsg and recvfrom)

Merge pull request #213 from chutz/add-mprotect-to-seccomp-whitelist

seccomp: add mprotect to seccomp whitelist

seccomp: add mprotect to seccomp whitelist

tests: more fix for pcap test

tests: fix previous commit by making LLDP-MED part optional

tests: use a PCAP file to do some integration tests

med: fix parsing of LLDP-MED LCI when TLV size exceeds addr size

Some equipments are sending an address whose size is less than the
declared TLV size. The TLV is padded with 0. We rejected this because of
a size mismatch.

Fix #210.

linux: comment out part of ethtool.h

It relies too much on kernel.h.

github: shows a tcpdump output

linux: pull up-to-date ethtool.h

client: fix extra warning about misleading identation

Use a code block instead.

github: provide an issue template

client: add an option to use pre-0.9.2 json-c format

The format has been changed in c8b8b858bbba to match the JSON format
used when compiled with Jansson. Some users may want to revert this
change. Add `--enable-json0` option for that.

travis: enable gold linker when NOT on OSX

travis: disable gold linker with OSX

It seems it is not compiled anymore. See:
 https://travis-ci.org/vincentbernat/lldpd/jobs/165445935

travis: display config.log when configure fails

release: prepare 0.9.5 for OSX

redhat: add LICENSE file to manifest

release: prepare 0.9.5 release

event: correctly free event base

This removes a bunch a false positive in valgrind memcheck where some
allocations where done on the base and not freed when only the
associated events where freed. Related to #192.

daemon: don't invoke lldpcli after dropping privileges

Otherwise, we don't have access to the Unix socket anymore...

valgrind: more suppressions for lldpd

daemon: make the message about going into background more apparent

Otherwise, people may think that lldpd just crashed.

daemon: better drop privileges earlier

Commit 06d83d0fb35c delayed privilege dropping because we delayed
daemonization. This may made sense to be able to display errors to user
when attached to the console, but this makes little sense from a
security perspective. So, put everything back to the earlier possible
location (just after setting up the socket).

daemon: invoke lldpcli after dropping privileges

We don't need to run it as root. Moreover, it will happen after
daemonization and won't clutter the screen in case lldpd dies too
early. On the other hand, if we fail to spawn it, the error message may
be hidden in the logs. Though choice.

daemon: wait a bit later to daemonize

This way, some additional errors may be advertised in a better way.

tests: add an __init__.py file to turn fixtures into a proper module

Dunno why this was not needed before.

event: remove some one-time memory leaks

Also provide a suppression file for valgrind for memory leaks that are
too difficult to remove.

doc: add a license file

This help audit software to automatically detect the license.

travis: try to workaround "unrecognized option '--push-state'"

See: https://travis-ci.org/vincentbernat/lldpd/jobs/159419045

lldp: only enable LLDP-MED capibility TLV if we have that capability

In fact, we are not quite coherent. With LLDP-MED disabled (lldpd with
"-M" option), we can still set some TLV. We should either disallow that
or at least not send the capability TLV since we cannot build a complete
one (we miss the device type). We chose the later.

Merge pull request #198 from The-42/fix-segfaults

lib/atoms/port: fix segfaults with minimal information

lib/atoms/port: fix segfaults with minimal information

When no packet has yet been received, querying LLDP information leads to
segfaults. Fix those by adding a couple pointer checks.

Signed-off-by: Bert van Hall <bert.vanhall@avionic-design.de>

lldpd: fix kernel version check

An erronous warning was displayed about kernel being too old while this
wasn't the case.

lldpcli: specify that custom TLV bytes are in hex format

build: test if libbsd is really usable

It is possible for libbsd to not be usable due to the use of
"-isystem". See for example:
 http://autobuild.buildroot.net/results/c8a/c8a6001f437701ecc75f6c9252935645bda8a8c8/lldpd-0.9.4/config.log

In this case, just don't use it.

doc: document the change about ethtool info grabbed without the monitor

build: ensure clang won't complain about documentation of linux headers

priv: don't use monitor for ethtool except for old kernels

Since 2.6.37, we don't need any privilege for ethtool GSET. Not using
the monitor will increase performance.

interface: use 10GBASE-CX4 for copper

That's the best approximation available in RFC4836.

interface: more media for *BSD

OpenBSD and FreeBSD didn't agree on symbol names. Use #ifdef everywhere.

lldp: update RFC3636 to RFC4836

Unfortunately, RFC4836 still lags behind. For example, no more than 10G
for speeds, no 10GBASE-T. For advertised modes, we try to be more
future-proof by using "other" for any unknown mode. Moreover, document
the fact that we advertise 10GBASE_T as 10GBASE-R and on Linux, we have
to guess when the port is fiber (and we guesss 10GBASE-X).

include: update linux/ethtool.h to 4.7

This is needed to support ETHTOOL_GLINKSETTINGS.

priv: don't use ethtool to get real MAC

It was used if /proc/net/bonding was not available but:

 1. it shouldn't happen
 2. some drivers don't have a permanent MAC
 3. we may override a user setting
 4. more code to maintaing

priv: document functions that should not use the monitor process

We should use them only if "oldies" are enabled.

NEWS: add an entry for 106aa50d

Revert "lldp: skip empty LLDP-MED inventory TLV instead"

This reverts commit aa357d2b6b54c966353bcee4d7f14640540a8cf2. The
previous behaviour was equivalent. Let's stick to that for now.

lldp: skip empty LLDP-MED inventory TLV instead

travis: fix issue with Travis and libtool

build: provide a better date format when using SOURCE_DATE_EPOCH

And define the appropriate date at configure time.

client: flush output after each block

This enables to get realtime output from "lldpcli watch".

See #187.

build: ensure typeof is correctly defined

This could be typeof/__typeof/__typeof__.

Fix #186.

build: make generation of atom-glue compatible with older gcc versions

With old versions, cpp doesn't accept several files as input. See #186.

release: prepare 0.9.4 for OSX

release: prepare 0.9.4 release

osx: update lldpd to 0.9.3

doc: document latest commit

lldp: add ability to control propagation of LLDPDU

In 802.1AB-2009, two additional target addresses were added to allow an
LLDPDU to not be stopped by some equipments. Expose this ability as a
configure command.

Fix #171

lldpcli: fix watch return code

When watch terminates normally (after hitting a limit), ensure lldpcli
exits with status code 0.

lldpcli: fix memory leak with use of readline

The lines returned by readline() should be freed.

interfaces: blacklist cdc_mbim

This exposes a weird interface.

lldpcli: add a test for return code

lldpcli: ensure we exit with an error on incorrect commands

lldpcli: fix manual page for port status

Use rx-only, tx-only, rx-and-tx instead of rxOnly, txOnly, rxAndTx. Also
add some tests.

tests: remove useless variable

build: fix build issue introduced in previous commit

$^ was used for a reason: automake would fix the paths used in
dependencies using VPATH or something similar. For some reason, this
even break with GNU make when not using OOT build. Long story short, we
use a pattern substitution to ensure that the atom files are looked up
in the correct directory.

This use of substitutions is mandated by POSIX (the 2013 version I
think). See:
 http://austingroupbugs.net/view.php?id=519

It is however believed that most make implementation had support for
this since a long time.

build: don't rely on GNU make $^ automatic variable

Not supported by BSD make. And not really needed either.

Fix #185.