client: fix additional memory leaks detected by Coverity Scan

daemon: rewrite `mkdir_p()` to not use strtok/strcat

daemon: fix some coverity annotations

client: make it easier for Coverity to understand commands_new()

Never returning NULL is not enough to make Coverity understands we
don't leak anything. Remove the branch in commands_new() as it must
never happen, except for the root node.

build: allow to manually trigger Coverity Scan

daemon: enforce limits when receiving arbitrary-length data in privsep

interfaces: use an array of MAC addresses when defining supported protocols

In interfaces.c, we were handling it as a table while in lldpd.c, we
were copy-pasting the same condition three times. This was confusing
for analysis tools.

priv: abort on allocation error

This helps Coverity Scan.

daemon: annotate "daemonisation" to help Coverity

When daemonizing, we need to use /dev/null for stdin, stdout, and
stderr. If one of these file descriptors happen to be already closed,
we need to close the new file descriptor only if > 2. This is
confusing confusing for Coverity, annotate it correctly.

Not all locations are annotated because not all of them are detected,
for some reason.

client: do not allow memory allocations to fail

This leads to some memory leak (quite improbable), but we don't want
to continue running with an incomplete set of commands.

netlink: fix socket leak on some error condition

netlink: remove dead code, iface2 cannot be NULL inside the iterator

build: reenable Coverity Scan

protocols: fix more memory leak when decoding multiple TLVs

In a8d3c90feca5, some memory leaks were fixed when a TLV is present
multiple times. There were other occurrences in LLDP, CDP and EDP
handling. We ensure we free before overwriting with the new TLVs.

Fix #436

Eliminate unused arguments to priv_init() with --disable-privsep.

This seems a bit cleaner than passing dummy values (e.g. 0) that
aren't ever used.  Additionally, it means that PRIVSEP_CHROOT no
longer needs to be defined to a dummy value if building with
--disable-privsep.

Fix relative include paths to be more correct and portable.

Some relative include paths in subdirectories (src/daemon/protocols
and src/lib/atoms) were written relative to the parent directories
(src/daemon and src/lib).  This was okay in automake builds but
caused errors when porting to other build systems (for example,
Android make).

client: don't show age/via for local interfaces

client: display port administrative status with "show interfaces"

Fix #432.

tests: also test if we send interface description if present

interfaces: do not use interface alias if we set it ourselves

Fix #431

tests: add a test for "configure system interface description"

doc: tell content of lib should be in lib64 for Android

doc: tell minimum Android NDK this has been tested with

With ndk21r, `LD` should be set to `ld.lld`.

doc: update instructions for Android

Android NDK is a less and less capable toolchain. The standalone
toolchain is deprecated and the new way require you to override a
bunch of variables. Documentation also contains mistakes.

lldpd: override compiler and linker option strings

Compiler and linker option strings contains absolute path, so
replace them with disclaimer. Thos strings used only as
debug information to show not actual compilation options.

Signed-off-by: Oleksiy Obitotskyy <oobitots@cisco.com>

release: 1.0.8

doc: add NEWS entry about CVE-2020-27827

doc: update NEWS file

routing/linux: check IPv6 forwarding status when enabling Router capa

Consider also IPv6 when deciding whether to enable the Router capability.
This way, if a host is a router for IPv6 only, it will still be
advertised as Router to its neighbours.

Signed-off-by: Antonio Quartulli <a@unstable.cc>

tests: don't install libxml2 on MacOS

It is shipped and Homebrew will fail if we try to install it.

Merge pull request #423 from jow-/bugfix-prevent-macro-conflict

build: prevent conflict with official AX_LIB_READLINE macro

build: prevent conflict with official AX_LIB_READLINE macro

On systems where the official AX_LIB_READLINE (ax_lib_readline.m4) is
present in a globally shared autoconf include directory, auto(re)conf
will prefer including that offical version over the local variant due
to the offical macro having a higher serial number.

