lib/atoms/port: fix segfaults with minimal information

When no packet has yet been received, querying LLDP information leads to
segfaults. Fix those by adding a couple pointer checks.

Signed-off-by: Bert van Hall <bert.vanhall@avionic-design.de>

lldpd: fix kernel version check

An erronous warning was displayed about kernel being too old while this
wasn't the case.

lldpcli: specify that custom TLV bytes are in hex format

build: test if libbsd is really usable

It is possible for libbsd to not be usable due to the use of
"-isystem". See for example:
 http://autobuild.buildroot.net/results/c8a/c8a6001f437701ecc75f6c9252935645bda8a8c8/lldpd-0.9.4/config.log

In this case, just don't use it.

doc: document the change about ethtool info grabbed without the monitor

build: ensure clang won't complain about documentation of linux headers

priv: don't use monitor for ethtool except for old kernels

Since 2.6.37, we don't need any privilege for ethtool GSET. Not using
the monitor will increase performance.

interface: use 10GBASE-CX4 for copper

That's the best approximation available in RFC4836.

interface: more media for *BSD

OpenBSD and FreeBSD didn't agree on symbol names. Use #ifdef everywhere.

lldp: update RFC3636 to RFC4836

Unfortunately, RFC4836 still lags behind. For example, no more than 10G
for speeds, no 10GBASE-T. For advertised modes, we try to be more
future-proof by using "other" for any unknown mode. Moreover, document
the fact that we advertise 10GBASE_T as 10GBASE-R and on Linux, we have
to guess when the port is fiber (and we guesss 10GBASE-X).

include: update linux/ethtool.h to 4.7

This is needed to support ETHTOOL_GLINKSETTINGS.

priv: don't use ethtool to get real MAC

It was used if /proc/net/bonding was not available but:

 1. it shouldn't happen
 2. some drivers don't have a permanent MAC
 3. we may override a user setting
 4. more code to maintaing

priv: document functions that should not use the monitor process

We should use them only if "oldies" are enabled.

NEWS: add an entry for 106aa50d

Revert "lldp: skip empty LLDP-MED inventory TLV instead"

This reverts commit aa357d2b6b54c966353bcee4d7f14640540a8cf2. The
previous behaviour was equivalent. Let's stick to that for now.

lldp: skip empty LLDP-MED inventory TLV instead

travis: fix issue with Travis and libtool

build: provide a better date format when using SOURCE_DATE_EPOCH

And define the appropriate date at configure time.

client: flush output after each block

This enables to get realtime output from "lldpcli watch".

See #187.

build: ensure typeof is correctly defined

This could be typeof/__typeof/__typeof__.

Fix #186.

build: make generation of atom-glue compatible with older gcc versions

With old versions, cpp doesn't accept several files as input. See #186.

release: prepare 0.9.4 for OSX

release: prepare 0.9.4 release

osx: update lldpd to 0.9.3

doc: document latest commit

lldp: add ability to control propagation of LLDPDU

In 802.1AB-2009, two additional target addresses were added to allow an
LLDPDU to not be stopped by some equipments. Expose this ability as a
configure command.

Fix #171

lldpcli: fix watch return code

When watch terminates normally (after hitting a limit), ensure lldpcli
exits with status code 0.

lldpcli: fix memory leak with use of readline

The lines returned by readline() should be freed.

interfaces: blacklist cdc_mbim

This exposes a weird interface.

lldpcli: add a test for return code

lldpcli: ensure we exit with an error on incorrect commands

lldpcli: fix manual page for port status

Use rx-only, tx-only, rx-and-tx instead of rxOnly, txOnly, rxAndTx. Also
add some tests.

tests: remove useless variable

build: fix build issue introduced in previous commit

$^ was used for a reason: automake would fix the paths used in
dependencies using VPATH or something similar. For some reason, this
even break with GNU make when not using OOT build. Long story short, we
use a pattern substitution to ensure that the atom files are looked up
in the correct directory.

This use of substitutions is mandated by POSIX (the 2013 version I
think). See:
 http://austingroupbugs.net/view.php?id=519

It is however believed that most make implementation had support for
this since a long time.

build: don't rely on GNU make $^ automatic variable

Not supported by BSD make. And not really needed either.

Fix #185.

bsd: fix use of ifmediareq for OpenBSD

They changed the struct. Other BSD didn't. Use of "typeof()" to
circumvent this.

http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys/net/if.h#rev1.167

cdp: don't use comma expression and ternary operator

This seems to confuse some versions of gcc.

event: fix formating of time_t

On OpenBSD, this is now a long long. Just cast to long long to be safe.

build: unsilent silent rules when running with V=1

lldp: accept LLDP frames sent through S-VLAN/C-VLAN bridges

See #171.

Merge pull request #184 from chutz/add-fcntl-and-getsockname-to-seccomp-whitelist

seccomp: add fcntl and getsockname to seccomp whitelist

seccomp: add fcntl and getsockname to seccomp whitelist

Recent versions of lldpd make calls to fcntl and getsockname, but they
are not in the seccomp whitelist. This patch adds them. Reported by Gentoo
users in these bugs:

https://bugs.gentoo.org/577576
https://bugs.gentoo.org/564566

build: display if instrumentation and code coverage are enabled

build: add a configure option to enable gcov

