automake: libevent.la is not a convenience library

Since libevent.la is not a convenience library, we cannot merge it
into liblldpd.la. Therefore, we link it to lldpd directly (and don't
try to build a convenience library from it).

client: fix unused variable when LLDPMED support is not compiled

marshal: use zero-sized array instead of flexible arrays

With gcc 2.96, flexible arrays are not supported, even with
-std=gnu99. And with gcc 3.x, zero-sized arrays cannot be statically
initialized (with more than one element).

Don't mix declarations and statements.

Also enable warnings when we try to. Compatibility with gcc 2.96 requires that.

configure: don't check for SLIST_* stuff: we don't use them

snmp: only call run_alarms() on SNMP timeouts

See snmp_alarm(3snmp) manual page for the
rationale. `NETSNMP_DS_LIB_ALARM_DONT_USE_SIG` is set by
`init_agent()`.

configure: make ./autogen.sh non recursive

configure: use AC_PROG_CXX macro

configure: always configure libevent to make "make dist" work

ctl: no need to keep a list of active clients

Really ship libevent.

Ship an embedded copy of libevent.

This embedded copy will only be used if no system libevent is
available.

configure: don't test for XML2 linking, useless

Just check for its presence with pkg-config and xml2-config.

configure: remove old unwanted cruft

It seems easier to fail on compilation if we cannot provide a
workaround in configure.

configure: drop dependency with libtool

configure: enable compilation of a libevent archive

README.md has been updated for those instructions. We still rely on
libtool to make this work. Unfortunately, the generated libtool comes
too late: we can't use it...

Update changelog, travis and packaging for libevent

lldpd: switch to libevent loop

This is a major change. Instead of a custom loop, we switch to
libevent. Only `levent.c` is aware of the event loop. Other parts do
not manipulate events. Currently, we still have a loop triggered every
30 seconds. This could change to be a per port handling.

configure: upgrade to libevent 2.x

Also, when linking with a static version of libevent, we enforce the
use of libtool ".la" file instead of ".a" to get appropriate
dependencies automatically and avoid a warning when linking inside a
".la".

Integration of libevent into the build system.

"configure" will check that libevent is present. It is possible to use
--with-libevent option to tell where libevent lives. We include the
possibility to use a static copy of libevent but with the absence of a
.pc file, we are not sure to handle all the dependencies correctly.

README file is updated with instructions about libevent, including
steps to build libevent inside lldpd.

lldpd: on exit, remove remote ports before removing local ones

lldpd: fix chassis copy

When a chassis is already known, we copy the new one over the old one
and keep the old one. With the addition of management addresses as a
list, the copy was incorrect. We fix it. This is kludgy. A better way
would be to use marshaling but marshaling is not able to do inplace
copy and it is not able to restore full list (previous pointer is not
restored).

valgrind: initialize some buffers to make valgrind happy

Valgrind does not understand some ioctl and thinks we are using
unitialized buffers while the ioctl will initialize them. We get rid
of those warnings by initializing them ourselves.

When running on valgrind, don't chroot or drop privileges

