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.

lldpcli: pretty print json

Signed-off-by: Kanna Rajagopal <kanna@cumulusnetworks.com>

systemd: fix systemd unit file

@sbindir@ was not properly replaced.

debian: update symbols file

interfaces: bond listening was broken only for old kernels

With a recent kernel, we can listen to individual member. lldpd is still
listening on master to keep compatibility with older kernels and because
it is difficult to detect if the kernel gives us this ability or not. We
can't rely on version numbers as some kernels have backported
features (like 2.6.18 from RHEL 5).

interfaces: fix initialization of hardware h_data for bonds

On discovery, the master index was not correctly initialized and
therefore, the information was not retrieved correctly and therefore
listening on bond devices was broken since 0.7.0 (netlink rewrite).

frame: mention the licensing issue with POKE/PEEK macros

They were taken from ladvd but were originally based on the ones from
libcdp (under the Net::CDP umbrella) which is LGPLv2 licensed. To avoid
any grey area, the original author, Michael Chapman, kindly accepted to
relicense those under the ISC license.

Closes: #79

lldpcli: handle comments in tokenizer

This enables us to also ignore end-of-line comments. Completion is a bit
grumpy when done in comments but I doubt anyone would care.

Merge pull request #77 from Authority1980/master

Fixed SPEC file changelog dates

Fixed Changelog dates

rpmbuild on RHEL7 checks for bogus changelog dates

lldpcli: don't complain when reading a commented line

Since we ship a default configuration file with only a comment, this was
annoying.

redhat: mark /var/run stuff as a ghost directory

redhat: fix group tag

Dunno if they are correct for Redhat, but they are correct for SuSE.

redhat: provide a longer description for devel package

redhat: fix RPM building on SuSE with respect to /usr/share/zsh

An empty directory have to be referenced.

redhat: add completions for bash and zsh

redhat: everything in /etc/lldpd.d is a configuration file

redhat: reference /etc/lldpd.d/README.conf

We consider this is a configuration file event if it isn't.

redhat: use appropriate macros for directory

release: prepare 0.7.11

man: sync lldpcli description of pattern matching

Manual page for lldpcli didn't specify the `!!` notation.

man: say that by default only physical interfaces are used

lldpd: check if capabilities are available/supported before enabling them

Different platforms can support different capabilities. we need to enable
only those that the platform can support

Signed-off-by: kanna <kanna@cumulusnetworks.com>

build: hide invocation of sed command to generate some files

Replace them with `AM_V_GEN`.

lldpd: also try /usr/lib/os-release if /etc/os-release is not present

Usually, `/etc/os-release` is a symlink to `/usr/lib/os-release` but on
some of them, programs are expected to fallback to
`/usr/lib/os-release`.

build: remove useless variables from pkgconfig file

Because the content of liblldpctl.pc is now fully expanded, the
variables at the top are now useless. Remove them.

build: use @sbindir@ instead of @prefix@/sbin

Thanks to the previous commit, this is now possible.

build: use AC_CONFIG_FILES only to generate Makefiles

Other kind of files may get partially expanded variables. For example,
`@sbindir@` is expanded to `${exec_prefix}/sbin` which won't work in a
regular configuration file.

While this could be done easily with GNU Make, using automake doesn't
allow us to use pattern rules or stuff like that. Therefore, there are a
lot of repetitions to declare the files we need to transform.

debian: ship /etc/lldpd.d

client: ship /etc/lldpd.d and put a README file in it

build: allow use of older versions of json-c

json-c library was first just named json. When requesting compilation
with json-c, also check for this library.

lib: don't check for NULL before free

lib: make a copy of state_data, since pointer can be re-used

make a copy of state_data, since pointer can be re-used and the comparison checks
can still give false positives

Signed-off-by: kanna <kanna@cumulusnetworks.com>

debian: also install zsh completion

debian: also install bash completion

lldpd: abort when some options are repeated

Some options don't have to be repeated. Later options would override
earlier and this is usually an error. Better just abort in this case.

Revert "debian: ship files in debian/etc/lldpd.d"

This reverts commit 8e9bf5cd846b462bc7811a58133811f9082779d9.

interfaces: only use physical interfaces to get a chassis ID

debian: ship files in debian/etc/lldpd.d

build: check if pkg-config is correctly installed in ./autogen.sh

Absence of pkg-config leads to some odd messages:

    configure.ac:27: error: possibly undefined macro: AC_SUBST
          If this token and others are legitimate, please use m4_pattern_allow.
          See the Autoconf documentation.
    configure.ac:163: error: possibly undefined macro: AC_MSG_FAILURE
    configure:21413: error: possibly undefined macro: AC_MSG_CHECKING
    configure:21418: error: possibly undefined macro: AC_TRY_LINK
    configure:21423: error: possibly undefined macro: AC_MSG_RESULT
    configure:21432: error: possibly undefined macro: AC_MSG_NOTICE
    configure:21894: error: possibly undefined macro: AC_PATH_TOOL
    configure:21896: error: possibly undefined macro: AC_MSG_ERROR
    configure:21975: error: possibly undefined macro: AC_DEFINE_UNQUOTED

