build: check runtime support of constructor/destructor

With some libc, we may have a compile-time support (through GCC), but
not a runtime support. For example, with uclibc, support for
constructors/destructors is optional. They can be enabled with
`CFLAGS=--uclibc-ctors` on some versions. On some others, they need to
be builtin by compiling uclibc with UCLIBC_CTOR_DTOR.

Unfortunately, cannot be tested when cross-compilation is in effect.

build: check for support for constructor/destructor function attributes

README: add a word about `--enable-oldies`

tests: add missing include for `read()`

tests: fix SNMP tests

Restore the `tohex()` function in `check_snmp.c`. Use a simpler version
in `decode.c`.

protocols: don't use assert on paths that can be reached

Malformed packets should not make lldpd crash. Ensure we can handle them
by not using assert() in this part.

lldp: fix small indentation issue

NEWS: add entry for last commit

lldp: fix a buffer overflow when handling management address TLV

When a remote device was advertising a too large management address
while still respecting TLV boundaries, lldpd would crash due to a buffer
overflow. However, the buffer being a static one, this buffer overflow
is not exploitable if hardening was not disabled. This bug exists since
version 0.5.6.

tests: add instructions on how to use afl for fuzzing

tests: add a simple program to test frame decoding

lldpd: remove outdated comment about custom TLV

debian: only enable systemd unit file on system recent enough

interfaces: remove spurious #ifdef directive

interfaces: remove old bridge code

It has been ages since sysfs contains the interface for querying a
bridge (2.6.7). And nowadays, sysfs is always enabled.

Merge pull request #138 from commodo/copy-paste-fix

client: fix copy+paste for some vars

client: fix copy+paste for some vars

Only affects logging.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

debian: on old Debian releases, enable "oldies"

doc: add a NEWS entry for latest commit

lldpd: add a warning if the kernel seems to be too old

redhat: on RHEL 6, enable oldies support

build: declare kernels before 2.6.39 "too old"

Unfortunately, netlink support for bond devices has been introduced in
2.6.33 (commit 88ead977109da926a03068e277869ea8fedd170d). For bridge
devices, this is an addition of Linux 2.6.39 (commit
bb900b27a2f49b37bc38c08e656ea13048fee13b).

cdp: fix log message spelling

redhat: for RHEL 7 and CentOS 7, install systemd unit file

build: fool autoreconf to do a recursive reconfiguration

We just redefine AC_CONFIG_SUBDIRS to our own empty version. autoreconf
will see this during the tracing step and will automatically reconfigure
libevent and libnl as well. This may unbreak people relying on tool
trying to regen configure without using the "autogen.sh" script.

Fix #133 (with a bit of luck)

release: 0.7.17 released

build: .git may be a file

It happens when using a worktree.

build: tell seccomp is experimental

I don't think many people are using it and the risk for lldpd to not
work is therefore increased.

apparmor: fix profile so that it works on Ubuntu 15.04

apparmor: provide an apparmor profile

Currently, lldpd has to be installed in /usr/sbin/lldpd. Will change
that later.

priv: no need to access bridge files unless "oldies"

However, bonding stuff is still needed because we get the permanence MAC
address this way.

lib: map_reverse_lookup needs to be always available

Ability to enable/disable RX/TX individually on each port

lldpcli: fix per-port occurrences of "conf lldp"

Some commands were documented as not accepting a per-port setting while
they accepted it (without any effect). On the other hand, some other
commands were accepting and used per-port setting while not
documented. Fix all that.

lldpd: ensure retransmit is triggered when a port goes up

lldpd: fix a memory leak in the way interfaces were cloned

interfaces: clean neighbors if an interface goes down

As we don't delete down interfaces anymore, we must at least clear
neighbors correctly.

interfaces: don't discard down interfaces

This is an easy way to keep the configuration associated to those
interfaces. However, we still need to discard any neighbors in this
case.

lib/atom: only allow custom TLV modification for local ports

Add a default local port as a template for any future local port.

This default local port can be modified through the library and any
future ports will use it as a template.

client: factor port modification in a dedicated function

marshal: add a macro to repair TQ lists

NEWS: add some entries for 0.7.17

lldpcli: display a warning when a configuration file is not found

Otherwise, the output of lldpcli would look pretty empty.

netlink: make error message more explicit

But there is some other reason behind it. This needs to be investigated
at some point.

snmp: make use of newer snmp_select_info2() function

When available, it allows to not hit the 1024 FD limit of
snmp_select_info().

interfaces: fix incomplete declaration of interfaces_cleanup

For BSD and Solaris, the declaration was incomplete.

build: ignore libnl documentation errors

build: fix test building with libnl dependency

build: better find system libnl3

build: don't limit dist tarball files to 99 characters

Otherwise, some files from libnl will be missing.

build: don't forget to ship libnl directory too

