release: 1.0.4

lldpd: document expectation for LLDP-MED device configuration

When using class II and class III endpoints, some additional TLVs are
mandatory. This is not enforced by lldpd.

See #335.

netlink: make the netlink socket totally non blocking

See #333.

netlink: handle blocking read from netlink socket

It seems it is possible to run into a condition where the netlink
socket is not available for read. Set the MSG_DONTWAIT flag and fetch
an error if there is any.

Fix #333

snmp: implement lldpRemOrgDefInfoTable for remote custom TLVs

As a simplification, lldpRemOrgDefInfoIndex is 1 for the first custom
TLV of a given port and is increased by 1 for each new TLV. This is
not what is encouraged in the MIB:

> An agent is encouraged to assign monotonically increasing index
> values to new entries, starting with one, after each reboot. It is
> considered unlikely that the lldpRemOrgDefInfoIndex will wrap
> between reboots.

However, it is simpler to implement it this way as we don't need to
record the index inside the `lldpd_custom` structure. Also, the index
will increase even for a different OUI or subtype as we do not want to
sort the custom TLVs.

Fix #330.

debian: depends on lsb-base

This is needed for SysV init script

debian: no need to BD on lsb-release anymore

doc: update Android instructions for ARM64

Nowadays, this is a more common platform than ARM.

client: use bold instead of a color for command completions

The color needs to be readable on both light and dark backgrounds and
should be readable on most themes. 1;35m would be a fit, but let's
stay safe by just using bold.

Fix #326

agent: do not increase statsAgeoutsTotal when receiving a shutdown frame

Fix #325

lldp: increase statsTLVsUnrecognizedTotal on unknown TLV

This was done for organization TLVs, but not for other TLVs.

Fix #323.

tests: update pytest

And also, don't specify too many dependencies, just the essential
ones.

client: rename max-neighbor command

tests: add test for global configuration settings

client: ability to set maximum of neighbors from lldpcli

priv: include limits.h for PATH_MAX

lib: use an unique variable as iterator in foreach macro

This lessen the chance of the `iter` variable to shadow a user-defined
variable. This is also a tentative to help #312, even if the scope of
the `iter` variable should ensure we can nest two loops without any
issue.

build: warn on implicit fallthrough for case statements

build: enable increased reliability of stack overflow detection

See https://developers.redhat.com/blog/2018/03/21/compiler-and-linker-flags-gcc/

interfaces/linux: make dsa special