We try to detect this situation by looking for `PKG_CHECK_MODULES` macro
in `aclocal.m4`. If not present, we issue some useful tips, like
installing the package or telling aclocal the location of
`pkg.m4` (mostly for OSX users).

build: check for needed commands in autogen.sh

And display a more informative message.

build: provide a sysusers.d file

This allows systemd-sysusers to create _lldpd user/group. See:
 http://www.freedesktop.org/software/systemd/man/sysusers.d.html

This is currently untested.

lldpd: handle early cleanup in case interface cannot be created

When an interface is about to be created but an external condition makes
it impossible, the cleanup may crash because the interface was
incomplete. Check things exist before cleanup.

lib: handle read failure in sync case

This amends the previous commit with a more concise condition.

lib: really correctly handle read failure

lldpcli: disable SIGPIPE

We handle EOF inside the code.

lib: correctly handle read failure

lib: stricter checks when resuming an operation in progress

When an operation is in progress, we used both state management and some
canary value to check if the user is resuming the right operation. The
canary was just a pointer and it was easy to get the same pointer as the
previous operation while not really resuming the previous operation. We
turn the check into a string comparison instead.

Moreover, add checks when setting configuration items too.

lldpcli: add editor local variables to bash completion

lldpcli: add editor local variables to zsh completion

lldpcli: indent multiline texts (like system description)

This should fix #73 in an alternate way.

lldpcli: fix style convention to match other files

There are still other files around with this convention that would need
to be fixed.

bash/zsh: ensure completions are present in dist tarball

NEWS: add a word about bash/zsh completion

zsh: add ZSH completion

bash: move bash completion outside Debian packaging

release: new version 0.7.10

NEWS: add a word about json-c library support

lldpcli: fix jansson implementation of the JSON output

configure: if --with-json, default to jansson implementation

cdp: complete manual page with CDP-related options

cdp: Make it possible to enable CDPv2 without enabling CDPv1

lldpcli: Add json description to lldpcli usage

cdp: Add power requirement to CDPv2 frames

LLDP already supports sending power requirements, for
IEEE 802.3at. Older Cisco PoE switches do not support
LLDP and need a custom CDPv2 frame. This patch enables
support if the LLDP MED PoE frame is also sent.

lldpcli: Add support for JSON-C

OpenWRT is shipped with the JSON-C library, which is used for
several other packages. Having a separate JSON library (Jansson)
doing the exact same thing is wasting flash space.

lldpcli: make complete command work on privleged commands as well

This avoids the use of sudo.

lldpcli: provide a hidden complete command for shell completion

lldpcli: change how privileged commands are declared

Even when not privileged, we declare privileged commands but tag them
accordingly. We need to be careful when executing the commands to check
the new bit.

This is a preliminary change to allow a completion command to complete
privileged commands even when not privileged.

lldpcli: reformat a bit bash completion to be more readable

priv: avoid a socket leak when interface is already in promisc

snmp: avoid deferencing a pointer when it may be NULL

dcbx tlv recd are printed as log_debug

Changed dcbx tlv recd to be printed as log_debug
so that we dont flood daemon log (N3K sends dcbx
tlv by default.

Signed-off-by: Kanna Rajagopal <kanna@cumulusnetworks.com>
Signed-off-by: Sam Tannous <stannous@cumulusnetworks.com>

lldpcli: use protocol map from liblldpctl to select protocol

lldpcli: document `-u` argument in synopsis

lldpcli: add display filter to show nbrs running specific protocols

LLDPD allows a peer to run multiple protocols (LLDP, CDPv1, CDPv2).
lldpcli can be used to show the primary protocol for a nbr or all
the protocols using the "hidden" cmdline arg
This allows lldpcli to filter the nbr display based on the
protocol that is running on it.

Signed-off-by: Kanna Rajagopal <kanna@cumulusnetworks.com>
Signed-off-by: Sam Tannous <stannous@cumulusnetworks.com>

lib: update liblldpctl versioning

Previous commit added a function to the API.

Add call to process more messages from data already read.

Currently, the calls within lldpctl are assumed to be all associated with
synchronous sockets. With async sockets, the data read can be varied and so
its useful to process as much of the data as possible. However, lldpctl_recv
only processes one message and there isn't a call to be made to process
data that's sitting in a control connection's buffer. This patch adds such a
call for external users to call. It simply calls check_for_notification and
returns whatever check_for_notification returns with 0 signalling more
processing is possible and non-zero indicating buffer is fully processed
(though there may still be data).

Signed-off-by: Dinesh G Dutt <ddutt@cumulusnetworks.com>
Signed-off-by: Sam Tannous <stannous@cumulusnetworks.com>

Add error code to the multicast address add/delete failure msg

Move mac filter delete info msg to debug and add error
to the multicast address add/delete failure msg.

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

Move interface update msg to debug level

Move interface update msg to debug level so that we don't see
so many of them on large boxes.

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

Add ignore handler for SIGHUP in lldpcli

lldpcli which is execed by lldpd is expected to resume lldpd operations.
But in some cases, lldpcli can exiting because of SIGHUP.  This fixes
these cases.

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: Kanna Rajagopal <kanna@cumulusnetworks.com>
Signed-off-by: Sam Tannous <stannous@cumulusnetworks.com>

This patch adds bash completion for lldpcli.

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