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

interfaces: management address is set to pattern on exact match

When the management pattern is an exact IP address, use it as is instead
of trying to match an interface. When a user doesn't want this
behavior (keeping the check), they should use something like
`192.168.1.15,0` or `!*,!!192.168.1.15` instead of the exact IP address.

This feature allows a user to advertise an out-of-band management IP
address.

Closes #103

lldpd: add port description override via lldpcli

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

lldpd: implement basic support to override port IDs via lldpcli

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

.gitignore: ignore vim swap files

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

style: remove `if (...) free(...)` pattern

Using the following Coccinelle patch:

    @@
    expression ptr;
    @@

    - if (ptr) {
    -   free(ptr);
    - }
    + free(ptr);

    @@
    expression ptr;
    @@

    - if (ptr) free(ptr);
    + free(ptr);

    @@
    expression ptr;
    @@

    - if (ptr != NULL) {
    -   free(ptr);
    - }
    + free(ptr);

    @@
    expression ptr;
    @@

    - if (ptr != NULL) free(ptr);
    + free(ptr);

interfaces: don't test for NULL before `free`

This is unneeded.

priv: document improvement in crash handling in NEWS file

Revert "priv: on Linux, when monitor dies, kill child"

This reverts commit 4747e964096a1b490de062460608805c46f0cfd2. This
commit is now useless due to the previous (more portable) commits. It is
harmless but also unneeded.

priv: use SOCK_STREAM for monitor communication

SOCK_SEQPACKET is better but not available everywhere. We can't just
fallback to SOCK_DGRAM since we lose the ability to detect when the
monitor dies. So, we use SOCK_STREAM (now possible with commit
004b5f944539). The code (stolen from OpenBSD) was already ready for
that.

priv: don't log anything when handling SIGCHLD

Logging is not signal-safe. Notably, we use date-functions which are
using locks.

priv: ensure we write exactly what will be read

When using SOCK_DGRAM/SOCK_SEQPACKET, it doesn't matter to write a bit
more since it is truncated. However, if we want to switch to
SOCK_STREAM, we must exactly write the right amount of bytes.

ctl: create control socket with close-on-exec semantic

Otherwise, when we spawn lldpcli, it inherits the file descriptor and
doesn't detect if the daemon dies just after because it is listening
itself to the Unix socket.

priv: fallback to SOCK_DGRAM if SOCK_SEQPACKET not available

priv: use a SOCK_SEQPACKET for communication with monitor

This has the same semantics than SOCK_DGRAM but it also supports the
ability to notify the other end of end-of-file events. However, it may
not be widely implemented, hence the separate commit.

priv: monitor the monitor process

By monitoring the socket connection with the monitor process, we ensure
that we kill ourselves if the monitor disappears and fails to notice us.

This doesn't work yet because there is no "end of file" event for a
SOCK_DGRAM. See the next commit.

log: add token to all `fatalx` instances

The inability to use a token is quite artificial.

priv: on Linux, when monitor dies, kill child

When the monitor dies, it sends its child a TERM signal but if it
segfaults or is killed with an uncatchable signal, the child will
continue to run "unharmed". On Linux, it is possible to send a signal to
the child in this case, with `prctl()`. Do that.

event: don't remember (in cfg) control socket event

We don't need it outside of event loop initialization.

event: don't ignore SIGHUP twice

build: don't use `sort -V`

Not portable enough. Use `sort -k ... -k ...` instead.

check: add missing license headers to some files

check: fix SNMP test format error

check: more comparison with `ck_assert_ptr_eq()`

check: compare pointers with `ck_assert_ptr_eq()`

check: enforce correct alignment during SNMP tests

This should fix the following clang error, despite the fact that we know
that the alignment was correct:

```
check_snmp.c:900:9: fatal error: cast from 'u_char *' (aka 'unsigned char *') to
      'unsigned long *' increases required alignment from 1 to 8 [-Wcast-align]
                            *(unsigned long int *)result,
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/include/check.h:220:22: note: expanded from macro 'fail_unless'
        _fail_unless(expr, __FILE__, __LINE__,\
                     ^
1 error generated.
```

travis: don't configure with CFLAGS=-Werror

This is unsupported. Instead, call `make CFLAGS=-Werror`.

configure: move LDFLAGS to LLDP_LDFLAGS

configure: move CFLAGS/CPPFLAGS to LLDP_CFLAGS/LLDP_CPPFLAGS

The goal is to avoid the user to override those flags when using `make
CFLAGS=somethingelse`. This is to enable a user to use `make
CFLAGS=-Werror` since this flag cannot be used with `./configure`.

configure: put cpp flags into CPPFLAGS instead of CFLAGS

configure: enforce the use of C99

Merge pull request #98 from baloo/features/document-lldp.d

client: document lldp.d configuration directory

client: document lldp.d configuration directory

Signed-off-by: Arthur Gautier <baloo@gandi.net>

interfaces: fallback to link local addresses for management

When no global address is found for a family, we fallback to a link
local address instead. For IPv4, this is 169.254.0.0/16 and for IPv6,
this is fe80::/64.

Closes #96.

interfaces: exclude IPv4 link local addresses from management addresses

Link-local IPv4 addresses (169.254.0.0/16) are not an acceptable
management address. Closes #96.

interfaces: enforce redefinition of IN_IS_ADDR_* macros

For IPv4, some definitions may differ from one OS to another. Better
safe than sorry: use the same definition for everyone.

priv: don't lookup for _lldpd when privsep is disabled

Closes #95

lib: fix "unknwon" typo

lib: fix compilation error when Dot3 is disabled

Some power maps are shared between Dot3 and LLDP-MED. Closes #94.

priv: replace the use of `gethostbyname()` by `getaddrinfo()`

`gethostbyname()` was uses to get the canonical name. `getaddrinfo()`
should work even if there is no IPv4 at all in the system.

priv: rename "gethostbyname" priv function to "gethostname"

Because, it is not a wrapper around `gethostbyname()` but a way to get
local hostname. It currently uses `gethostbyname()` and therefore relies
on the presence of an IPv4 address in the system.

build: use version sort for changelog