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.

interfaces: move bitmaps function to a dedicated file

Also add some simple tests.

snmp: additional fix around NetSNMP 5.8+ and function pointers

snmp: tentative to fix compilation with older versions of NetSNMP

In fact, gcc doesn't consider the signatures to be compatible when
using a function pointer. Let's use a preprocessor trick to detect the
version and hope gcc is OK with `char*` changed to `void*`.

interfaces: correctly handle bridges with VLAN filtering enabled

With VLAN-aware bridges, bridges are usually configured to filter
VLAN. In this case, we should not propagate VLAN from upper
interfaces.

Fix #377

daemon: remove unused interface types

tests: check we don't mix VLAN among interfaces

snmp: update signature for create_ostring

Again, this is backward compatible as an `const u_char*` can be casted
to `const void*`.

snmp: update signature of fmtaddr and send functions

This is backward compatible as a function accepting `const void *`
will also accept `void *`.

debian: update watch file

Merge pull request #376 from stannous/remote-vlan-limitations

Fix for limitation of 10 VLANs for LLDP .1q feature

tests: use "-Ln" to avoid logging with snmpd

This is more efficient than using "-Lf /dev/null". It was a tentative
to fix tests when running snmpd 5.8 but it seems to be related to
parallel run.

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