As a consequence, @READLINE_LIBS@ will not be substituted in *.in files,
eventually failing the compilation with errors similar to:

    gcc: error: READLINE_LIBS@: No such file or directory

Avoid this problem by renaming the incompatible local macro to
AX_LIB_READLINE_LLDPD which is sufficient to prevent any clashes.

We encountered this problem on OpenWrt which uses GNU autoconf-archive
to provide commonly used M4 macros through a global include directory,
which happens to ship AX_LIB_READLINE as well.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>

Merge pull request #422 from lldpd/fix/lldp-med-loc-parsing

lib: fix LLDP-MED location parsing in liblldpctl

lib: fix LLDP-MED location parsing in liblldpctl

Some bounds were not checked correctly when parsing LLDP-MED civic
location fields. This triggers out-of-bound reads (no write) in
lldpcli, ultimately leading to a crash.

Fix #420

Merge pull request #418 from orgcandman/memleak_fix

lldp: avoid memory leak from bad packets

lldp: avoid memory leak from bad packets

A packet that contains multiple instances of certain TLVs will cause
lldpd to continually allocate memory and leak the old memory.  As an
example, multiple instances of system name TLV will cause old values
to be dropped by the decoding routine.

Reported-at: https://github.com/openvswitch/ovs/pull/337
Reported-by: Jonas Rudloff <jonas.t.rudloff@gmail.com>
Signed-off-by: Aaron Conole <aconole@redhat.com>

doc: fix lldpd new URL

doc: move lldpd to a dedicated organization

github: move away from Travis to GitHub Actions

libevent: update to 2.1.12-stable

daemon: remove implicit fall-through

release: 1.0.7

Merge pull request #414 from vincentbernat/fix/missed-lldp-packets

interfaces: listen to all incoming packets on Linux, not just LLDP ones

interfaces: listen to all incoming packets on Linux, not just LLDP ones

This mostly reverts fc5526dae75f. Listening only on ETH_P_LLDP makes
us miss incoming packets on enslaved interfaces to an Open vSwitch.
Therefore, prefer listening to ETH_P_ALL instead of ETH_P_LLDP. It is
likely that enslaved interfaces do not fully process Ethernet packets
and `type` is not correctly filled.

Fix #413

tests: fix tests around XML by canonicalizing XML representation

Since Python 3.8, insertion order is respected for attributes, so we
cannot just compare strings as previously. Python 3.8 also introduces
a `canonicalize()` function to normalize XML for digital signature. We
apply this function if it exists.

linux: ignore outgoing packets at the socket level

Since Linux 4.20, it is possible to ignore outgoing packets. This is
already done like this on *BSD.

osx: update formula for HomeBrew (with upstream additions)

osx: update HomeBrew formula

release: 1.0.6

Merge pull request #410 from awujek/develop

#403 ports: make possible sending LLDP frames on a specified VLAN

ports: allow sending LLDP frames on a specified VLAN

The current limitation is that the frames are sent only on one VLAN per port.

client: fix spelling error

Fix #409

doc: add a NEWS entry for latest commit

lib: remove limit on system description length

The limit was introduced in 9c49cedf8e75 while fixing a memory leak.
The state data is used to ensure we don't interleave operations. We
need to handle the case where the value is truncated because it is
larger than the allocated size.

Fix #408.

ci: silent a bit `brew update`

It's quite verbose (and long...)

Merge pull request #401 from vincentbernat/fix/allowlist

code: remove use of blacklist/whitelist

ci: don't run unittests with clang

libcheck introduced a change in 0.15 which is incompatible with clang.

ci: use "brew bundle" instead of "brew install"

Otherwise, when a dependency is already installed and not up-to-date,
we get an error.

code: remove use of blacklist/whitelist

Use allowlist and denylist and adapt the documentation.

netlink: make "no name/no address" message a debug message

It seems it can happen on WLAN interfaces during AAA.

Fix #398

doc: fix wording about interface exclusion

Reported in #398.

build: add lldpd.stp to CLEANFILES

