build: allow building cli without dynamic librarires support

Commit 3cb28534d34392ceec4adead0cfa97039796ccb7 enables building
of cli always as part of `make check`. As cli previously always
included <dlfcn.h>, this broke tests for building with toolchains
that don't support dynamic library loading.

Add a configure check and disable dlopen() based on whether
<dlfcn.h> is available.

Signed-off-by: Thomas Haller <thaller@redhat.com>
https://github.com/thom311/libnl/pull/141

build: don't build cli libraries by default

When configuring with --enable-cli=no, we should not build
the libraries related the cli.

This fixes commit 3cb2853 (build: enable building cli during tests)
which enables building of cli during `make check`. However, during
regular build, these libraries should not get build.

Signed-off-by: Thomas Haller <thaller@redhat.com>

addr: add AF_VSOCK to translation table

Add AF_VSOCK to the address family translation table.

https://github.com/thom311/libnl/pull/142

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: Thomas Haller <thaller@redhat.com>

rule: merge support for l3mdev in FIB rules

http://lists.infradead.org/pipermail/libnl/2017-May/002321.html

Signed-off-by: Thomas Haller <thaller@redhat.com>

rule: change API for setting/getting l3mdev rule property

- for rtnl_rule_set_l3mdev(), also allow unsetting the l3mdev field.
  In practice, kernel only allows for two options: either omit
  tb[FRA_L3MDEV] or set it to 1. As such, rtnl_rule_set_l3mdev()
  allows for both of these. In principle the setter could get
  extended to set other values. Such values are reserved.

- for rtnl_rule_get_l3mdev() also return an error code. I think it
  is appropriate to mix value and negative error code, as long as
  the range of values cannot overlap with error codes.
  Arguably, the outcome is a bit awkward, as the function now is
  expected to return -NLE_MISSING_ATTR or 1. So, the best check
  is probably

      if (rtnl_rule_get_l3mdev(r) > 0) { ... }

The reason for this change is that libnl should expose the netlink
API without coercing uint8 to boolean. That way, future changes
in kernel don't require update to libnl3.

Signed-off-by: Thomas Haller <thaller@redhat.com>

rule: Add support for l3mdev in FIB rules

Add support for the l3mdev option in FIB rules. If l3mdev is set
then the lookup is directed to the table associated with the l3mdev
(e.g., VRF) device.

If the l3mev attribute is set the table id is not, so update the table
id attribute to make sure r_table is non-0.

iproute2 shows the rule as:
    1000: from all lookup [l3mdev-table]

where [l3mdev-table] infers the dynamic nature of the table id. Keep
that notation for libnl.

Signed-off-by: David Ahern <dsahern@gmail.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

Update fib_rules.h to latest kernel

Signed-off-by: David Ahern <dsahern@gmail.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

route: merge branch 'netconf'

(v1): http://lists.infradead.org/pipermail/libnl/2017-April/002308.html

http://lists.infradead.org/pipermail/libnl/2017-May/002316.html

Signed-off-by: Thomas Haller <thaller@redhat.com>

route: Add support for MPLS to netconf

Add support to netconf for MPLS address family.

v2
- change get method to return 0/error and take 'int *val'
  which is set to the value requested
- added rtnl_netconf_get_input to libnl-route-3.sym

Signed-off-by: David Ahern <dsa@cumulusnetworks.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

nl-monitor: Add support for netconf caches

Allow user to monitor netconf changes.

Signed-off-by: David Ahern <dsa@cumulusnetworks.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

nl-monitor: All user to specify line format

nl-monitor defaults to stats. Allow user to select brief and details.

The known_groups is moved up to a global to allow a proper print_usage
function.

Signed-off-by: David Ahern <dsa@cumulusnetworks.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

route: Add support for netconf

Add route/netconf cache type to monitor RTM_*NETCONF messages.

v2
- change get methods to return 0/error and take 'int *val'
  which is set to the value requested

Signed-off-by: David Ahern <dsa@cumulusnetworks.com>
Signed-off-by: Balakrishnan Raman <ramanb@cumulusnetworks.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

