Fix NULL pointer dereference in lldpd_dot3_power_pd_pse

selected_port might be NULL after the loop if, for example,
no PSE was found among the neighbours.

This fixes a crash that occured with a device connected to
a non-PoE, LLDP-enabled switch via a PoE midspan.

build: don't enable sanitizers and safe-stack at the same time

They are not compatible. At least on clang, they both try to
instrument the same functions.

interfaces: explain why we cannot get the permanent MAC address

build: isolate SNMP-related files during compilation

@NETSNMP_CFLAGS@ is likely to contain some extensively damaged
flags. Ensure we use them only to compile SNMP-related files.

This should fix #231.

lib: lldpctl_k_config_paused is writable

doc: add ./autogen.sh to build instructions

But stress users should look at the website instead.

Fix #268.

osx: use "squiggly" heredoc

The least indented line is use as a base. Ruby 2.3.

Merge pull request #266 from dennypage/master

Add missing options to usage

Add missing options to usage

client: change description for chassis ID override

daemon: move vfork/fork handling directly in lldpd.c

This is not needed elsewhere. This way, we ensure redefinition of
vfork() to fork() through a define is working as expected, even when
some system headers may try to rename vfork too.

Fix #264

Merge pull request #263 from vic-lin-accton/master

Check if there is user defined string before use chassis name for cha…

lib: ability to configure a local chassis ID

Also, remove the `-c` flag for lldpd that was previously introduced.

daemon: ensure chassis-related changes are updated immediately

lib: move new lldpctl_k_config_cid_string to end of section

Otherwise, ABI would change.

lldpd: check for defined string before use chassis name for chassis id

osx: update Homebrew formula

daemon: don't use @mkdir_p@ substitution for systemd service file

It is the path to the host mkdir, not to the target mkdir. Just use
/bin/mkdir.

configure: remove check on CXX compiler

lldpd fails to build if the toolchain doesn't have a C++ compiler
because configure fails with the following error:

  checking how to run the C++ preprocessor... /lib/cpp
  configure: error: in `/home/dkc/src/buildroot/build-zii/build/lldpd-0.9.4':
  configure: error: C++ preprocessor "/lib/cpp" fails sanity check

Since "8d92800b: build: cleaner way to not alter CFLAGS/CPPFLAGS/LDFLAGS",
it seems that the dependency on C++ is not required anymore, so there
is no reason to keep this restriction. Dropping AC_PROG_CXX allows to
build with a toolchain that doesn't have C++ just fine.

doc: add a word about Intel X710 with embedded LLDP daemon

doc: rename OS X to macOS

LICENSE: add title and copyright notice (#259)

* LICENSE: add title and copyright notice

The title is not legally mandated, but it's recommended in the license template text (see http://choosealicense.com/licenses/isc/ and https://opensource.org/licenses/isc-license).

The copyright notice is required, IIRC.

* fix date in copyright notice

* add notice about additional licensing terms

release: force S3 deployment whatever the branch is

release: prepare 0.9.9

dot3: as PD device, echo back PSE allocated value

Dot3 power TLV contains an allocated value and a requested value. When
PSE allocates some power and says so in its TLV, PD device is expected
to echo back (within 10 seconds) the received value in its own TLV. We
handle this part automatically.

Fix #243
Fix #249

travis: upload OSX build to some S3 bucket

Still a WIP.

priv: also allow the use of openat for seccomp

Merge pull request #253 from gregoryp/document_json0_in_cli

lldpcli: document json0 output format

lldpcli: document json0 output format

Merge pull request #251 from vincentbernat/fix/reap-children

priv: correctly handle lldpcli exit

priv: provide a simpler sig_chld when priv sep is disabled

We restore the original sig_chld() helper function when privilege
separation is enabled but we use a very simple one when not. This
should still fix the zombie issue.

priv: correctly handle lldpcli exit

When lldpcli exits, we must acknowledge its death with
`waitpid()`. There were two missing cases:

 - when lldpcli exits before setting the SIG_CHLD signal
 - when privilege separation was not configured

For the first case, we call the function associated to the signal
ourselves after the signal is configured. This function is idempotent.

For the second case, we setup the SIG_CHLD signal even when privilege
separation is disabled.

Fix #250

daemon: don't fork at all when using upstart

It seems there is a bug in Upstart state machine. If the process fail
before daemonizing, Upstart says the process will be respawned but that's
not the case. Moreover, Upstart has difficulty to correctly track
daemonization of lldpd (it is confused by the additional process
spawning after initial daemonization, for some reason?).

Fix #245 (for real)

daemon: don't use SIGSTOP when using Upstart

This is buggy because if lldpd happens to stop before sending SIGSTOP,
Upstart won't respawn it. See:
 https://bugs.launchpad.net/upstart/+bug/712351

Instead, let Upstart detects the double fork. This should work just
fine as there is no "trap" before daemonizing.

Fix #245

Merge pull request #246 from vincentbernat/feature/safestack

build: enable clang safe-stack feature if available

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