build: make unit tests work when systemtap support is enabled

travis: fix systemtap package name for dtrace support

libevent: upgrade to 2.1.11-stable

travis: check build with dtrace support

priv: fix missing semi-colon

build: fix syscall names extraction

build: ensure build process stops if not able to generate syscall-names.h

build: do not hardcode cpp invocation

This needs to be overridable by using $(CPP) instead. Patch from
Patrick McLean (Sony Interactive Entertainment Inc).

Merge pull request #393 from vincentbernat/fix/snmp-empty-sysname

agent: fix SNMP walk on lldpRemTable when missing remote sysName

agent: fix SNMP walk on lldpRemTable when missing remote sysName

When enumerating lldpRemSysName (and some others), one row could have
a NULL value because the remote system didn't provide a value. In this
case, we should return the next row.

There was already some code around that but it was not systematically
used. Therefore, we fix the issue for lldpRemTable and
lldpLocalSystemData. To ensure we catch future cases, we ensure
helpers functions use `default: return NULL` when no missing value is
allowed (no `break`, compiler would catch if it was the case) and
therefore, we don't need to try next OID and `default: break` when a
value may be missing and in this case, the caller should try next OID
upon receiving NULL.

Fix #392

Merge pull request #391 from Polynomial-C/1.0.5-gentoo_seccomp

a couple of necessary seccomp enhancements

v1.0.5: seccomp - add gettime

Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>

v1.0.1: seccomp - add brk

Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>

v0.9.5: seccomp - add socket ops

Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>

Merge pull request #388 from vincentbernat/fix/locked-conn

lib: introduce lldpctl_watch_callback2()

lib: introduce lldpctl_watch_callback2()

This is similar to `lldpctl_watch_callback()` (which is getting
deprecated), except the callback won't receive the current connection.
This prevents a user to use the connection which is unusable because
it is now dedicated to watch events.

Minor ABI dump due to new function, but everything is
backward-compatible, except you may now get an error if you use the
connection while watching (but this was already not supported).

Fix #380

lib: stricly-prevent use of a connection used to watch events

The protocol cannot handle received events mixed with regular
messages. Once the connection is dedicated to a watch, prevent its
regular use.

lib: emphasize `conn` cannot be used after lldpctl_watch_callback()

agent: simpler code with #ifdef

The use #ifdef to avoid some line duplications makes the code harder
to read. Prefer duplicating the lines when needed.

Revert "agent: fix crash on AgentX reconnect with NetSNMP 5.8"

This reverts commit c0b9ad5fc83e913e6ef761390c6211d6d240c212. This is
not needed: global and static variables are zero-initialized in C89.

agent: fix crash on AgentX reconnect with NetSNMP 5.8

Merge pull request #386 from vincentbernat/fix/chassisid-interface-flap

interfaces: don't modify chassis ID when overriden

interfaces: don't modify chassis ID when overriden

If the chassis ID has been overriden by the user, do not modify it.

Fix #385

tests: don't fail on __exit__ if we failed in __enter__

tests: fix bitmap test on 32bit

tests: do not check for MED inventory if /sys/class/dmi not available

tests: do not enable asan on anything else than x86_64

It seems buggy.

tests: display tests/test-suite.log on errors

build: provide a default.nix file

osx: update recipe for HomeBrew

release: 1.0.5

include: remove in6.h from linux/bridge.h

Otherwise, we are redefining in6_addr on old distributions.

tests: provide a Dockerfile to build with wheezy

This is for testing with a really old userland.

include: don't use __kernel_sa_family_t

It's not compatible with older kernels.

include: don't include linux/sysinfo.h

Older kernels don't have it. It doesn't seem useful.

interfaces: include "netinet/in.h" before kernel headers

This should ensure definition of stuff like in6_addr are done by the
libc, not by the kernel headers. Recent kernel headers know how to
handle that when loaded second.

interfaces: fix usage of inet_pton()

This would trigger a failure on RHEL 6.

travis: remove ARM64

This seems quite unreliable.