route: merge branch 'shanmugh-master'

Related: https://github.com/thom311/libnl/pull/137
Related: http://lists.infradead.org/pipermail/libnl/2017-March/002303.html
Related: http://lists.infradead.org/pipermail/libnl/2017-March/002304.html

https://github.com/thom311/libnl/pull/138

route: fix symbol versioning

Once released, an existing symbol version must not be modified.
Move the new symbols to the right section.

Do not increment refcount in rtnl_*_get_action APIs.

Provide accessors for actions (rtnl_act).

Add accessor functions to retrieve actions on a tc object
and provide additional function to navigate the collection
of actions.

route/link: add accessor API for IPv6 flags

Add functions to access the IPv6 specific flags of a link object.
Also the functions for IPv6 link flags translation are now exported, similar
to the non IPv6 specific translation functions.

https://github.com/thom311/libnl/pull/136

tools: fix building doc in build_release.sh

Fixes: 3b2071e970995a9a22d46457e60a762161b46231

libnl-3.3.0 release

Signed-off-by: Thomas Haller <thaller@redhat.com>

all: don't use math.h or link with libm.so

lib: escape usage of strerror_l() if it doesn't exist in libc

uClibc doesn't implement strerror_l() and thus libnl starting from
3.2.29 couldn't be compiled with it any longer.

To work-around that problem we'll just do a check on strerror_l()
availability during configuration and if it's not there just fall back
to locale-less strerror().

See-also: 6c2d111177e91184073c44f83d4a6182aaba06d7

http://lists.infradead.org/pipermail/libnl/2017-March/002301.html

Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

libnl-3.3.0-rc1 release

First release candidate for upcoming libnl 3.3.0 release.

Note that from now on, future libnl3 release will bump the
second version number, instead of the third. So the next
version will be 3.3.0 instead of 3.2.30.
This does not imply a break of API/ABI, it is only a change
of the versioning scheme.

Signed-off-by: Thomas Haller <thaller@redhat.com>

include: merge branch 'include-cleanup'

include: restore linux header includes in public headers

The previous commits reorganized the public headers to drop includes
of linux kernel headers.

Restore the previous situation because otherwise the change might
break compilation for users who rely on certain headers getting dragged
in by libnl3.

include: don't include kernel headers in public libnl3 headers

It would be desirable not to include kernel headers in our public
libnl3 headers. As a test, remove all those includes, and fix
compilation by explicitly including the kernel headers where needed.
In some cases, that requires forward declaration for kernel
structures, as we use them as part of our own headers.

Realistically, we cannot drop those includes as it probalby breaks
compilation for users that expect to get a certain kernel header
when including a libnl3 header. So, this will not be done and the
includes will be restored in the next commit.

Do this step to show how it would be and to verify that we could
build with such a change. The reason not to do this is backward
compatibility (at compile-time).

build: add tools/build_release.sh script

Add script to do a release.

build: reorder checks in configure.ac

build: style cleanup in doc/Makefile.am

build: fix creating directories for generated sources

Fixes: a572196366cfc1938596052f8e16121d0d374164

build: pass --disable-dependency-tracking to `make distcheck`

build: ensure build directory for generated sources exist

For out-of-tree builds, we must ensure that the build directory
for the generated sources exists. We do that by adding a dependency
to the .dirstamp file.