interfaces: fix setting of local value for port ID

This was broken in a403df6662cd506fbf7fee8a46ebbf209142ffd7 (0.9.1). Add
some integration tests to ensure it works as expected.

Fix #183.

doc: explain how to get coverage reports

The base-directory option seems a bit off?

tests: fix SONMP test whose result depends on indexes

lldpcli: display chassis TTL

Fix #180.

lib: allow retrieval of chassis TTL

lldpcli: remove unused variable in display_custom_tlvs()

lldpcli: display "Unknown TLVs" with a space

This should be an human readable string.

daemon: add a `-p` option to specify PID file

Fix #181

privsep: split privsep_io.c in 3 files

This enables us to express correct information about copyright.

debian: update debian/copyright

compat: ensure ranlib is happy on OSX by providing one symbol

We were already doing that but recent versions of ranlib became smarter
and don't fall in the trap of a static variable. Provide a non-static
version instead.

debian/redhat: prepare 0.9.3

tests: fix `lldpcli watch` XML test

tests: add test of `lldpcli watch`

client: add a way to limit the number of events received by "watch"

lib: remove unused variable

compat: use strtonum instead of atoi

When possible, use strtonum as it is more robust than atoi(). Also
replace some strtol() call when possible.

interfaces/linux: make veth special

veth is always a physical interface. However, it may be hard to detect
because when they are created, lower interface for the first one is none
and lower interface for the second one is the first one. Hence, no loop
detected, hence the second one is not considered as a physical
interface.

This is madness, just whitelist "veth".

netlink: don't complain of removal of inexistant IPv6 addresses

This happens with link-local addresses.

client: fix spurious warning in XML writer

It seems that libxml doesn't like generating an empty
document. Therefore, we initialize the writer only when we are sure we
will get content.

client: fix memory leak introduced n XML writer

client: remove memory leak introduced in JSON writers

doc: add an entry in NEWS for last two commits

client: flush XML object once we have one

See previous commit. This fixes `lldpcli -f xml watch`.

Fix: #179.

client: flush JSON object once it's done

This enables `lldpcli -f json watch` to work as expected. See #179.

travis: test json-c support in lldpcli as well

travis: no need for --with-{json,xml}

It's automatically enabled.

log: update manual page to reflect the situation

log: make a copy of va when logging to both stderr and syslog

On common platform, the copy is cheap. In case it isn't we note that the
copy doesn't happen if using a log handler and syslog cannot be enabled
with debug messages.

log: don't fallback to stderr for out-of-memory situation

We don't know if "ap" is still valid. Just don't do anything.

log: always log to stderr

Otherwise, error messages are not displayed on startup and this may
confuse users.

daemon: tell user to create user/group when missing

daemon: give the name of the control socket

Otherwise, people have to enable debug to get the information.

interfaces: ensure we don't break strict aliasing rule

Use an union to manipulate IPv4/IPv6 address. Other occurrences are
using memcpy (notably with "struct sockaddr_storage"). This is
preventive since gcc seems to rely more on this strict aliasing rule
since gcc-6.

Merge pull request #177 from jonasj76/fix-apparmor-build

Fix AppArmor "no" option in configure script

build: fix AppArmor "no" (default) option in configure script

Signed-off-by: Jonas Johansson <jonasj76@gmail.com>

tests: do not compile tests/decode unless running tests

Fix #175

Merge pull request #174 from jonasj76/fix_discard_uint32

Fix size of PEEK_DISCARD_UINT32()

Fix size of PEEK_DISCARD_UINT32()

Signed-off-by: Jonas Johansson <jonasj76@gmail.com>

doc: mention Windows agent "WinLLDPService"

lldpcli: remove bogus pasting error from manual page

debian: do not remove _lldpd user

While not strictly enforced, it is now considered better to not remove a
user on purge. We have no guarantee that the user is not owning some
files we didn't remove and those files could be attributed to another
user if the current user is removed. Moreover, less code.

travis: install libseccomp-dev as well

seccomp: fix a compilation warning with respect to uninitialized seccomp

travis: also test with seccomp enabled

This functionality has been untested for far too long.

build: don't include merges into changelog

travis: check we can build static lldpcli

build: silence libtool warning about using cru

Use "cr" instead. Snippet stolen from freeciv.

lib: use C preprocessor to build list of init functions

Because some features can be disabled, we cannot just parse the C file,
we need to run the C preprocessor on them. Hopefully, the GNU make stuff
(filter) should be portable enough.

lib: don't rely on atom-glue.h

This file is useless. Just use atom-glue.c.

build: ensure atom-glue.h is built first

Forcing direct dependencies is incompatible with VPATH. Use
BUILT_SOURCES instead.

lib: don't rely on constructors

Constructors do not work when compiled as a static libraries. Moreover,
some dynamic linkers have still support for constructors optional (for
example, uclibc). Since this can be tested only at runtime, this is not
possible to detect that during configure when crosscompiling.

Emulate this with our own glue code.

lldpd: don't log error string for missing _lldpd user

The additional error message (ENOENT) could just confuse the user.

tests/integration: only spawn a receive-only lldpd in first namespace

We don't need it to send anything.

osx: update HomeBrew recipe for 0.9.2

travis: remove coverity stuff

It's broken. Maybe it would be better to use clang-analyzer instead.