doc: explain which tags to use for Docker

Well, it is not totally true yet.

ci: fix ppc64el platform

ci: run coverity only once a week

ci: build and publish a Dockerfile

doc: change title format to use pound for markdown

tests: remove old Dockerfile.wheezy

It does not work anymore as wheezy is too old.

github: templates for bug report and feature request

And also add a link to discussions for generic questions.

github: add funding using BMC

lldpcli: require powerpairs for Dot3 power even when PD

This reverts commit 7056d802b9c595dd16f1354649bb7ae2f8b8880c. I may
have been hasty in removing this as my understanding is that the PSE
would impose the value to the PD, but it could be the other way
around. Some implementations do not like to have 0 as the value here
and will ignore the whole TLV.

Updates documentation for Android target
Adds specific compilation steps
Adds links to source and NDK
Adds installation script example

client: put lock file in the same directory as the socket

The whole deal I was trying to solve is the fact that I cannot put the
lock here because I was unprivileged. Just let lldpd create the lock
in the same way it creates the socket (same privileges).

tests: use a tmpfs for /var/lock and handle broken symlinks

tests: create /var/lock if it does not exist

client: use a dedicated file lock to prevent concurrent changes

We were using a lock on the Unix socket. This was working on Linux but
this is not portable. Therefore, we have to use a dedicated file for
this purpose. We use /var/lock by default.

We don't do a secure creation as the lock file is only opened in
append mode, so a symlink attack could only create empty file or reset
the timestamp of a file. No content can be erased this way.

Fix #445

release: 1.0.11

priv: explain why we don't use ethtool

I have tested on a few servers I have access to and none of them
supported the use of this ethtool command. So, keep using sysfs
instead (with the drawback that 1. debugfs should be mounted, 2. we
need to tell systemd we want to tune through sysfs).

priv: disable LLDP in firmware for Intel X7xx cards

This requires to configure systemd to not protect sysfs.

priv: ensure Intel E8xx cards can transmit LLDP packets

Fix #444

daemon: fix some use of "if defined"

`#if defined HOST_OS_FREEBSD || HOST_OS_NETBSD` is not the same as
`#if defined HOST_OS_FREEBSD || defined HOST_OS_NETBSD`. This doesn't
really matter in our case, but for consistency, fix these cases. Also,
don't use parentheses as they are not useful in our case (or we should
put them everywhere).

github: no need to be smart to show how to format code

People should see the markdown code, not the rendered code, from the
template.

github: explain people how to format lines of code

osx: update Homebrew recipe

release: 1.0.10

tests: catch problem with creation of chroot in tests

priv: fix chroot directory creation

The last component was not created!

osx: update version in Homebrew recipe

tests: fix a command in tests/lldpcli.conf

This file is not really used, but it contains a non-working command.

release: 1.0.9

osx: add license to HomeBrew recipe

osx: update URL for HomeBrew recipe

osx: handle empty groups correctly when looking for a free UID

This was already done this way with Homebrew. Fix #438.

build: cache coverity toolchain

daemon: fix mkdir_p() implementation

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