xfrm: merge branch 'sb/xfrm-sp-fixes' (#133)

https://github.com/thom311/libnl/pull/133

xfrm: NUL terminate the ctx_str buffer in xfrmnl_sa_set_sec_ctx()

and xfrmnl_sp_set_sec_ctx(). The user already must pass a NUL
terminated string, where the NUL is included in ctx_len.

Just allocate one more by and ensure the buffer is '\0' terminated.

xfrm: allow quering optional arguments from xfrmnl_sp_get_sec_ctx()

The previous API of xfrmnl_sp_get_sec_ctx() is totally broken,
as it requires all out-arguments to be set. The user can thus
not know how large the ctx_str buffer must be.

Fix the API by allowing all arguments to be optional. Thus,
a user can first query the size only, and then in a second
step query the ctx_str. Previous version are broken.

fix of boolean operators

Mixed up boolean operators in commit f38fb7a.
This commit is the appropriate fix.

update documentation of xfrmnl_sp_get_sec_ctx

add possibity to delete policy without index

security policy can be identified with direction, selector and security context
too. Therefore the code to append approptiate data to delete message is added.

update sp_attr condition in build_xfrm_sp_message

Identification of policy are possible with:
    1. direction and index
    2. direction and selector
Theoretically second one needs a security context, but non existing context is
valid too.

little style fixes.

fix xfrmnl_sp_set_sec_ctx length attributes

Fixed xfrmnl_sp->sec_ctx length parameters in xfrmnl_sp_set_sec_ctx,
because former use of only one value wasn't right.
Therefore parameter len is unsued and could be removed.

fix build_xfrm_sp_message index condition

Index is not needed and not known if adding a policy. Seems to be copied from
build_xfrm_sp_delete_message.

route: neigh: use NDA_MASTER for neigh->n_master if available

fdb cache is per bridge and hence hashed by:
<bridge_ifindex, family, mac>

newer kernels send bridge ifindex in NDA_MASTER.
Use NDA_MASTER for neigh->n_master when available.

Also imports a few more NDA_* attributes from upstream
to keep linux/neighbour.h NDA_* attributes in sync with
upstream.

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

build: enable more warnings

all: enable -Wmissing-prototype warning for all components

build: merge branch 'th/non-recursive-make'

Refactor makefiles to use one top-level Makefile.am
and don't do recursive make.

Exception is doc/Makefile.am, which is still separate
like before.

build: move compiler warning flags to separate autoconf variable

build: enable building cli during tests

The cli programs don't have any additional external dependencies.
So, during a `make check`, we should always build them, even if
they were disabled during configure and won't be installed.

build: merge src/Makefile.am into top-level makefile

build: merge src/lib/Makefile.am into top-level makefile

build: merge tests/Makefile.am into top-level makefile

build: merge python/Makefile.am into top-level makefile

build: merge man/Makefile.am into top-level makefile

build: merge lib/Makefile.am into top-level makefile

build: merge include/Makefile.am into top-level makefile

build: cleanup top-level Makefile.am

lib: check for integer-overflow in nlmsg_reserve()

In general, libnl functions are not robust against calling with
invalid arguments. Thus, never call libnl functions with invalid
arguments. In case of nlmsg_reserve() this means never provide
a @len argument that causes overflow.

Still, add an additional safeguard to avoid exploiting such bugs.

Assume that @pad is a trusted, small integer.
Assume that n->nm_size is a valid number of allocated bytes (and thus
much smaller then SIZE_T_MAX).
Assume, that @len may be set to an untrusted value. Then the patch
avoids an integer overflow resulting in reserving too few bytes.

lib: fix comment for nl_recv() about return value for non-blocking read

Since commit 375a6294a41e003f873821a01d947f0ecfaf76d4, nl_recv()
would fail with -NLE_AGAIN if there is nothing to read. Fix the
documentation.

https://github.com/thom311/libnl/issues/129

all: merge branch 'tklauser-missing-prototypes'

https://github.com/thom311/libnl/pull/125

build: distribute in.h in6.h libc-compat.h

https://github.com/thom311/libnl/pull/130
Related: https://github.com/thom311/libnl/pull/127
Related: https://github.com/thom311/libnl/pull/121

build: Add -Wmissing-prototypes to CPPFLAGS

Now that all -Wmissing-prototypes warnings are fixed, add it to the
default CPPFLAGS.

This should hopefully prevent additions of public functions with wrong
prototype or no prototype at all to the public header.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

ematch: Add missing function prototypes

Older versions of flex (namely the one used on Travis CI) don't properly
emit function prototypes for ematch_get_column() and
ematch_set_column(), leading to GCC -Wmissing-prototypes warnings. Fix
them by manually adding these prototypes.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

pktloc: Add missing function prototypes

Older versions of flex (namely the one used on Travis CI) don't properly
emit function prototypes for pktloc_get_column() and
pktloc_set_column(), leading to GCC -Wmissing-prototypes warnings. Fix
them by manually adding these prototypes.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

xfrm/template: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings.

Also adjust the implementation of xfrmnl_user_tmpl_set_family() to match
the declaration (family parameter is unsigned int, not int).

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

xfrm/selector: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings.

Also adjust the implementation of xfrmnl_sel_set_family() to match the
declaration (family parameter is unsigned int, not int).

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

xfrm/sa: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings.

Also adjust the function prototype of xfrmnl_sa_get_sec_ctx() to
match the definition.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

xfrm/lifetime: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

xfrm/ae: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

idiag/req: Add missing function prototype

Add missing function prototype for idiagnl_req_parse() to the public
header.

This fixes the following GCC warning when compiling with
-Wmissing-prototypes:

  idiag/idiag_req_obj.c:189:5: warning: no previous prototype for ‘idiagnl_req_parse’ [-Wmissing-prototypes]

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

netfilter/exp: Add missing function prototypes

Add missing function prototypes for nfnl_exp_unset_flags(),
nfnl_exp_flags2str() and nfnl_exp_str2flags() to the public header.

This fixes the following GCC warnings when compiling with
-Wmissing-prototypes:

  netfilter/exp_obj.c:458:6: warning: no previous prototype for ‘nfnl_exp_unset_flags’ [-Wmissing-prototypes]
  netfilter/exp_obj.c:475:8: warning: no previous prototype for ‘nfnl_exp_flags2str’ [-Wmissing-prototypes]
  netfilter/exp_obj.c:481:5: warning: no previous prototype for ‘nfnl_exp_str2flags’ [-Wmissing-prototypes]

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

netfilter/queue: Add missing prototype for nfnl_queue_msg_build_verdict_batch()

Add the function prototype for nfnl_queue_msg_build_verdict_batch() to
the public header.

This fixes the following GCC warning when building with
-Wmissing-prototypes:

  netfilter/queue_msg.c:190:1: warning: no previous prototype for ‘nfnl_queue_msg_build_verdict_batch’ [-Wmissing-prototypes]

Fixes: d612180cda43 ("netfilter/queue: introduce nfnl_queue_msg_send_verdict_batch()")
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

link/ipgre: Add prototype for ABI-preserving wrapper rtnl_link_get_pmtudisc()

Avoid a GCC -Wmissing-prototypes warning by explicitely adding a
prototype for the ABI-preserving wrapper added in commit 8d04ebf55400
("lib/route: preserve old ABI for rtnl_link_get_pmtudisc()") and add an
explanatory comment.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

link/sit: Add missing prototype for rtnl_link_is_sit()

Add the function prototype for rtnl_link_is_ipip() to the public header.

This fixes the following GCC warning when compiling with
-Wmissing-prototypes:

  route/link/sit.c:400:5: warning: no previous prototype for ‘rtnl_link_is_sit’ [-Wmissing-prototypes]

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

link/macsec: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

link/ipvti: Fix and add function prototypes in public header

Add missing "_ipvti" part to function prototypes so they match the
exported name. Also add the missing prototype for rtnl_link_is_ipvti().

This fixes the following GCC warnings when compiling with
-Wmissing-prototypes:

  route/link/ipvti.c:281:5: warning: no previous prototype for ‘rtnl_link_is_ipvti’ [-Wmissing-prototypes]
  route/link/ipvti.c:369:10: warning: no previous prototype for ‘rtnl_link_ipvti_get_ikey’ [-Wmissing-prototypes]
  route/link/ipvti.c:403:10: warning: no previous prototype for ‘rtnl_link_ipvti_get_okey’ [-Wmissing-prototypes]
  route/link/ipvti.c:437:10: warning: no previous prototype for ‘rtnl_link_ipvti_get_local’ [-Wmissing-prototypes]
  route/link/ipvti.c:471:10: warning: no previous prototype for ‘rtnl_link_ipvti_get_remote’ [-Wmissing-prototypes]

Also fix the corresponding names in documentation.

Fixes: 8f6301426a00 ("ipvti: introduce vti tunnel support")
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

link/ipvti: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

link/ipip: Add missing prototype for rtnl_link_is_ipip()

Add the function prototype for rtnl_link_is_ipip() to the public header.

This fixes the following GCC warning when compiling with
-Wmissing-prototypes:

  route/link/ipip.c:296:5: warning: no previous prototype for ‘rtnl_link_is_ipip’ [-Wmissing-prototypes]

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

link/ipip: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

link/inet6: Include own public header for function prototypes

Include the own public header in order to get function prototypes for
all public functions defined in this module.

This fixes GCC -Wmissing-prototype warnings.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

fib_lookup: Add missing prototypes to public header

Add the function prototypes for the struct flnl_result attribute access
functions to the public header fib_lookup/lookup.h

This fixes the following GCC warnings when compiling with
-Wmissing-prototypes:

  fib_lookup/lookup.c:284:5: warning: no previous prototype for ‘flnl_result_get_table_id’ [-Wmissing-prototypes]
  fib_lookup/lookup.c:289:5: warning: no previous prototype for ‘flnl_result_get_prefixlen’ [-Wmissing-prototypes]
  fib_lookup/lookup.c:294:5: warning: no previous prototype for ‘flnl_result_get_nexthop_sel’ [-Wmissing-prototypes]
  fib_lookup/lookup.c:299:5: warning: no previous prototype for ‘flnl_result_get_type’ [-Wmissing-prototypes]
  fib_lookup/lookup.c:304:5: warning: no previous prototype for ‘flnl_result_get_scope’ [-Wmissing-prototypes]
  fib_lookup/lookup.c:309:5: warning: no previous prototype for ‘flnl_result_get_error’ [-Wmissing-prototypes]

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

qdisc/red: Add missing prototypes for rtnl_red_set_limit() and rtnl_red_get_limit()

This fixes the following GCC warnings when compiling with
-Wmissing-prototypes:

  route/qdisc/red.c:136:6: warning: no previous prototype for ‘rtnl_red_set_limit’ [-Wmissing-prototypes]
  route/qdisc/red.c:152:5: warning: no previous prototype for ‘rtnl_red_get_limit’ [-Wmissing-prototypes]

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

sriov: Add missing prototype for rtnl_link_vf_vlan_free()

Add missing function prototyoe for rtnl_link_vf_vlan_free() to the
public header.

This fixes the following GCC warning when compiling with
-Wmissing-prototypes:

  route/link/sriov.c:1339:6: warning: no previous prototype for ‘rtnl_link_vf_vlan_free’ [-Wmissing-prototypes]

Fixes: 5d6e43ebef12 ("lib/route: SRIOV Parse and Read support")
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

sit: Fix invalid function prototypes in public header

Change rtnl_link_get_sit_local() rtnl_link_sit_get_local() as this is
the function's exported name.

This fixes a GCC -Wmissing-prototypes warning.

Fixes: d715b8a5f6ec ("introduce sit tunnel support")
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

lib/attr.c: check for valid length argument in nla_reserve()

https://github.com/thom311/libnl/issues/124

fopen: add O_CLOEXEC

Add O_CLOEXEC to various fopen() calls. This avoids file descriptors
leaking across an exec() boundary in a multi-threaded program. Please
see "man 2 open" for additional information about O_CLOEXEC.

Signed-off-by: Nick Kralevich <nnk@google.com>
https://github.com/thom311/libnl/pull/128

lib/utils.c: merge branch 'nickkral-psched_lazy'

https://github.com/thom311/libnl/pull/123

lib/utils.c: add mutex to get_psched_settings()

Let's add a mutex to get_psched_settings() hoping to solve worst
case scenarios when calling get_psched_settings() from multiple
threads.

Also, only read the environment variables once, who knows whether
they are modified concurrently.

lib/utils.c: ensure calling get_psched_settings() for nl_us2ticks()/nl_ticks2us()

lib/utils.c: lazy initialize user_hz and psched_hz

Rather than initializing user_hz and psched_hz when libnl is loaded,
defer initialization of these variables to the first time they are used.
This has several advantages:

1) Avoids an unnecessary permission denied error on /proc/net/psched,
which can occur on systems where /proc/net isn't readable due to
security policy.
2) Allows program code to initialize the environment variables
PROC_NET_PSCHED and/or PROC_ROOT prior to the first libnl call, giving a
program more flexibility about where libnl should look.
3) Trivially faster startup time (although unlikely to be significant).
4) Compiler may be able to prove that the get_psched_settings() function
is unreachable and optimize appropriately, because the callers never
(directly or indirectly) use this method. This could occur, for
instance, in doing dead code elimination for programs which statically
link libnl.

