osx: fix postinstall script

`@PRIVSEP_USER@` and `@PRIVSEP_GROUP@` were not substituted with the
appropriate values because they were not declared as an output
variable.

packaging: change `PreReq` to `Requires(pre)` in lldpd.spec

PreReq is deprecated and makes the build fail on OSC.

packaging: update to 0.7.6

README: show how to use tcpdump for troubleshooting

priv: correctly declare `priv_cmd` type

`enum {} priv_cmd` was the declaration of a variable `priv_cmd` as an
anonymous enum. This is not what was expected. Instead, we want `enum
priv_cmd` to be a declaration of a named enum: `enum priv_cmd {}` (we
could also use `typedef enum {} priv_cmd`, but in lldpd, `typedef` is
seldomly used).

When `priv_cmd` was moved out of `priv.c`, its symbol was defined in
several objects. It seems that recent versions of gcc are able to cope
with that (because `priv_cmd` variable was not used, so it was
optimized out I think, not sure). However, some older versions like
4.2.1 complained about duplicate objects during the link phase.

doc: add a CONTRIBUTE document

linux: define IFALIASZ for old systems

lldpd: fix a fd leak when we cannot write the new alias

lldpd: append "was" to an interface description when a neighbor disappear

This allows a user to know what neighbor was connected to a
server. This should closes #16.

client: add "configure/unconfigure system interface description" command

This allows to enable/disable updating interface description based on
visible neighbors.

lib: `lldpctl_k_config_receiveonly` is a read-only property

Update `lldpctl.h` to reflect this fact.

interfaces: overwrite interface description with neighbor found

If no neighbor is found, interface name is `lldpd: no neighbor
found`. If one neighbor is found, this is `lldpd: connected to XXXXX`
and if several neighbors are found, this is `lldpd: XXX
neighbors`. Smart filter is used to count neighbors.

Currently, this is not possible to disable this. Works with FreeBSD,
OpenBSD and Linux.

priv: avoid to leak generic socket

Due to previous commit, a socket was requested for each operation
needing it but it was not released.

privsep: separate OS specific code to dedicated files

Linux only stuff goes in `priv-linux.c`. This includes interface
handling, ethtool and ability to open files (which is not Linux
specific but only Linux requires this).

BSD stuff goes in `priv-bpf.c` since it only includes interface
handling.

Moreover, `privsep_fdpass.c` is merged into `privsep_io.c`, a new file
for almost everything about IO (read/write and passing FD). The global
`remote` object is put into this file.

This commit is mostly moving stuff around. This will enable the
ability to write interface aliases without putting too much #ifdef.

redhat: require groupadd/useradd

Those are needed on RHEL/CentOS if lldpd is installed early (for
example, in Kickstart). Thanks to Larry Brigman for correcting this.

osx: don't repeat `pkg-ref` several times in `distribution.xml`

osx: don't use `domains` tag in `distribution.xml`

This triggers the use of a new buggy "Destination Select" pane.

osx: add a way to compile for older versions of OS X

osx: `make pkg` can now build universal binaries

Next step: compile for another SDK.

osx: don't try to build universal binaries

The use of multiple arch is not safe with autotools. I believe that
libevent compute some arch-dependent stuff at configure-time. Just say
in README.md that the package will only work for the same version of
OSX and the same architecture.

For the future:

  - real universal binaries can be built by configuring and building
    each arch into a separate directory, then merge the result with
    `lipo`.

  - building a package for an older versions of Max OS X can be done
    by using `-mmacosx-version-min=10.6` and `-isysroot
    /Developer/SDKs/MacOSX10.5.sdk` on both `CFLAGS` and `LDFLAGS`.

configure: remove out-dated instructions to regen autoconf out-of-git

There is no need to hack anything, thanks to the use of
`./get-version` script.

snmp: fix a typo in `agent_priv.c`

configure: ensure that osx/scripts are executable

osx: use HTML files instead of RTF for welcome message

Also add a conclusion file.

osx: simpler instructions in README.md

osx: add a target to build an OSX package

We use pkgbuild and productbuild (which seem to have superseded
PackageMaker). Those tools are shipped from OSX 10.6.6. There are only
a few examples of how to use those tools with daemons. You may also
look at Jenkins or ngircd.

See `README.md` for more information on how to invoke the build of a
package.

configure: fix a typo in the description of `--with-embedded-libevent` option

