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

doc: fix example of "configure med location address" command

doc: fix documentation of "configure lldp portidsubtype"

tests: remove old reference to ifdump.txt

It was used for testing getifaddrs() but we don't use it anymore.

doc: add an entry for setting custom TLV in lldpcli manual page

doc: add a NEWS entry for custom/unknown TLV

Merge pull request #109 from commodo/add-custom-tlvs

Add support for custom TLVs

build: compile libevent with -fPIC

When embedded, libevent is configured with `--disable-shared
--enable-static`. This makes libevent compiled without `-fPIC` and we
cannot build a PIE. Use `--with-pic` to force compilation with `-fPIC`.

build: try -Wl,-pie before trying -pie for clang

build: don't enable PIE when using embedded libevent

build: also try to build a PIE when hardening is not disabled.

build: add a switch to disable hardening functions

Even when the tests succeed, on some platforms, hardening CFLAGS
fail. For example, this is the case on OpenWRT. It seems that the
executable needs to be linked to libssp. Difficult to test if the
problem only happens at runtime since we can execute anything.

lldpd: convert 'oui_info' member to dynamic array

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

lldpd: make custom TLV code optional

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

client: display custom TLVs when lldpctl is called

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

client: implement client interface for custom TLVs

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

daemon: implement client interface for custom TLVs

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

lib: add custom TLV + list atom(s)

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

daemon: add basic custom TLV send & receive

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

daemon: reduce log_info() to log_debug() for unknown TLV types

To reduce some log spam.

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

lib: move lldpd-structs.h include into atom.h

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

lib: add null check to lldpctl_atom_inc_ref()

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

lldpd: move LLDP_TLV_ORG to lldp-const.h

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

lib: enable ability to change log level on-the-fly

And remove the previous commit, use internal logging instead.

lldpcli: ensure we also display logs from liblldpctl

tests: fix typo in SNMP checks with protocol identity

frame: change checksum function

At the end of function, we used to call ntohs() but we also call the
same function before providing the checksum to POKE_UINT16 (which will
in turn call ntohs). We remove those two first extra calls as they
aren't necessary.

Moreover, the SNMP agent uses this function and this makes it
endian-dependant. We don't want that.

The function is quite convulated and isn't a classic checksum
function. I don't remember how I came with this function but since there
is the special case for Cisco devices and I know this special case work
now, I don't want to change to another function which may breaks this
special case.

Add a NEWS entry to let people know, just in case.

tests: fix SNMP tests

Management address need to be provided in network byte order.

tests: fix signed/unsigned discrepancy

build: cleaner way to not alter CFLAGS/CPPFLAGS/LDFLAGS

build: enable JSON/XML support automatically by default

redhat: enable JSON support if recent enough

debian: enable JSON support

build: automatic detection of optional features

`./configure` can now detect optional features. This works with XML and
JSON output as well as for SNMP support. The defaults are still the
same, so a user has to pass `auto` keyword to the requested option (for
example `--with-json=auto`).

client: only output supported formats

client: complain if the provided format is not supported

client: fix a trivial typo

Merge pull request #106 from commodo/make-some-tlvs-optional

Make some TLVs optional

client: add some log_debug() messages

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

tests/check_lldp: add global config object for testing

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

lldpd: make management addresses advertisement optional

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

lldpd: make chassis capabilities advertisement optional

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

daemon/lldpd.c: free() cfg at end of main function

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

daemon/protocols/lldp.c: do not send system capabilities if none available

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

lib: don't define __constructor__ twice

Merge pull request #105 from commodo/cleanup

Some Cleanup & prep-work for further updates

release: prepare 0.7.14

daemon: move protocol files into subfolder

lib/atom.c: move here remaining code from atom-private.c

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

lib/private.h: rename to atom.h

Makes sense to have it named like this now.
Would have been a bit nicer to do it earlier, but now should be fine too.

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

lib/atom-private.c: break it into smaller modules

This is a huge commit, but 90%+ of it is just splitting
'lib/atom-private.c' into smaller atoms and putting them
into the 'lib/atoms' folder.

Some glue code has been added, for constructing maps and builders.

For the 'struct atom_builder' type a 'nextb' field has been
appended at the end, to help make it a single-linked list of
'struct atom_builders's.
The 'next' field in 'struct atom_builder' is taken (hence 'nextb').

For the maps, a wrapper struct (called 'struct atom_map')
has been added on top of 'lldpctl_map_t' in order to create
single-linked list construct of maps.
The struct looks like:
    struct atom_map {
        int key;             /* lldpctl_key_t equivalent */
        struct atom_map *next; /* next 'struct atom_map' */
        lldpctl_map_t map[]; /* has to be at the end to
                                to be statically init-ed */
    }
The point of this is to register them at load-time so
that 'lldpctl_key_get_map()' can return them.

Each file in 'lib/atoms' registers it's own 'struct atom_builder's
and 'struct atom_map's.

The whole point of this is to be able to add a 'custom' and/or
'custom_list' atom(s) to be able to configure/broadcast other TLVs
in the future.
This mechanism is extensible to add other TLVs defined by
future standards.

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

lib/atom-private.c: move 'struct atom-builder' to private.h

Move this here, before splitting lib/atom-private.c.

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

configure.ac: add 'subdir-objects' option

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

NEWS: add an entry to NEWS file for previous changes

lib: update revision info to reflect ABI update of latest changes

lib: a change to port description is transmitted now

client: when receiving a port update, transmit it now

lldpcli: use "description" keyword when setting port description

lldpcli: document ability to change port id and description

lldpcli: ensure optional local port description cannot be repeated

Also fix some descriptions (no capitalization).

Merge pull request #101 from commodo/port_label2

lldpd: implement basic support to override port IDs via lldpcli