There used to be specific exemptions carved out for "veth" and "dsa",
which were removed in b8db52bd7c7d ("interfaces/linux: blacklist some
drivers instead of whitelisting"). "veth" was restored in 2958b9d48940
("interfaces/linux: make veth special"). This commit restores the
whitelist for dsa devices as well.

build: fix compilation with `--enable-fdp --disable-cdp`

Fix #315

daemon: allow lseek for seccomp

osx: update version in HomeBrew formula

release: 1.0.3

daemon: fix creation of chroot directory

Because the current path was uninitialized, we were creating
semi-random directory at the root of the filesystem.

osx: update version in HomeBrew formula

osx: sync HomeBrew formula

daemon: check if started through systemd first

When running with `-d`, we may still need to send a notification to
systemd, even if we didn't plan to daemonize.

Fix #309.

daemon: remove unused agent_default_agentx_socket() function

It's not needed anymore since 83577a2b3b88 and on older versions of
Net-SNMP, it uses an unexposed directive.

release: 1.0.2

daemon: don't enable ProtectSystem by default

If the chroot is in `/usr` (like `/usr/local/var/run/lldpd` which is
the default), neither systemd nor lldpd will be able to create and
write to it. This may be solved with `ReadWritePaths` (unsure if it
would create the directory), but this doesn't exist in older versions
of systemd.

Just comment the directive to let people know it exists and should
work in most cases.

Fix #308

interfaces: remove specific handling for bonds except with --enable-oldies

Starting from Linux 4.19, LLDP packets are transmitted back to the
bond devices and it seems the original interface is lost in the
process. Therefore, packets are duplicated to both members. Upstream
commit is:

    commit 6a9e461f6fe4434e6172304b69774daff9a3ac4c
    Author: Mahesh Bandewar <maheshb@google.com>
    Date:   Mon Sep 24 14:39:42 2018 -0700

    bonding: pass link-local packets to bonding master also.

    Commit b89f04c61efe ("bonding: deliver link-local packets with
    skb->dev set to link that packets arrived on") changed the behavior
    of how link-local-multicast packets are processed. The change in
    the behavior broke some legacy use cases where these packets are
    expected to arrive on bonding master device also.

    This patch passes the packet to the stack with the link it arrived
    on as well as passes to the bonding-master device to preserve the
    legacy use case.

Fixes: b89f04c61efe ("bonding: deliver link-local packets with skb->dev set to link that packets arrived on")
Reported-by: Michal Soltys <soltys@ziu.info>
Signed-off-by: Mahesh Bandewar <maheshb@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
The code to handle bond devices is not needed since Linux 2.6.27.
Therefore, move it to the `--enable-oldies` option.

Fix #306.

doc: update documentation for Android

See #305 for more details.

Allow linux kernel to autoprobe correct socket_nl.nl_pid address

daemon: do not explicitely inline functions

As we are using `-Winline`, if it fails, we get a warning. Let the
compiler decide if something has to be inlined. As we use only static
functions, it should be easy to inline if possible.

build: ability to disable libbsd with --without-libbsd

This is useful when user wants to ensure reproducibility of the build
whatever libbsd is present or not.

marshal: avoid NULL pointer arithmetic

This is UB and clang warns about this. Ensure we use an int of the
same size as a pointer and convert it directly.

lib: use constants from lldp-const.h for Dot3 MAU

dot3: fix 100BASE-T4 MAU

Fix #297

interfaces: provide clearer messages about ethtool problems

Logs can confuse people on how important a problem is. Tell them
exactly what this would affect.

daemon: implement mkdir -p directly in lldpd

It's difficult to know the path to mkdir. If we use the one from
autoconf (@mkdir_p@), we get the path from the host, not the target.
If we hardcode `/bin/mkdir`, we may not work on platforms like NixOS.

See https://github.com/NixOS/nixpkgs/issues/44507.

doc: add latest commit to NEWS

Merge pull request #289 from GustavWi/master

Add support for CDP PD PoE negotiation.

Add support for PD PoE negotiation.

Power requests refer to the power at the PSE.
Thus the loss offset caused by the cable has to be added
to the power request. Also the power received from the PSE
must subtract the cable loss to be compatible with lldp.
There are three TLVs for CDPv2 PoE negotiation.
Power Consumption: Current maximum power consumption of PD.
Power Request: Wanted maximum power consumption of PD.
Power Available: Power output from PSE.

Only used if lldp PoE is not supported by switch.

A cisco switch which does support both lldp and cdp will
use the protocol which is first to transmit a package.

Change-Id: Ib45ae582799c50a8d839120c28a5080e582c66e9

redhat: install lldpcli as setuid _lldpd

osx: update brew formula

Merge pull request #285 from vincentbernat/feature/linux-caps

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

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.

Merge pull request #286 from GustavWi/master

Read all notifications in lldpctl_recv.

Read all notifications in lldpctl_recv.

Can otherwise lead to unbounded growth in input_buffer if
lldp devices send notifications simultaneously thus
a socket callback contains multiple notifications
and only the first one is cleared. This leads to continous
growth of the input buffer and will crash the system.

Change-Id: I3331de2d8201f574a91b8e8f2d51a98010f7a7ab

lib: document the inability to do anything else when processing notifications

The code doesn't handle notifications with regular operations on the
same connection. Tell the user to open a another connection.

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…