release: 0.7.19 released

osx: update HomeBrew recipe

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.

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.

release: 0.7.18 released

interfaces: restore bridge and bond detection without netlink

With 2.6.32, it's not possible to use netlink for that.

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.

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.

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: 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>

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.

lib: ability to get and query local chassis

Up until now, to get information about the local chassis, we had to get
one interface and query the associated local port, then get chassis
information out of it.

With this change, a dedicated `lldpctl_get_local_chassis()` function is
provided and returns a chassis atom. All the chassis-related properties
are moved to this kind of atom. We also add a `lldpctl_k_port_chassis`
key to get the chassis associated to a port.

Left for next commits:
 - lldpcli hasn't been updated
 - compatibility layer to query chassis-related properties from port

build: fix URL to library versioning documentation

lldpcli: move "conf sys bond-slave-src-mac-type" in man page

Merge pull request #119 from kcgthb/master

redhat: fix minor issues in .spec file

redhat: fix minor issues in .spec file

- fix BuildRequires package name (pkgconfig instead of pkg-config)

build: fix help string for --enable-hardening for consistency

Closes #118.

build: ship lldpctl.map in tarball

build: use symbol versioning for liblldpctl

Merge pull request #115 from alagoutte/travis_container

Don't use container-based Travis CI

Merge pull request #114 from alagoutte/clang

Add -Wdocumentation (Clang CFLAG) and fix warning

Don't use container-based Travis CI

Add -Wheader-guard and -Wdocumentation to CFLAG (Use by Clang)

Disable -Wdocumentation for some include using pragma

osx: tell the build fix will be here in 0.7.16.

osx/homebrew: use --without-json and --without-snmp to avoid autodetection

build: create parent directory before editing files

osx: add appropriate dependency on pre/postinstall scripts

osx: ensure pre/postinstall scripts are created correctly

Because we declare the scripts/*install target in `edit.am` and in
`Makefile.am`, only one of them is used. With automake, it doesn't seem
possible to easily extend a target. Instead, do the chmod just before we
need it.

osx: in README.md, add `--without-json`

Otherwise, we may try to link to a library present through homebrew and
the package won't be portable.

osx: ship *.in files with tarball

Due to commit 95493844623c46e803d31302ac7758ce01580be7, we didn't ship
the templates with the tarball anymore. Fix that.

client: change location of bash completion

fix parameter 'details/hidden' not found in the function declaration [-Wdocumentation]

fix parameter 'lldpctl' not found in the function declaration [-Wdocumentation]

lib: don't use priority with constructor

It is not supported on older versions of gcc. Like the one in RHEL 5. It
should not matter for our case as atoms and builders can be added in any
order. Cc @commodo for confirmation.

redhat: don't build with JSON by default

json-c-devel is available in CentOS 7, but not RHEL 7. It's available in
Fedora. Just disable it everywhere, not an essential feature.

build: only uses -pie when linking programs

We can't rely on libtool to remove the flag on libraries.

build: when checking for additional flags, be fatal on errors

lldp: do appropriate cleanup when unable to allocate memory

travis: don't install libtool/autoconf/automake

They are altready installed through homebrew.

tests: document the suppression file for valgrind

build: don't use -Wl,-pie

This flag seems to work when running on i386 on Linux but isn't handled
correctly by libtool and liblldpctl.so ends being without any
symbol. Use -pie only and try an up-to-date libtoolize on OSX to see if
it performs better.

redhat: fix dates in .spec

build: require json-c >= 0.10

`json_object_object_get_ex` is only available from this version. json-c
name has been introduced from 0.11, so we can safely assume that with
this name, we have the right version. Do the checks only on old names.

redhat: fix a typo in conditional JSON support

redhat: provide a simpler buildroot

It seems that %() may not be expanded as expected.

release: prepare new version

lldpcli: fix another memory leak when querying statistics

lldpcli: fix a memory leak when querying custom TLVs

build: rename valgrind suppression file to match the right tool

ctl: don't use C99 designated initializer for structs to travel on a socket

While this is harmless, we prefer to know that each byte is correctly
initialized to avoid Valgrind warnings.

tests: provide a complete test file for lldpcli

Unfortunately, to be fully automated, we should be able to run lldpd as
root and run the test as root as well.

lib: use ssize_t to store result from write()

doc: fix example of "configure dot3 power" command