travis: check that we can build lldpd with embedded libevent

Since we got an option to force the use of embedded libevent, let's
test that.

configure: add a flag to force use of embedded libevent

This will prove useful for OSX when building as a package.

debian: update packaging to match downstream

compat: include config.h in compat.h

Without this inclusion, no `HAVE_*` macro was defined. This was
triggered because with some libc, `strndup()` is defined by a macro
and therefore cannot be redefined as a function.

compat: fix strndup

NEWS: add an entry about the use of `strndup()`

compat: add `strndup()` compatibility function

This function does not exist on OS X 10.6 for example. This was a long
time GNU extension.

compat: remove copyright notice for setproctitle()

It's an empty function!

tests: define ck_assert_ptr_eq if check is too old

osx: use SHA1 instead of MD5 in homebrew formula

check: use `ck_assert_ptr_eq` instead of `ck_assert_int_eq`.

style: use C99 designated initializers instead of memset() when possible

packaging: update to 0.7.5

tokenizer: input buffer was too small

packaging: update packaging

bpf: on OpenBSD, invert the filter direction

From the manual page:

> Sets or gets the status of the `direction filter` flag.  If
> non-zero, packets matching the specified direction (either
> `BPF_DIRECTION_IN` or `BPF_DIRECTION_OUT`) will be ignored.

NEWS: add an entry for the use of `setproctitle()`

compat: declare `setproctitle()` as a printf function

debian: build-depends on libbssd-dev

lldpd: when no `setproctitle()` available, don't count neighbors

lldpd: count number of chassis to get number of neighbors

Also, if only one neighbor, just display its name.

lldpd: don't call `lldpd_count_neighbors()` too many times

build: use libbsd if available, also use `setproctitle()`

The monitor process will be titled "monitor", while the unprivileged
one will have the number of neighbors displayed. We provide an empty
fallback since this function is not essential.

On Linux, we expect `setproctitle()` to be available in `libbsd`. This
makes functions like `strlcpy()` and `fgetln()` also
available. However, the headers are `bsd/string.h`, so we either need
to declare the prototype or include those new headers (or use the
overlay system). A simple thing to do is to detect the usage of libbsd
and include the appropriate headers in this case.

lldpcli: `configure system platform` to override platform description

lldpcli: `configure system description` to override chassis description

lldpcli: split `conf.c` into `conf-system.c` and `conf-lldp.c`.

lldpcli: `configure system interface pattern` instead of `configure lldp iface-pattern`

`configure system` will contain non-LLDP related stuff. Moreover, we
may add `configure system interface alias rewrite` to enable the
rewrite of ifalias (a long awaited feature).

libevent: use GitHub git tree

Sourceforge one is out-of-date

debian: sync packaging with official Debian package

libevent: switch to patches-2.0 branch to get the appropriate automake fix

libevent 2.0.21 is not compatible with automake 2.13. libevent 2.0.22
will be but is not released.

git: ignore test-driver, generated by recent automake versions

Fixed some boundary conditions in code that cleans
remote ports on ttl expiry.

Problem:
- ttl expires, cleanup check fails to detect that ttl has elapsed
and skips cleanup
- After that, The ttl timer set routine again ends up setting the timer to ttl
resulting in 2 * ttl time for the expired rport to get released

(The test I did was with 1 port)

liblldpctl: fix infinite loop when user is reading fixed chunks

ISSUE:

_lldpctl_do_something() has the following loop for receiving a complete
message:

while ((bytes_needed = ctl_msg_recv_unserialized(...)) > 0) {
    _lldpctl_needs(bytes_needed)
}

ctl_msg_recv_unserialized() processes the received message and determines
how many more bytes are needed. First time, it requests "header" worth of
bytes. Once it receives that, it looks at header->len to determine next
set of bytes.

_lldpctl_needs() calls the connection's receive() routine to receive a
chunk of data and copies that over the a buffer in the connection structure,
either by malloc/copy or realloc/memmove. This buffer is called
conn->input_buffer. conn->input_buffer_len stores how many bytes are
currently in conn->input_buffer.

The issue is with the following statement in _lldpctl_needs():

        if (conn->input_buffer_len >= length) return 0;

where length is bytes_needed in the above illustration.

Suppose the message requires 16578 bytes. The connection's receive()
routine provided by the application reads in chunks of 1024 bytes.
With the above logic, the loop will execute successfully for 16 times
with the following resultant state:

