release: prepare 0.7.14

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

NEWS: add an entry for the two previous commits

lldp: fix boundary checks when decoding LLDP protocol identity

lldp: fix boundary checks when decoding LLDP management address

lldpcli: add "tagged" keyword to "configure med policy"

This allows a user to set the tagged flag for a network policy for a
given application.

Fix: #88

travis: use stable NetSNMP from Homebrew

The devel version is not here anymore and the stable one has been
upgraded to 5.7.3.

libevent: enable silent rules by default

libevent: update embedded copy to 2.0.22-stable

NEWS: add an entry for the two previous commits

lldp: send a shutdown LLDPDU on exit

With the previous commit, it is now easy to send a shutdown LLPDU when
exiting lldpd. Let's do that.

lldp: when MSAP changes, send a shutdown LLPDU

Chassis ID and port ID are not supposed to change. However, it is
possible for the port ID to change (for example, if the port name
changes or if the MAC addresses change). Since they should be constant,
we "fix" that by sending a shutdown LLPDU before any broadcasting any
change.

As a provision, this also applies to chassis ID even if it is currently
not possible for it to be changed.

marshal: don't marshal previous port value

It is only used by the daemon.

travis: don't compile embedded libevent + clang on OSX

We already try with Linux, that should be enough.

travis: don't fail on unused functions (second try)

travis: don't fail on unused functions

There are some of them in embedded libevent.

travis: don't allow any warning or error

version: prepare 0.7.13 release

configure: let define pkg-config directory with `--with-pkgconfigdir`

This needs a recent version of pkg.m4. However, we provide a fallback if
not present.

Closes: #87

man: `-I` option and the like also affect sending

Instead of using `listen` to describe the `-I` option, specify that it
also used to decide which interfaces LLDP will be enabled on, both for
listening and sending.

lldp: fix display of unknown organizational ID

Because it was stored in a signed char array, it was incorrectly
displayed with the "%x" formatter. Store it into an unsigned char array.

lldpd: don't rely on a checksum to detect a port change

Instead, just compare the serialized copies. This is more reliable than
a checksum and also (a bit) faster.

lldpd: use a macro to define offset of permanent port info

NEWS: add an entry for the latest commit

lldpd: fix coding style in latest commit

Merge pull request #84 from IcyMetric/master

Allow setting control socket on CLI

Allow setting control socket on CLI

lldpd: decrease probability of collisions on port checksums

Instead of a 16-bit checksum, we use two 16-bit checkums, one for each
half of the serialized port. We now need more than 70000 ports to get a
50% chance of collision while we previously needed only 300 ports.

lldpd: decrease probability of collisions on port checksums

Instead of wasting a whole bit (getting 2**15 valid values), just waste
one special value (2**16-1 valid values).

lldpd: really ensure that port checksum can never be 0

lldpd: fix timer initialization when checksum is 0

If the checksum computed over the port happens to be 0, the LLDPDU is
never scheduled for emission on this port. We ensure that a computed
checksum is never 0. If two different ports happen to have the same
checksum, we consider it unlikely enough to bear with the fact that the
timer will not be reset in this case.

marshal: ensure we correctly handle signed result from serialization

We must handle correctly `ssize_t`. Try to not cast this to another
type, let alone a non signed type.

xml: check that libxml-2 is really usable

snmp: `agent_init()` won't modify AgentX string

This fixes another compiler warning.

lib: silent a warning about uninitialized value

This is a false positive as the value was guarded by another
value. Nevertheless, let's please the compiler.

redhat: do not install the chroot

SuSE doesn't allow it anymore. This is not needed since lldpd will do
it at start.

redhat: requires pkg-config to build devel package

This is not really needed but SuSE is a bit picky in their checks.

release: prepare release of 0.7.12

lib: fix pkgconfig file substitutions

client: fix "update" command

Commit e4c51716848a639daf61eff3e96ad242af3e316b broke this command
because we were checking for `tx_hold` instead of `tx_interval`. Let's
fix that.

travis: travis comes with autotools and friends already installed

If they are not quite up-to-date, we get an error, so don't try to
install them.

debian: update symbol versioning for incompatibility of `lldpctl_atom_set_str()`

lib: bump versioning

Even if the ABI stays the same, the ability to pass NULL is an interface
change (backward compatible). We don't want new versions of lldpcli to
interact with an old version of the library.

lib: make it safe to pass NULL to `lldpctl_atom_set_str()`

Even if we don't allow such a thing.

lib: document the possibility of using NULL with `lldpctl_atom_set_str()`

lldpcli: allow to unset patterns and system description/hostname

lib: replace macro by a function for `_lldpctl_atom_set_str_config()`

lib: factor out some code in `_lldpctl_atom_set_str_config()` in a macro

NEWS: add an entry about some flags not being persistent accross time

See e4c51716848a639daf61eff3e96ad242af3e316b.

build: hide renaming of template-generated files

lldpd: use strdup() on arguments that could be changed by lldpcli

And add the appropriate "const" modifier for those that don't. The
compiler should warn us if we did something wrong.

client: fix configuration modification

Handling of configuration change was messy. Whatever configuration item
needed to be updated, a whole lldpd_config structure was sent. The
daemon part was trying to guess what changed by assuming non-0 fields
need to be updated. However, when flags were added, the implementation
became inconsistent. Some flags used 1/2 for true/false, some others
kept 0/1. So, some flags were detected as changed while they were not.

Since we require to provide a current configuration before making any
change, we just copy the whole structure, modify what we want and send
it to the daemon. The daemon will then compare the new structure with
the old one to detect changes. Flags that were using 1/2 logic are now
just using 0/1 like others.