Allow to force a protocol.

This fixes ticket #34. By using the option to enable a protocol twice,
we are able to speak a protocol even when no peer is detected. In this
case, packets are sent even when another protocol is detected. A new
parameter has been added to get this behaviour for LLDP as well.

Add specifically a "-h" switch for lldpd and lldpctl

Add "format" option.

Use some conditionals to display some command line options.

Add a summary of available options in usage() of lldpd and lldpctl

Patch from Jorge Boncompte (see ticket #29).

Update README with the latest information

After displaying PVID, close the section tag.

Add XML support for Redhat as well.

Compile with XML enabled for Debian.

Remove unused chassis in lldpd_cleanup() instead of lldpd_port_cleanup().

In some cases, lldpd_port_cleanup() is called when a reference to the
chassis that will be removed is hold for updating. This happens when
we need to update the information on a chassis known by only one port:
 1. we find that the chassis is still the same
 2. we remove the port and hence remove the chassis which becomes
    unused and freed
 3. with the previous reference to the chassis, we try to update it
    with new information while it has been freed.

Freeing in lldpd_cleanup() is more natural and less risky.

Retrieve FD_SETSIZE in a sane environment.

Since we import linux/if.h, in some environment, we may end up
importing kernel FD_SETSIZE that may be inappropriate (too large). For
example, on Sarge systems, FD_SETSIZE becomes 8192 instead of 1024 but
FD_ISSET is unable to handle this size.

Increase refcount of the local chassis when a new port is created.

Also add an explanation of refcount handling in a part of the code
that was a bit difficult to follow. Now, each time that p_chassis is
affected to a new value, c_refcount should be updated accordingly.

Add a CHANGELOG entry for XML output functionality.

Fix wrong age display.

There are only 24 hours and 60 minutes per hour ...

Use LLOG_WARNX in xml_writer.c and remove superfluous newlines.

Add the text output label as "label" attribute to all xml elements.

Might be helpful when creating html via xslt etc.

Fix error handling in location data.

Instead of just logging an invalid data length for location data, an error
element is emited to the output.

An unknown location fromat is now enclosed in an "unknown" tag to prevent
that the "location" element ever contains character data.

Implement an XML writer.

XML output can be enabled with the "--with-xml" configure option. libxml2 is
used to create the actual XML output.

Implement a basic "writer" infrastructure to emit structured text.

This patch implements a basic plain text writer and uses that in display.c.

Rip display related stuff out of lldpctl and put into display.c

Avoid dereferencing a type-punned pointer.

When passing a file descriptor, we were dereferencing a unsigned char
pointer. This breaks strict-aliasing rules in C99. Moreover, we should
care about the alignment (even if in this case, this is aligned
because the previous member of the struct is an int). Therefore, we
use memcpy instead.

Free unused chassis.

When reference count for a chassis drops to 0, we free the chassis.
This cannot happen to the local chassis because a reference should be
held to it even without any local port.

Don't listen/send LLDP packets on bridge.

A bridge, like a VLAN or a bond device is no place to listen or send
LLDP packets.

Do not require libtool 2.2

... 1.5.26 apparently does nicely.

Make lldpctl refuse attempts to change anything when running setuid.

When running setuid, lldpctl now refuses to set location data. This should
make it save to run the program setuid root by normal users including a
web-server f.e.

Fix autoconf complains about libtool.

Add AC_PROG_LIBTOOL and removed AC_PROG_RANLIB.

Add a manual page for the new -X option.

Allow to specify the AgentX socket to use (instead of the default one).

This means that AgentX can be configured to use tcp:127.0.0.1:705 (for
example). Since we are running in a chroot, we cannot use hostnames.
Any network socket should work. Unix sockets should work as before
(using privilege separation).

Ensure that umask is set at a sensible value before building the chroot.

Since the unprivileged process needs to be able to read /etc/localtime
in the chroot, we ensure that the umask is set correctly for this.

For Redhat, adapt init.d script to initialize the chroot.

The chroot is an empty directory containing a copy of /etc/localtime
to enable the use of the correct timestamp when logging for the
process trapped in the chroot.

For Debian init.d, build the chroot before starting the daemon.

The chroot is an empty directory with a copy of /etc/localtime. This
file ensures that we are able to write logs using the correct time.

Change handling of conditional builds in the spec file

Change handling of build options from using simple define statements
to using the RPMs bcond_without functionality. bcond_without instead
of bcond_with ensures that the old behaviour is preserved (all
options are still enabled by default). This change allows the
convenient "rpmbuild --without xxx" and is in line with:

http://www.rpm.org/wiki/PackagerDocs/ConditionalBuilds

Merge branch '30-vlanid-without-name'

Don't set MFS with MTU value.

Because there is no way to retrieve MFS value from Linux (unless we
try to set it to some arbitrary value), we did use the MTU instead.
This is incorrect, as notified in ticket #31.

Since some equipements could use this value to trigger an alert, we
prefer not to send any value. We don't seem able to detect for VLAN
support or Jumbo frames support.

Display PVID even when we don't know its name.

We also display it if we have names for some VLANs but not for the
PVID one.

Cope with Port VLAN ID but no VLAN names.

This fix ticket #30.

Make netsnmp headers compatible with C99.

Disable inlining by adding -DNETSNMP_NO_INLINE to CFLAGS. Since we
only use header files, this should not be a problem. A more valid fix
would be to turn NETSNMP_INLINE into "extern inline" instead of just
"inline".

Without this fix, we get a lot of warnings with gcc 4.3.

Make lldpctl display the age of displayed information.

We rely on recorded time for last change and display the difference
with the current time. Information is displayed like this:

Interface: veth1 (via LLDP) - RID: 1 - Time: 0 day, 00:04:42

Any change will reset this time (for example, a new VLAN).

This should close ticket #21.

Add some additional GCC options by default.

We enable most warnings (but disable unused-parameter and
sign-compare). We also enable stack protector.

Get timezone info before being in the chroot.

Depending on the libc, first implicit call to tzset() can be done when
invoking syslog() the first time. In most situation, this is done when
we are in a chroot and therefore, we cannot read /etc/localtime and
get an accurate timezone information. Therefore, we invoke tzset()
before being in chroot.

Fix file descriptor leak when enabling "listen on VLAN" feature

The VLAN list was always reinitialized...

Reset "sent" flag for each port when sending.

On ports not receiving anything, we send an LLDP packet only on the
first running port. Others get ignored. We reset "sent" flag to 0 for
each port.

Update CHANGELOG to reflect recent feature merges.

Add CHANGELOG from 0.4.1.

Merge branch '15-old-compat'

Fix a conflict in src/lldpd.h. This is just a space story.

Also ship CHANGELOG file

Define ioctl for VLAN and bond.

We also give some kind of advice for people without if_vlan.h or with
an outdated if_bonding.h (for example, people with RHEL 2.1).

Fix some warnings about incompatible pointer types.

Save cache at various points of configure script.

linux/filter.h may need linux/types on old systems.

Add support for libc not defining __progname symbol.

This may happen with ancient releases of uClibc.

Rework autoconf stuff.

Update to autoconf 2.64.

Use custom macro for ethtool, netsnmp and argument handling to shorten
main configure.ac file.

Use less verbose build process by default.

For headers, missing header is fatal for now because we don't know how
to replace them. Better handling will be done later. We only target
Linux for now with GNU LibC: portability issues are narrow.

Display results at the end of configure process.

AC_C_CONST is obsolete

See:
 http://www.nabble.com/fix-AC_C_CONST-to-work-with-CFLAGS--O2--Wall--Werror-td17164627.html
 http://www.gnu.org/software/autoconf/manual/autoconf.html#index-AC_005fC_005fCONST-790

Some useful documentation about autoconf:
 http://sources.redhat.com/autobook/ (a bit outdated)
 http://www.freesoftwaremagazine.com/books/autotools_a_guide_to_autoconf_automake_libtool
 http://www.flameeyes.eu/autotools-mythbuster/

Fix some warnings in tests.

In C99, large integers are put in long long instead of unsigned long
int in C90. Or something like that.

Include manual pages into distribution.

By default, manual pages are not included in "make dist". We add them
by adding them in EXTRA_DIST. See:
 http://gnu.huihoo.org/automake-1.4/html_node/automake_50.html

Use "void" instead of empty parameter list for function prototypes.

Using empty parameter list means that we don't know what are the
parameters of a function, while using "void" means that there is no
parameter for this function. Some compilers are ticky about this. See
C99 6.11.6 and 6.7.5.3 §14.

This also allows to fix a little bug: priv_ctl_create was called with
a parameter.

Use libtool convenience libraries for common files for lldpd and lldpctl.

Instead of directly linking common files, we use another libtool
convenience library (libcommon.la). We also separate server part of
ctl.c into a separate file to avoid -DCLIENT_ONLY flag which was
causing recompilation of most files.

We also move compatibility objects (like strcpy() function) into
libcommon.la.

Replace index() by strchr()

strchr() is POSIX while index() is deprecated.

When SYSFS_BRIDGE_* stuff does not exist, just use old /proc.

Compatibility with gcc 2.95.x: add spaces around __VA__ARGS__

More ethtool compatibility

Compilation fix for ancient gcc.

gcc 2.96 does not like "char frame[]" declaration and complains with
"array size missing in frame'". We circumvent this limitation by using
a one byte array instead of an undefined one.

Add '-k' flag to disable kernel version advertising.

From a suggestion and a patch proposed in ticket #19. With this flag,
system description is just the kernel name and software version as
advertised if LLDP-MED is enabled is set to "Unknown".

Bridge ioctls cannot be used with a 32bit userland on a 64bit kernel.

Don't output any warning when ioctl fails.

Fix LSB header in provided Debian init.d script.

Thanks to a patch from Petter Reinholdtsen:
 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=542161

Add missing argument in some call to LLOG_INFO in lldp.c.

This fixes #17. Also add __attribute__ to log_* functions to let gcc
automatically check for this.

Do not include <ifaddrs.h> since we want to replace it.

Provide another getifaddrs implementation, stolen from USAGI project

Add getifaddrs() replacement for system without it.

Remove *.pcap file on clean.

Add unit tests for EDP

Add SONMP unit tests.

Add CDP unit tests.

Move pcap stuff into a separate file to allow sharing with future tests.

Disable some tests when some features (dot1, dot3, lldpmed) are not
enabled.

fail_* macros do not break execution flow.

Unlike what happen with unit tests framework in other languages,
fail_* macro does not jump to next test. Therefore, we have to break
flow ourselves.

Add LLDP receive tests

Add LLDP send tests.

Add unit tests for lists packing/unpacking.

Unit tests for packing/unpacking.

We build a static library liblldpd.la to ease linking to tests.
liblldpd.la is equal to lldpd except:
 - no main (but lldpd_main instead)
 - no link to netsnmp (because libtool try to link many many more
   things)

Update signature of transport string function for SNMP.

Fix bug in CDP packet generation: CDP version was always 1.

"(a && b) || c" gives 0 or 1, not b or c. Replace with a?b:c.

Fix compilation issues when DOT1 or DOT3 are not enabled.

Fix comment of iface_port_name_desc()

Use ifAlias from newer Linux kernels if available.

From 2.6.28, Linux allows to attach a description to an interface. We
grab this description from /sys/class/net/$IFACE/ifalias if it exists
and is not empty and use it as port description. In this case,
interface name is used as port ID.

 PortID:    veth0 (ifName)
 PortDescr: Test interface

This closes ticket #10.

Set h_ifindex correctly.

We also use this index as a way to define an interface. The following
comment has been added to lldpd.h:

An interface is uniquely identified by h_ifindex, h_ifname and h_ops.
This means if an interface becomes enslaved, it will be considered as
a new interface. The same applies for renaming and we include the
index in case of renaming to an existing interface.

Add a callback mechanism to watch for incoming data on sockets.

This callback mechanism is used to handle socket control. It could
also be used when we will need to monitor sockets outside of a port
context.

Add a note about EDP in changelog.

Update changelog.

Make "listen on vlan" feature optional at compile-time.

The code to handle this feature is a bit kludgy and it seems safer to
compile out this feature by default.

There are duplicate in the list of address we give to handler
functions. We need to acknowledge this correctly.

For bonds and regular interfaces, just check that h_flags is 0 before
trying to get information one more time.

For VLAN, check if the VLAN is already present in the list.

Add back "listen on VLAN" feature.

One VLAN can be associated to multiple interfaces through bridge or
bond. Handle those cases appropriately.

Add back support for bonding using the new architecture.

Don't free statically allocated port.

Change the way interface information are collected.

Move interface related stuff into interfaces.c.
A set of handlers are called sequentially to handle interfaces.

Move chassis update to a function.

Later, the function could be replaced with something less hackish if
needed.

Fix SNMP support to handle multiple systems on the same port.

Fix support for EDP with respect to multisystem support.
EDP VLAN are transmitted in another VLAN and therefore need to be
attached to the right existing system.

Add some debug output to track know remote systems/ports.

Adapt lldpctl to display received systems for each port.