conn->input_buffer_len = 16384
bytes_needed = 194

The above conditional statement in the beginning of _lldpctl_needs()
will cause it to return 0. ctl_msg_recv_unserialized() will keep asking
for 194 more bytes. The while() loop goes ad infinitum.

PATCH:

Remove the above statement. It is not needed.

marshal: fix alignment issue when unserializing

On some archs (Sparc and sometimes ARM), unaligned access are
forbidden. Instead of copying unaligned structures with `memcpy()`, we
ensure that `struct marshal_serialized` structures are always
correctly aligned, including when they are serialized. This is done by
adding some padding before appending such a structure to a buffer.

log: fix memory leak when using a log handler

This is quite important for liblldpctl.

marshal: don't declare serialize/unserialize helper for string

When using convenience library, this may lead to multiple definition
of this function. We are better off not defining it and using
`marshal_unserialize()` in `client.c` instead.

lldpd: fix an alignment (non-)issue

lldpcli: don't use an hack to pass the writer to watch callback

Use a proper structure for this purpose.

marshal: provide type-specific serialization/unserialization functions

Those functions should be used in place of `marshal_serialize()` and
`marshal_unserialize()` if possible.

client: fix a memory leak when changing lldpd configuration

lldpd_config_cleanup() does not free the configuration structure.

ctl: fix a memory leak when cleaning control socket

client: trigger interface refresh when updating interface pattern

lib: fix a memory leak when modifying interface pattern

lldpcli: document "configure lldp iface-pattern" in manual page

lldpctl.h: document possibility to write iface-pattern

lldpcli: Add support to set iface-pattern

This patch adds a new configure lldp command to set iface-pattern

Currently only sets iface-pattern on the server.
More incremental work is needed to trigger an interface
update and send.

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>

sonmp: allow SONMP unittests to work without a global config

man: document tx-hold

man: tx-interval cannot be set per port

lldpd: Add tx_hold

This patch adds a new parameter tx_hold and also
changes txTTL calculation to be a multiple of
tx_hold and tx_interval

Also adds cli support to configure tx-hold:
#lldpcli configure lldp tx-hold 1

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>

lldpcli: don't rely on `fgetln()` providing a NULL-terminated string

From the manual page of some BSD, this is not the case.

lldpcli: don't free input line

Line returned by `fgetln()` should not be freed, including when we use
the implementation we provide in `fgetln.c`. This fix an heap
corruption.

lib: keep space for new values in `lldpctl_key_t`

By inserting new values in the middle `lldpctl_key_t`, we break the
ABI of the library. To ensure future backward compatibility, we
introduce some space between each block. Unfortunately, this breaks
the current backward compatibility.

debian: synchronize packaging with Debian

release: prepare 0.7.3

NEWS: add a word about Solaris support

solaris: less intrusive way to get CMSG_* macros

The use of _XOPEN_SOURCE breaks most BSD.

solaris: assume that IP forwarding is disabled

We don't know how to detect that while in the chroot.

solaris: preliminary support

Basic functionalities are present. However, the interface support is
very poor. There is no way to detect bridges, VLAN and
aggregates. There is no MAC/PHY support. There is no detection of
wireless devices.

The code to detect IP forwarding is here but does not work inside the
chroot.

NEWS: add an entry about LLDP-MED fast start.

lldpcli: allow to enable/disable fast-start and set delay

lldpd: enable fast start by default

lldpd: make fast start work only when receiving LLDP-MED LLDPDU

lldpd: fast start support

This patch adds support for lldpd fast start.

The feature is currently disabled by default.
A cli command to enable the feature is in the
works.

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>

configure: use `AC_PROG_EGREP` to find a suitable `grep -E`

On some systems, this is `ggrep -E`.

lib: use uint8_t instead of u_int8_t

`uint8_t` is part of C99 and comes from `stdint.h`. `u_int8_t` works
on many OS but not on some. There are many other occurrences but we
fix those here because this is the only public header.

configure: display configured OS

lldpcli: complete the manual page with the syntax of "show statistics"

lldpcli: summary of statistics is done on provided ports only

lldpcli: fix two warnings introduced by the previous commit

NEWS: add an entry for "show statistics summary" command

lldpcli: global stats command

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>

os: add support for DragonFly BSD

This is similar to FreeBSD except that DragonFly BSD does not support
the following:

 - link aggregation
 - interface description
 - DMI