At least, on Linux. Therefore, dist tarballs should now be released on
Linux (otherwise, libnl won't be shipped).

build: ensure we can build libnl with -Wall -Werror

We need to silence out unused variables, in this case.

build: only test for libnl on Linux

travis: install libnl-3-dev

Even if it will not be used due to being too old.

netlink: workaround veth being peered together through IFLA_LINK

This is a change introduced in Linux 4.1. It's easy to detect, so,
workaround it. See:

    commit a45253bf32bf49cdb2807bad212b84f5ab51ac26
    Author: Nicolas Dichtel <nicolas.dichtel@6wind.com>
    Date:   Thu Apr 2 17:07:11 2015 +0200

        veth: set iflink to the peer veth

        Now that the peer netns is advertised in rtnl messages, we can set this property
        so that IFLA_LINK will advertise the peer ifindex. It allows the userland to get
        the full veth configuration.

Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

netlink: use libnl3 instead of custom netlink code

The main goal of this change is to implement a caching system for
netlink and avoiding the full scan done when a change was
triggered. Implementing a netlink cache is not as funny as it seems and
therefore, it seems just better to use libnl3.

Licensing issues are explained in README. People concerned with that
should just use dynamic linking. An embedded copy of libnl3 is also
provided, just like libevent.

build: add a marker at the end of third-party configure

build: make --with-embedded-libevent a tri-state switch

It defaults to auto (same as before) but not embedding libevent can be
enforced with `--without-embedded-libevent`.

build: don't let autoreconf rely on AC_CONFIG_SUBDIRS macro

We don't use it anymore.

build: automatically reuse regular configure args for distcheck

build: run ./configure for libevent even when not needed

We need it for "make dist"...

build: only enable silent rules if they are not disabled

build: replace AC_CONFIG_SUBDIRS with custom version

Modifying `$ac_configure_args` breaks maintainer mode. Moreover,
./configure is run even when not needed. We use a custom version
allowing to pass custom parameters and not delaying ./configure
invocation.

cdp: compute default platform when updating the chassis

Like for other data, collect the default platform name for CDP in
`lldpd_update_localchassis()`. Code in `cdp.c` should be independant of
the environment. Also, fix the unittests.

cdp: make the default platform name really the kernel name

Instead of hardcoding 'Linux" directly query the kernel name of the user
is not overwriting it.

build: propagate appropriate build flags in src/compat

osx: remove unneeded patch for launchd

It is not needed since 0.7.6.

osx: fix configure command line in Homebrew

The array concat operator has an higher precedence than the ternary
operator in Ruby.

osx: restore use of _lldpd/_lldpd user/group in Homebrew

This has been wrongly removed in a previous commit. Homebrew don't want
to do that, but I feel this is better to keep that here.

osx: synchronize a bit with official HomeBrew formula

README: incorrect instructions for OS X

A backslash was missing.

debian: update symbol versioning

Since all the work is now done in the `.map` file, we don't need to say
more here. As the symbols were previously unversioned, we have to start
both generations at 0.7.16.

client: parse options only once

getopt() will parse them in order. If a user specifies `-d` after `-c`,
they will be ignored but this seems like a corner case.

client: make lldpctl also accepts `-u` flag

Close: #129

Merge pull request #128 from baloo/patch-1

Typo in README.md

Typo in README.md

debian: don't complain about too recent Standards-Version

debian: add a lintian override for debian-revision-should-not-be-zero

This is expected.

redhat: on RHEL 5, use --enable-oldies

Kernel is too old to support everything we need (2.6.18).

travis: also test when oldies are enabled

travis: don't add too many alternatives in configure

release: prepare 0.7.16

lib: accept "language" as a valid MED location type

Closes #127

lldpd: add version string to "starting" log entry

Merge pull request #126 from idryzhov/master

daemon: fix missing parenthesis

daemon: fix missing parenthesis

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

Merge pull request #124 from commodo/rename-unconfig-custom-tlvs-cmd

lldpcli: rename "unconfigure lldp custom-tlvs" to "unconfigure lldp custom-tlv"

lldpcli: rename "unconfigure lldp custom-tlvs" to "unconfigure lldp custom-tlv"

There was a bit of confusion regarding this command.
It's documented as 'unconfigure lldp custom-tlv', when in fact it's
'unconfigure lldp custom-tlvs' (plural).

That's also my fault, since I did not update much (or take care about)
documentation.
It seems simpler to rename the command as is in the doc than change
the doc.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

lldpcli: remove spurious space before port aggregation

netlink: use netlink to retrieve bridge/bond/vlan information

On Linux, Netlink is more efficient to get this kind of information. Use
it all the time, unless we are configured to run on old kernels. In this
case, fallback to the previous functions.

An old kernel is now < 2.6.32 (RHEL 6, Ubuntu Lucid, Debian Squeeze).

build: add a note to remember why setproctitle may not work

On libbsd < 0.5, this is only a stub function. So, it is expected to do
nothing on OS like Ubuntu Precise.

doc: fix latest NEWS update

0.7.16 was not released yet.

lib: fix memory leak when handling MED location data

When formatting fixed precision data, the result was copied into the
atom but the original wasn't freed.

doc: add a NEWS entry for the chassis-related stuff

lldpcli: add "show chassis" command

This demonstrates the use of `lldpctl_get_local_chassis()` function.

lldpcli: get chassis atom to query chassis-related information

Don't rely on the previously introduced compatibility layer. There is no
plan to deprecate this layer, but as an example, we prefer to show how
stuff should be done.

Revert "lib: bump ABI"

This reverts commit 8ee9bc2cec66b080746372e0a7b5f3ec1363b694. Thanks to
the compatibility layer, we can only do a minor ABI bump.

lib: add compatibility layer with chassis-related information

When querying a port, if we don't find the requested information,
check if the chassis has the information as well.

lib: bump ABI

Due to the absence of compatibility layer, all atom-related functions
are considered incompatible since they don't work on ports like they
used too. This is not strictly an ABI incompatibility, but we consider
the change important enough to do like it was.

This commit will be reverted once the compatibility layer is here.