netlink: fix detection of VLAN 1

netlink: use an optimized version of num_bits_set

This is a classic problem. See
<https://stackoverflow.com/questions/109023/how-to-count-the-number-of-set-bits-in-a-32-bit-integer>

interfaces: fix for limitation of 10 VLANs for LLDP .1q feature

Max 10 VLAN ids are supported on a port for .1q feature

Root Cause: All the VLANs learnt from netlink is dropped after 10 VLANs due
to the static array allocation of only 10 VLAN ids in the interface structure.
Beyond 10 VLAN membership for a port are ignored and error message gets
printed causing flooding of messages when hundreds of VLANs are configured.

Fix: Changed the static VLAN id array to VLAN id bitmap. With the bitmap all 4k
VLANs can be stored and learnt from netlink messages.
- Added a message to indicate when the LLDP packet is not sent out because
its too big. This will be helpful for user when too many VLANs are
configured and LLDP packets are not sent out.

Limitation: Even though the VLAN learning from netlink messages has been
alleviated, due to the LLDP message size around 380 VLANs can be advertised in
the packet. This number can vary based on the number of other TLVs being
advertised by LLDP.

vlan info shows interface_name.vlan-id which makes it look like sub
interface and causes confusion

Root Cause: Vlan name is sent as part of the .1q TLVs. But, the vlan name
format was <nterface-name>.<vlan-id> which makes it look like a sub-interface.

Fix: The vlan name cannot be removed from the vlan-info display since it is
sent/received as part of the .1q TLV in LLDP packet. But, changed the
vlan name format to vlan-<vlan-id> to avoid the confusion.
(cherry picked from commit 38db598121f5ce615f98d6cdaf41d5360c40dc3c)

Conflicts:
src/daemon/interfaces-linux.c
src/daemon/lldpd.h

lib: don't break ABI by moving new config_tx_interval_ms to end

Merge pull request #379 from jp-t/master

Allow configuring tx-interval in milliseconds

lldpd: set a 30 seconds lower limit to the safeguard timer

This timer is a safeguard to refresh information about all networks
interfaces at regular interval, in case there was something wrong with
the event-based refresh.

When using milliseconds-grained tx-interval, this could happen several
times per second, which is not the intended use of the safeguard.

To quote Vincent Bernat:
"Minimal value of 30 seconds even if we don't have event-based
refresh (so outside the if). Most people have the event-based
refresh, so lowering the pace for the others should be pretty
invisible."

doc, tests: milliseconds tx-interval finishing touch

- Added a test in test_lldpcli.py to check that milliseconds delay can
  be read back in either seconds or milliseconds units.
- Updated the manual page for lldpcli; warn about performance issue.
- Added an entry in NEWS

snmp agent: fix tests suite for milliseconds units

Allow configuring tx-interval in milliseconds

This is a prerequisite for TTDP support.

Usage: add "ms" suffix to the delay value to force interpretation as a
milliseconds value:

 lldpcli configure lldp tx-interval 200ms

When computing a TTL (TTL = tx-interval * tx-hold), the result is rounded
up to the next second, as LLDP frames need a value in seconds.

A new tx-interval-ms item is added to the json output. The old tx-interval
is kept in seconds (rounded up) for compatibility.

tests: remove hack around libtool on OSX

tests: skip tests requiring kernel features not present

Notably, "team" is not available in GCP kernels...

travis: update to Bionic

travis: also build for arm64

Merge pull request #375 from stannous/master

Fix for LLDP related netlink error messages

Fix for LLDP related netlink error messages

Ticket: CM-27243
Reviewed By: CCR-9608
Testing Done: Unit tseting, PTM Smoke

