build: only enable safe-stack if linker supports it

build: enable clang safe-stack feature if available

netlink: be more precise in how we handle link deletion of bridged interface

If link family is AF_BRIDGE, transform deletion into update of
interface. Otherwise, skip the update.

tests: fix previous testcase to fail without previous commit

Only the bridge ioctl can trigger this bug.

netlink: don't remove interfaces when they are released from a bridge

When released from a bridge, a RTM_DELLINK is sent. We detect there is
an upper device and turn thay into a RTM_NEWLINK. If the interface was
really removed, we would get two RTM_DELLINK (one with master, one
without).

On most setups, another RTM_NEWLINK was sent due to change of promisc
status.

travis: fix broken build due to incomplete python install

Unsure of the root cause. See this build log:
 https://travis-ci.org/vincentbernat/lldpd/jobs/275129851

priv: add getpid() to seccomp filter

src: #include <limits.h> when using strtonum()

`strtonum()` seems to work without it, but there is a hint in issue #242
that there is something fishy around that.

osx: sync homebrew formula

client: add NEWS entry for before-last commit

client/tests: remove standalone "return" at the end of functions

client: add show interfaces command (#240)

There are several lldpcli configure commands that take port name(s)
as an optional parameter.

Currently, it is not possible to verify interface specific configuration
for lldpd without access to a neighbour switch/host or packet capture.

It would be useful to have a show interfaces command, similar to the
show neighbors command that allows a user/agent to query the local
TLV information.

This commit leverages the existing port display infrastructure to
enable display of local interfaces.  There are a few differences:

- Do not display the chassis RID, as the local index is always 0
- Show TTL as a seperate TLV based on tx_hold, tx_interval config,
  as internally the port TTL atom info is 0, and the chassis TTL
  atom info has been depreciated.

client: fix typos in manual page (#239)

lldpd: add option to override default lldpcli config file locations (#238)

osx: update Homebrew recipe

release: prepare 0.9.8

client: fix JSON output when a special character is in string

client: add "json0" to the list of available formats

This enables a more verbose JSON output whose structure is kept
the same with one or several neighbors. It's easier to parse than the
current structure.

Fix #236

tests: upgrade pyroute2 version

tests: replace depcreated pyroute2 functions

tests: enable test for team device

lldpcli: add "configure system hostname ." to use node name

This is a shortcut to "configure system hostname nodename". By default,
lldpd will use the FQDN (resolving the node name using the local
resolver). We let lldpcli interpret "." as a special request to override
this value with the node name (output of "uname -n").

Fix #154

Merge pull request #235 from vincentbernat/feature/ethtool-no-root

priv: don't run ethtool as root

interfaces: show a warning if we don't have permissions for ethtool ioctls

interfaces: fix compilation without Dot3 feature

interfaces: don't retry GLINKSETTINGS probing if it failed

priv: don't run ethtool as root

Kernels older than 2.6.19 won't get any link information. Kernels 4.6,
4.7 and 4.8 won't get information from GLINKSETTINGS (higher speeds)
but they are not LTS kernels so we should be fine. This removes a
bunch of code.

interfaces: use ethtool to get permanent MAC for bonds/teams

As this doesn't work for all devices (like dummy and veth), we still
fallback to old code using /proc/net/bonding for bonds. No fallback
available for team devices.

interfaces: enable team interfaces like a bond interface

Unfortunately, we are not able to retrieve the real MAC address as the
kernel doesn't know it. Only the teamd daemon knows it. We could ask for
it through ethtool but it would be the permenant MAC address which may
be incorrect.

Fix #234

daemon: enable recent systemd directive by default

This will generate a warning for some users, but that should be
sustainable.

daemon: don't try to restrict address families

lldpd needs access to the most invasive address family. It's pointless
to restrict something like that.

daemon: systemd unit cannot use ProtectSystem=strict

lldpd needs to create its socket in `/run`. It's put outside the chroot
on purpose but it's not on a directory on its own.

doc: add a NEWS entry for the last change

lldp: clear station bit if any other capability is enabled

The 801.1AB spec states that the station only capability should
not be set in conjunction with any other (capability) bits.

It is possible for the lldpd interface update routine to set the
station bit, while the lldpd chassis update routine may, at a later
time, set the router capability bit. In this case the station bit
is not cleared.

This commit clears the station bit if any other capability is set.

client: install ZSH completions in $(datadir)/zsh/site-functions

Instead of Debian-specific location.

Fix #232.

interface-linux: fix this statement may fall through [-Wimplicit-fallthrough=] with gcc7

interface: fix this statement may fall through [-Wimplicit-fallthrough=] with gcc7

privsep_io: fix this statement may fall through [-Wimplicit-fallthrough=] with gcc7

doc: update link to Homebrew

build: fix get-version script when no git and no dist version

osx: update Homebrew recipe

release: prepare 0.9.7

ci: don't install jansson

doc: document previous commit in NEWS

Increase number of lldp neighbors to 32.

redhat: ensure /etc/sysconfig/lldpd is correctly populated

Patch courtesy of Martin Hauke. See: https://build.opensuse.org/request/show/477422

daemon: don't log when going to background

Most daemons do not log such information.

interfaces: vxlan interfaces should be ignored

interfaces: include <errno.h> to use errno

This fix previous commit. See:
 https://travis-ci.org/vincentbernat/lldpd/jobs/203148213

interfaces: don't log an hard error when interface is down

When an interface transitions to the down state, we may be woken up. We
don't want to pollute the logs with errors in this case.

lldp: don't use chassis TTL anymore

Move it to global configuration instead. This is less confusing than
having it at two different places.

lldp: attach remote TTL to port instead of chassis

When receiving a 0 TTL, we should trigger expiry of the remote port, not
the remote chassis. Therefore, TTL is moved to port, except for the
local chassis (we need it even when we don't have a port). So, for
remote ports, TTL is on the port. For local ports, it is on the
chassis. This fixes two cases:

 - a shutdown LLDPU should only expire one local port
 - when a port is refreshed, another port of the same chassis shouldn't
   be refreshed

lldpd: correctly increase discarded count

When a frame cannot be decoded but has been guessed, increase the
discarded count.

Fix #223.

travis: don't install libevent and net-snmp on OSX

They seem to be already present in Travis environment.

client: built-in JSON support

Instead of relying on two different libs to serialize JSON, we provide
our own built-in serializer. This reduces the amount of build-related
code (detection of the correct library), as well as the amount of C code
with the exception of UTF-8 handling which was stolen from CCAN.

This will also make the life of some users easier.

The JSON format stays the same: quite verbose with --enable-json0
flag. And inconsistent without it.

Fix #188.
Fix #220.

osx: update Homebrew recipe

doc: keyword harmonization for NEWS file

daemon: when daemonizing, close stdout/stderr

Otherwise, when used with systemd/upstart, we write logs in two
different places.

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