Signed-off-by: Nick Kralevich <nnk@google.com>
https://github.com/thom311/libnl/pull/123

sriov: merge branch 'sriov-crash-issue126'

https://github.com/thom311/libnl/issues/126
http://lists.infradead.org/pipermail/libnl/2017-January/002270.html

Signed-off-by: Thomas Haller <thaller@redhat.com>

sriov: avoid buffer overrun in rtnl_link_sriov_parse_vflist()

Fixes: 5d6e43ebef12deadf31fccfa46c0b34892675d36
Signed-off-by: Thomas Haller <thaller@redhat.com>

sriov: fix crash in rtnl_link_sriov_parse_vflist

vf_vlan_info was incorrectly indexed with "len" (the length in bytes
of the entire vfinfo_list rather than list_len (the index of the
current end of the vf_vlan_info array)

https://github.com/thom311/libnl/issues/126
http://lists.infradead.org/pipermail/libnl/2017-January/002270.html

Fixes: 5d6e43ebef12deadf31fccfa46c0b34892675d36
Signed-off-by: Thomas Haller <thaller@redhat.com>

route/tc: Remove unused function tca_set_kind()

The public prototype and the last internal user of the function were
removed in commit 8eb5b5532e ("Unified TC API") and it was unexported in
commit 4280dfb85d ("build: don't export internal symbols"), so it is
safe to remove it.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: Thomas Haller <thaller@redhat.com>
https://github.com/thom311/libnl/pull/122

libnl-3.2.29 release

Signed-off-by: Thomas Haller <thaller@redhat.com>

macsec: merge branch 'macsec-sci-endianness'

Revert API change regarding endianness of MACSec's 'sci'
argument. During 3.2.29 development, that was changed. Restore
the released 3.2.28 behavior.

http://lists.infradead.org/pipermail/libnl/2016-December/002264.html

macsec: fix endianness of sci during dump()

Signed-off-by: Thomas Haller <thaller@redhat.com>

macsec: document byte order for the SCI and port attributes

Document that the SCI is in network order while the port is in host
order.

Signed-off-by: Beniamino Galvani <bgalvani@redhat.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

Revert "macsec: fix endianness of 'sci' parameter"

The commit changed the API from libnl 3.2.28: restore the old
behavior.

This reverts commit cd758fbfee07768ff200f46d7090fa8d0e6b300f.

Signed-off-by: Beniamino Galvani <bgalvani@redhat.com>
Signed-off-by: Thomas Haller <thaller@redhat.com>

utils/trivial: rename internal _nl_offset_plus_sizeof() macro to _nl_offsetofend()

Kernel calls a very similar macro "offsetofend". So use that name
as it migth be more familiar to the reader.

Signed-off-by: Thomas Haller <thaller@redhat.com>

libnl-3.2.29-rc1 release

Signed-off-by: Thomas Haller <thaller@redhat.com>

link: fix documentation for rtnl_link_get_carrier_changes

The doxygen comment of rtnl_link_get_carrier_changes still describes the
initially implemented behaviour and was not updated, when the signature
was changed as part of #119. Adjust it accordingly.

Fixes: 0c4b2ea6d17b ("link: add support for IFLA_CARRIER_CHANGES")
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
https://github.com/thom311/libnl/pull/120