Issue: Error messages "netlink: 8 bytes leftover after parsing attributes
in process `lldpd'"

Root cause: Root Cause: The length of the netlink message was not being set
properly for non-bridge family type messages. Same length was being used for
both type of messages even though bridge family type message has extra
attribute. This causes 8 extra bytes being left over in the non-bridge
family type netlink messages.

Fix: Calculating and setting the length separately for bridge and non-bridge
family type messages.
(cherry picked from commit aac76966539bf932d5923b165762db370990bf94)

Revert "client: remove useless variable declaration in zsh completion"

This reverts commit e37dd8d0a3f67b345681b46c175249eee286bed0.

client: fix zsh completion to not alter global environment

client: remove useless variable declaration in zsh completion

LLDPD should document system refresh timer (tx-interval * 20)

In LLDPD, each port has its own timer to catch port-related
changes and is modified by changing the tx-interval.

LLDPD also starts another system based refresh timer on each port
for changes like hostname. This is the tx-interval multiplied by
20.  This needs to be documented.

Signed-off-by: Sam Tannous <stannous@cumulusnetworks.com>

lldp: don't discard the whole LLDPDU when only one TLV is invalid

IEEE802.1AB-2005 says:

> If TLV_type_value is in the range of reserved TLV types in Table
> 9-1, the TLV is unrecognized and may be a basic TLV from a later
> LLDP version. The statsTLVsUnrecognizedTotal counter shall be
> incremented, and the TLV shall be assumed to be validated.

Fix #323

tests: add a test with different MTU interfaces

See #365.

redhat: fix license name in spec file

lib: fix memory leak when handling I/O

The state data is used to ensure we don't interleave requests of the
same kind (eg requesting data for eth0, then for eth1 while eth0 is
running). The data was freed only when reaching `CONN_STATE_IDLE`
again. Otherwise, there was a memory leak.

To avoid the memory leak, we avoid use a static allocation instead.

Fix #362.

lldpcli: ask for power pairs only when PSE

Merge pull request #350 from patrikdanielssonaxis/8023bt

lldp: Power via MDI TLV: Initial support for 802.3bt

test: add tests for 802.3BT additions

test: use scapy when sending pcap files

lldp: Power via MDI TLV: Initial support for 802.3bt

Change-Id: Ie2cb521a2433cee64c91e5146050814276931125

Merge branch 'feature/vlan-enabled-bridge'

lldp: when receiving a shutdown LLDPU, don't clear chassis information

The chassis may be shared with another port. When the MSAP is known
and we receive a shutdown LLDPDU, just leave the original chassis as
is instead of copying information from the new chassis to the old
chassis.

Fix #348.

lldp: validate a bit more received LLDP frames

Notably, we ensure the order and unicity of Chassis ID, Port ID and
TTL TLV. For Chassis ID and Port ID, we also ensure the maximum size
does not exceed 256.

Fix #351.

interfaces: only register protocol handler for LLDP when only LLDP enabled

On Linux, the drop counter is increased on unhandled packets. We are
using a raw socket with ETH_P_ALL, so we get a copy of the packet. The
original packet is ultimately dropped later and this increases the
drop counter associated to the interface on Linux.

When listening only to LLDP, use ETH_P_LLDP instead of ETH_P_ALL to
avoid this.

Fix #296.

lldpctl: don't reopen control socket each time we want to lock

lldpctl: put a lock around some commands to avoid race conditions

Fix #343

log: don't use black for coloring DBG

As foreground, it may be gray, but it could be too dark. Use dark cyan
instead.

netlink: support VLAN-aware bridges

netlink: ignore IFLA_LINK if we previously received IFLA_LINK_NETNSID

Starting from 5.2 (commit feadc4b6cf42), IFLA_LINK appear after
IFLA_LINK_NETNSID. Handle this case correctly.

interfaces: don't log errno when detecting loops

tests: fix skip instruction

tests: don't use pytest.config

It's deprecated in 5.0.

interfaces: enable matching on interface name for management address

We allow the user to match a management address using the interface
name by specifying the interface name as a pattern. The same rules as
for specifying IP patterns apply but there is no notion of exact match
for an interface.

Fix #349.

lldpd: fix bitfield

client: fix parameter order

lib: fix memory leak

fix typo

lib: fix enumeration

build: disable warnings on cast alignments

clang is often wront about it (it increases alignment requirement, but
the surrounding structure ensure the alignment is correct). Dunno if
gcc is smarter or just ignore most of these problems.

tests: be verbose when running make on CI when an error happens

Otherwise, we don't really know why we get errors.

client: don't display management interface index if unknown

interfaces: compute interface index for fixed management address

When management address is provided without a pattern, fetch the
appropriate interface index if the interface is known. Thanks to
@kefins for the actual patch.

Fix #338.

lib: expose management interface index

lib: fix missing words in lldpctl.h

include: update headers to 4.19

It makes things compile correctly with Jessie

June 15th 2019 was a Saturday, not a Sunday

This addresses the following issue,

RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.tUWJAJ (%prep)
    bogus date in %changelog: Sun Jun 15 2019 Vincent Bernat <bernat@luffy.cx> - 1.0.4-1

osx: update recipe for HomeBrew

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