Valgrind documentation says:

  You are encouraged to copy the valgrind/*.h headers into your
  project's include directory, so your program doesn't have a
  compile-time dependency on Valgrind being installed. The Valgrind
  headers, unlike most of the rest of the code, are under a BSD-style
  license so you may include them without worrying about license
  incompatibility.

I don't like copying headers around. Therefore, there is an autoconf
check for valgrind headers.

travis: add Travis CI link to README

tests: when Dot3 feature is disabled, MFS is not transmitted

Allow compilation without Dot1.

travis: use apt-get instead of aptitude

aptitude will try to remove unused packages. We don't need this. Also,
enable XML output.

travis: use provided configure argument

Also install "check" package to enable unittests.

Integration with Travis CI

mgmt: allow one to provide a pattern to specify management addresses

Update CHANGELOG with entry about management addresses.

snmp: add support for IPv6 management addresses

snmp: fix SNMP support for management addresses

edp: ensure we don't add the same address over and over

We still need to discards address that are not advertised
anymore. However, that's not easy.

mgmt: free management addresses when freeing the chassis

edp: steal IP addresses from the received chassis instead of copying them

The received chassis information will be discarded, there is no need
to copy them. We behave like for VLAN.

cdp: allow to send several IP addresses

Support for multiple IPv4/IPv6 management addresses.

This patch introduces a new 'struct lldpd_mgmt' to store IPv4/IPv6
management addresses. These addresses are stored in a tail queue.
For the local chassis we currently only use one IPv4 and one IPv6 address.

Options for IPv6 addresses are the same as IPv4 (use -6).

TODO: Fix SNMP agent.

tests: fix location of .h files (they are in top_srcdir, not in top_builddir)

lldpd: add a better error message when we cannot create the unix socket

configure: automatically compute version from git

Also, output this version in usage().

lldpd: add interface to message about unguessable frame type

lldpd: do not stay stuck in the receive loop

On some corner case conditions, we can stay stuck in the receive loop
because we did not send packets for more than 30 seconds. In this
case, we keep reset the timeout to 30 seconds and we cannot exit the
loop (since the first condition to exit the loop is to have reached
the timeout).

Therefore, we invert the condition. If we have stay too long in the
loop, just exit. We know for sure that we will hit lldpd_send_all()
soon and get back to a sane state.

Also, in receive only state, we are also stuck in the loop. This
forbids us to update data about local chassis and interfaces. We also
fix this.

snmp: better integration of SNMP logging subsystem

cdp: allow to override value of platform TLV (Linux by default)

redhat: make the init script compatible with RHEL < 5

On RHEL < 5, killproc, daemon and status does not allow to take a
pidfile as argument. We detect this and omit it to provide such an
argument if needed.

marshal: circumvent cast alignment problems using memcpy

gcc was issuing warnings like this:
 marshal.c:290:5: warning: cast increases required alignment of target type [-Wcast-align]

They are harmless because for one of them, the target has been
malloced (and therefore is safely aligned to 8 bytes) and for the
other because we know that the target location is the location of a
pointer. For those later cases, assignment is replaced by memcpy().

tests: when comparing port ID and chassis ID, uses strncmp

Even when Port ID and Chassis ID are known to be strings, they may not
be null-terminated.

Use %zu for printf() when argument is of kind size_t.

%zu is valid in C99 and is supported by GNU LibC. We'll see if there
exists a portability issue. In this case, %lu and a cast to unsigned
long should be used instead.

Add a flag to specify which interfaces to use for chassis ID.

lldpd will use the first interface matching the pattern for chassis
ID. This will enable for example to always use "eth0". This closes #12.

Change the way Chassis ID is assigned.

By default, chassis ID is assigned the chassis name. Once an
appropriate interface is discovered, the chassis ID is changed to the
MAC address of this interface. In most configurations, the chassis ID
is always set to the MAC address of the first interface, like this was
before this patch.

lldpd: remove unneeded forward declarations

interfaces: remove unneeded forward declarations

lldpctl: add a flag to display hidden ports

Update CHANGELOG with actual changes from 0.5.6.

Provide a .gitignore.

Sources are expected to be built in build directory:

     $ ./autogen.sh
     $ mkdir build
     $ cd build
     $ ../configure --with-snmp
     $ make && make check

Add missing lldpctl.h header file

Use LIBADD instead of LDADD when appropriate.

Compilation fixes when disabling some part of lldpd

Major rewrite: rewrite lldpd/lldpctl communication mechanism

The new marshalling stuff is used instead of the previous packing
mechanism. Less code. Better code.

marshal: fix list marshalling and various other changes

Since last element and previous element are a pointer to a pointer, we
should handle this case differently. Unit tests are not failing
because the original memory space is still available on the stack. We
just ignore last and previous. This still allows to use the list for
the most useful operations: TAILQ_EMPTY, TAILQ_FOREACH and
TAILQ_REMOVE. We can't use TAILQ_INSERT_TAIL!

Another important change is that by default, macros will declare an
external structure. MARSHAL_EXPORT variable should be defined only for
one module. Otherwise, marshal structures will be defined several
times.

We also rename `_marshal_*` to `marshal_*_`. This enables more
graceful logging messages.

core: SMART_HIDDEN macro does not really need the configuration

Point to appropriate LLDP page on Wikipedia instead of disambiguation page.

configure: fix an issue with netsnmp_enable_subagent detection

marshal: add a way to ignore a pointer (setting it to NULL)

marshal: handle fixed-size strings

marshal: allow to specify null-terminated strings

marshal: use shorter macro names

Add a new serialization engine.

The chosen approach is totally different compared to the existing
packing mechanism. Instead of describing the struct with some string
and trying to decode the structure like the compiler would do (with
alignments problem), the structure is now described by its size and
the list of pointers in it. Macros are provided to make this
easy. Here is an example:

struct struct_simpleentry {
TAILQ_ENTRY(struct_simpleentry) s_entries;
int g1;
struct struct_simple *g2;
struct struct_simple g3;
};
MARSHAL_DECLARE_BEGIN(struct_simpleentry)
MARSHAL_ADD_TQE(struct_simpleentry, s_entries)
MARSHAL_ADD_POINTER(struct_simpleentry, struct_simple, g2)
MARSHAL_ADD_SUBSTRUCT(struct_simpleentry, struct_simple, g3)
MARSHAL_DECLARE_END(struct_simpleentry);

This enables the use of pointers, with detection of circular
references and therefore support of lists.

While only pointers need to be described, it is also possible to add
sub structure to avoid specifying (again) its list of pointers if it
has already been specified.

The use of this mechanism should simplify the client/server code.

tests: only compile check_snmp if SNMP support is available

snmp: ensure single inclusion of agent.h header

snmp: add unit tests

This also happens to fix some minor issues with SNMP implementation.

tests: Simplify Makefile.am

Start branch 0.6.

snmp: factorisation of agent_h_*med*() functions

snmp: factorisation of local/remote port handlers

snmp: factorisation of local/remote handlers

snmp: use a header_*indexed function for local management address

snmp: rework indexing

Add helper functions to write indexed functions.

Continuation of the previous patch.

frame.h uses very generic memeber names that resusts in conflicts with other defines

snmp: turn the "try next OID" code block into a macro to avoid errors

core: optimize BPF filter

When monitoring high trafic interfaces, the BPF filter can become
quite important. We optimize it a bit by first checking if the
ethernet address is a multicast one. If it is not, we only have to
check for EDP.

configure: check more SNMP stuff.

First, we check that netsnmp_enable_subagent() exists. Otherwise, this
means that Net-SNMP has been compiled without this feature.

Second, we check if <net-snmp/agent/util_funcs.h> header is
usable. There was incomplete installation of files required by this
header with NetSNMP 5.4 and situation got worse with NetSNMP 5.7. If
the header is not usable, we fallback to define the two functions that
we need.

Whitelisted driver is not a very informative information. Remove it.

Add a whitelist for some drivers (currently: dsa and veth)

Don't send MFS equal to 0.

cdp: send native VLAN only with CDPv2

cdp: send native VLAN TLV

cdp: allow reception of native vlan

/trabmission/tranmission/

Export TZ=/etc/localtime to fix timestamps.

While chrooted, syslog timestamps does not respect the configured
timezone despite the copy of /etc/localtime inside the chroot. Setting
TZ variable seems to fix this problem.

Closes: #13.

Spelling corrections

Fix a missing semi colon.

Prepare 0.5.5 release.

Extend whitelist with possibility to blacklist.

It is possible to specify patterns like "*,!eth1" to blacklist "eth1"
or something like "eth*,wlan*,!wlan*master" to listen to all eth
interfaces and wlan interfaces with the exception of interfaces like
"wlan0master".

Fix a segfault when neither /etc/os-release not lsb_release is available.

Closes #11.

Don't pack struct hmsg since we are not able to do unaligned access on some archs.

Packing was used because data was cast as a pointer and therefore,
header was padded excessively. Since we don't use it as a pointer, we
cast it as an incomplete array of chars. We don't need its size.

dmi: only enable DMI for x86 platforms