tests: request CAP_DAC_OVERRIDE

CAP_FOWNER is for being able to use chown/chmod. The permission we
need to ignore permissions is CAP_DAC_OVERRIDE. It is quite a large
permission, unfortunately.

tests: add SNMP-related tests

priv: always request CAP_FOWNER

While setting ifalias has some additional checks to ensure we can do
that with CAP_NET_ADMIN, we also need CAP_FOWNER to pass the sysfs
owner check. And we have to have both as the other test still needs to
pass.

priv: request CAP_FOWNER only if SNMP is enabled on command-line

priv: don't use %m in format string

This is not portable, use %s, even if we are on Linux.

debian/redhat: build-depends on libcap-dev/libcap-devel

tests: don't start lldpd twice in a namespace

tests: add a test for interface alias handling

build: also test directly for -lcap when no pkg-config

priv: when SNMP AgentX socket is filesystem-based, acquire CAP_FOWNER

priv: drop most privileges in monitor, only keep CAP_NET_RAW/ADMIN

On Linux, we mostly rely on CAP_NET_RAW. Only keep that one. However,
we also write to ifalias, which needs CAP_NET_ADMIN. We could let user
choose at runtime if they want to grant this capability or not.
Currently, a user can turn it on/off at any time.

Access to SNMP socket may also be problematic. We need some solid
solution about that before merging.

Is it safe to use the same UID for the monitored and the unprivileged
process? Signals are mostly harmless. As for ptrace, since the
monitored process as more capabilities, this will not be allowed by
Linux.

build: use lowercase for PKG_CHECK_MODULES

Otherwise, this is a bit ugly.

doc: add more instructions for Android

build: no fatal errors when compiling with embedded libevent on macOS

build: don't override CFLAGS when building libevent

We still want for example the minimal macOS version.

release: 1.0.1

daemon: use "mkdir -p" instead of "mkdir"

We don't use `RuntimeDirectory=` because it needs to be relative to
`/run`. As PRIVSEP_CHROOT can be any value, this is not possible.

Revert "daemon: use RuntimeDirectory= instead of mkdir to create chroot"

This reverts commit 478ec07ddc343e68d1691d551dba00919fa79d70.

daemon: use RuntimeDirectory= instead of mkdir to create chroot

This fix a bug introduced in b0ca585f627aa where "mkdir -p" was
replaced by "mkdir" which would fail on restart due to the directory
already existing.

build: try another way to circumvent macOS warning in libevent

build: also build with embedded libevent in OSX

build: don't be picky about deprecated stuff in libevent

This should fix:

    evutil_rand.c:177:2: error: 'arc4random_addrandom' is deprecated: first deprecated in macOS 10.12 - use arc4random_stir [-Werror,-Wdeprecated-declarations]

release: prepare 1.0.0

Merge pull request #274 from vincentbernat/feature/keep-some-ports

lldpd: add an option to keep some specified ports

Merge pull request #272 from westermo/fix-snmp-remTablesLastChange

handle lldpStatsRemTablesLastChangeTime correctly when items are removed

lldpd: add an option to keep some specified ports

A user can specify a pattern of ports to not delete even when they are
removed from the system. If a port is removed from the system and
match the pattern, it will be kept in memory.

tests: update tests for configuration keeping on port down/removed

client: remove previous way to set port description from documentation

test: add test for new portdescription setting

interfaces: fix out-of-bound access when getting permanent MAC address

Also, use a classic malloc() instead of a weird union. This makes
compilers better to catch this kind of error (at least on runtime with
sanitizers).

tests: don't crash if lldpd crashed

Otherwise, we don't get the appropriate information!

Merge pull request #273 from westermo/config-portdescr

client: add alternative way to configure port description

doc: add entry for portdescription in lldpcli manual page

Signed-off-by: Jonas Johansson <jonas.johansson@westermo.se>

client: add alternative way to configure port description

The existing port description CLI command only allows user configured
description if setting port id subtype to local. This patch introduces
an subtype independent way to configure port description.

Signed-off-by: Jonas Johansson <jonas.johansson@westermo.se>

handle lldpStatsRemTablesLastChangeTime correctly when items are removed

When a port is removed, the time has to be updated. The last removal time is
registered per local port, and this timestamp will be used as
lldpStatsRemTablesLastChangeTime if it is the latest timestamp.
Also, the lldpStatsRemTablesDeletes is  always increased when an entry in the
table is deleted.

Signed-off-by: Jonas Johansson <jonas.johansson@westermo.se>

netlink: fix non-updated management address

daemon: ensure interval update is taken into account fast

Merge pull request #271 from axjowa/fix/dot3-pd-pse-echo

Fix NULL pointer dereference in lldpd_dot3_power_pd_pse

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.