link: Ignore bridging notifications in link cache manager

RTM_DELLINK and RTM_NEWLINK bridging notifications should not delete/add
entries to a link cache.

cache: event_filter() cache operation to filter notifications

Certain notifications need to be filtered out and should not be applied to
a cache when a cache is handled by a cache manager.

utils: Initialize list head after freeing translation list

Problem found and fix proposed by Andrew Kraslavsky <andykras@hotmail.com>

Only use the MULTIPATH attribute when adding routes with more than one next hop.

Only use the MULTIPATH attribute when adding routes with more than one
next hop.
This solves issues with two scenarios:
1. Adding an IPv4 route to a kernel configured without
CONFIG_IP_ROUTE_MULTIPATH=y.
2. Adding an IPv6 route in general, since the MULTIPATH attribute is not
supported there.

Signed-off-by: Michael Altizer <xiche@verizon.net>

Don't install CLI header files when --disable-cli has been configured.

Signed-off-by: Michael Altizer <xiche@verizon.net>

Fix rtnl_link object memory leak when freeing rtnl_addr objects.

Signed-off-by: Michael Altizer <xiche@verizon.net>

Add libnl-cli-3.0 to pkg-config tool

Signed-off-by: Jiří Župka <jzupka@redhat.com>

3.2.1 release

Added more details on the changes to where and how libnl is being
instaslled since 3.2.0

Provide micro version in <netlink/version.h>

Ingnore src/nl-link-enslave and nl-link-release

Bonding: Fix header guard of <netlink/route/link/bonding.h>

(Would be a good idea to change the header guard name when c&p
 an existing header file to create a new one.)

bonding: Install <netlink/route/link/bonding.h>

bonding: API to create/enslave/release

Although it has been possible to create bonding devices, enslave and
release using the regular link API. The added API simplifies usage
and hides some of the compatibility logic.

F.e. enslave() and release() will both verify that the master assignment
has in fact been changed and return -NLE_OPNOTSUPP if it did not.

Also the API will make sure to use RTM_NEWLINK or RTM_SETLINK depending
on what is availble.

Examples are provided in src/ as nl-link-enslave.c and nl-link-release.c

link: Eat ACK followed by RTM_NEWLINK when requesting single link

When requesting a single link with RTM_GETLINK (no dump) the
RTM_NEWLINK carrying the answer will be followed by an ACK
we have to wait for.

Fix a73cb2f26 fallout to allow building in separate directory

Commit a73cb2f26932d90a2d47a28e9c524e7f33dcffbd missed to change the
EXTRA_DATA section

tools: Use LDADD and link against .la files instead of LDFLAGS and -llib

route: Remove dead link_cache variable

addr: Add missing header to <netlink/route/addr.h>

Fixes a gcc warning

Switch to libtool versioning system

It has been a request that multiple libnl versions should be installabe
in parallel.

In order to achieve this, the basename of the library was changed to
libnl-3 which reflects the 3rd generation of libnl APIs. It also means
that release based library versioning is left behind and libtool
versioning is used instead.

Projects using pkgconfig will automatically link against the new library
basename and will not notice a difference.

The SO versioning is based on the glib model:
  current := 100 * minor + micro - revision
  revision := revision
  age := age (number of backwards compatible versions)

build: always install files into /etc/libnl

I observed that with the RedHat build target in the Open Build
Service, files were put into /etc rather than /etc/libnl.
Self-referential variables are a bad idea, and so just avoid this.

Inform users about changed include location at end of configure script

Install headers in ${includedir}/libnl3

This allows for multiple major versions to be installed in parallel. Pkg-config
files are adapted to provide appropriate cflags to find new header locations.

lib: Use @MAJ_VERSION and @MAJ_MINOR@ for -version-info

asciidoc xhtml stylesheet

prepare for 3.2 release

Fall back to using libnl-*.3.pc files. It may be common to have multiple
generations of libnl installed but unlikely within the same major release.

remove dist lines for non existing files

Remove GPL-3.0 license file.

There is no code licensed GPL-3.0 so we might as well remove this file.

fix license of lib/route/pktloc.c

As Jan Engelhardt pointed out, pktloc.c is currently licenses under the GPL
rather than the LGPL. This is a result of its previous existance in src/
which is GPL licensed. I missed to change its license when moving it to
lib/. Since I am the only contributor to the code, I am changing the license
hereby.

Do not require python and swig to be present

Allows building without python and swig. Readd this using --enable-python
at some point.

64bit unaligned access

This patch fixes an unaligned access for IPv6.  On systems with strict alignment requirements, the unaligned access will either result in garbage data or a crash.

genl: genl_ctrl_grp_by_name: fix retval in case group id not found

Now the return value in case group id is not find is 0. Change it to
return -NLE_OBJ_NOTFOUND which makes more sense. This should not break
anything because genl_ctrl_grp_by_name is static and called only from
genl_ctrl_resolve_grp. genl_ctrl_resolve_grp already might return
-NLE_OBJ_NOTFOUND.

Signed-off-by: Jiri Pirko <jpirko@redhat.com>

socket: fix two typos

Signed-off-by: Jiri Pirko <jpirko@redhat.com>

socket: introduce nl_socket_modify_err_cb

This function does the same as nl_socket_modify_cb except for error
callback

Signed-off-by: Jiri Pirko <jpirko@redhat.com>

nl-link-name2ifindex: fix usage text

Signed-off-by: Jiri Pirko <jpirko@redhat.com>

Local port leak on nl_socket_alloc/nl_socket_set_local_port(, 0)

I've found a bug in the following scenario (fragment of code):

while (1) {
    struct nl_sock *sk = nl_socket_alloc();

    if (sk == NULL) {
        fprintf(stderr, "Failed to allocate nl socket\n");
break;
    }
    nl_socket_set_local_port(sk, 0);
    nl_socket_free(sk);
}

The problem is that nl_socket_set_local_port(, 0) does not
release local port if it is allocated before.

htb: fix misplaced memset() overwriting already set htb prio option

Reported by Richard Cesar <spidla@spidlas.cz>

doc: need to include @srcdir@/src not @top_srcdir@/src

python: Include python/ in distribution and provide a README on how to build & install

link: fix unaligned access to 64bit link stats

Include all files necessary to build documentation in distribution

make port map thread safe

This patch has been in Fedora and RHEL for a while. It adds a mutex
to protect the port map from concurrent thread accesses.

Original patch from Stefan Berger <stefanb@us.ibm.com>.
Modified to use configure.in to check for libpthread

Avoid freeing memory if vasprintf() failed

Founds this bugfix in Fedora's SOURCES for libnl. Not sure who the
original author is but the bug should be fixed upstream as well.

Allow building documentation out of the tree

Patch originally provided by Gaudenz Steinlin <gaudenz@debian.org>

Put "break" inside the "if" block in route_compare()

Make some functions and global variables static

Remove write-only variables in lib/cache.c

3.1 release

Updated link documentation

- API documentation
- developer guide
- enabled doxygen navbar
- fixed css

Hack doxygen CSS to avoid stupid margins in modules listing

include <linux/if.h> from <netlink/route/link.h> to export IFF_* flags

use rtnl_link_set/get_type()

removed autogenerated swig interface files from git tree

VLAN: rtnl_link_is_vlan() function and API documentation

Use 'link type' instead of 'link info type'

The term 'link type' is much more easier to understand than 'link info type'

rtnl_link_set/get_info_type() left around for backwards compatibility

ignore python build directory

ignore various generated files

bonding link module

more documentation updates
 - improved stylesheets for both doxygen and asciidoc
 - use of xml doxygen layout
 - python script to resolve <<foo>> asciidoc references to
   <<foo, title>> based on the target caption
 - graphics for netlink and netlink error headers
 - more link documentation

Include IFLA_LINK in link messages

Documentation updates

- update to asciidoc 8.6.5
- added doc/README
- switched to toc2
- started adding link documentation

documentation updates

Support for NLM_F_INTR

Check if a dump was interrupted and needs to be redone

Merge branch 'master' of master.kernel.org:/pub/scm/libs/netlink/libnl

Don't redefine offsetof when already defined by e.g. stddef.h

socket: Set SOCK_CLOEXEC if available

Reported by Марк Коренберг <socketpair@gmail.com>

neigh: include ndm flags while building message

Reported by jeff courington <jeff_courington@hotmail.com>

Work on libnl-python

Changes too numerous to describe them

HTB: Add support for level and direct pkt stats, complete access functions

- Adds support for htb class level and direct packet
  statistics
- Complete all get/set access functions
- Complete error handling
- Documentation

Update local copies of include/linux

Provide documentation for rtnl_tc_data() and rtnl_tc_data_check()

Add nl_rate2str()

Add rtnl_tc_data_check()

Behaves like rtnl_tc_data() but verifies that the traffic control
object is of the expected type.

Provide silent variation of nl_cache_require()

Use in addr and tc layer to avoid warnings being printed if no
cache is available.

python interface to netlink protocols

currently includes experimental support for links, addresses
and some traffic control

Use name "global" instead of "universe" for largest scope

This name is more widespread and easier to understand even though
it is less correct.

addr: store link object and provide rtnl_addr_get()

stores rtnl_link object in address if cache is availble. Provide access
via rtnl_addr_get_link() and rtnl_addr_set_link().

Add rtnl_addr_get() which searches a address cache for an address
matching ifindex and local address.

Adding rule with "From" option doesn't work

OK i found the bug, is is the patch !!

Can someone push it into the git tree ?
Regards,

Romary Sonrier

=========================
My analysis:

I guess that the pay load is *struct fib_rule_hdr*
struct fib_rule_hdr {
__u8 family;
__u8 dst_len;
__u8 src_len;
__u8 tos;

__u8 table;
__u8 res1; /* reserved */
__u8 res2; /* reserved */
__u8 action;

__u32 flags;
};

> [PAYLOAD] 12 octets
> 02 00 00 00 0a 00 00 01 00 00 00 00 ............
so

family = 02 , fine

table = 10, fine
but
src_len =0 !!!! should be 32

Add missing declaration of rtnl_tc_get_link()

Provide rtnl_tc_get_link() and fix link refcnt

Adds rtnl_tc_get_link() returning the link associated with a tc
object.

Don't clone link associated with tc object when cloning the tc
object, refer to same link instead and bump refcnt.

Fix refcnt leak when assigning link in msg parser.

Fix clone() of AF specific link data

Passed the original link to ao_clone() instead of the new one.

Set tc->tc_link if link cache is available

When parsing tc objects, lookup link in link cache if such a
cache is available.

Improve readability of classid string representation

1: is more readable than 01:

Expose <netlink/route/link/inet.h>

Support link info types with no payload

Add support for ARPHRD_NONE

export rtnl_link_add() and rtnl_link_build_add_request()

dummy interface support

Support for rtnl_link_add()

API for adding virtual links

Improve rtnl_link_change()  behaviour

- avoid unncessary name change requests
    The kernel does not check if the specified IFNAME is different
    from the current name. It assumes that if IFNAME and ifindex
    are both specified, a name change is requested. Therefore avoid
    specyfing IFNAME if ifindex is provided and original and new
    name are identical.
- move link building to own function (to allow link add later on)
- error if immutable changes have been made
- better documentation

fix line removed by mistake

This line was removed by mistake in the last commit

API to issue direct GET requests to the kernel

Provide nl_pickup() to pick up an answer from a netlink request and parse
it using the supplied parser.

Add rtnl_link_get_kernel() which sends an RTM_GETLINK to the kernel to
fetch a single link directly from the kernel. This can be faster than
dumping the whole table, especially if lots of links are configured.

link: Provide rtnl_link_delete() to delete virtual links

Takes a link object and extracts ifindex and name to build a deletion request
message to delete virtual network links.

link: Make return and argument type match

Provide nl_cache_set_arg{1,2}() to specify cache args

Added based on a RFE. This is required if allocation and
(re)filling is to be splitted.

Provide nl_object_dump_buf() to easily dump to buffers

Initialize dump buffer in case caller missed it

(Set an end to all the bug reports of applications failing
to do so.)

Provide TC_HANDLE(maj, min) macro to generate tc handles

Move to asciidoc

The core library documentation has been converted to use asciidoc. It
provides better flexibility in creating documentation such as tables
and more powerful formatting rules.

The doxygen based API reference remains and three scripts have been
added to:
 - gen-tags.sh: extract tag information from doxygen reference
 - tags2dict.sh: generate a name=link dictionary file
 - doxygen-link.py: replace all references in the asciidoc documentation
                    refering to API functions, struct, etc. with a link
    into the doxygen API reference.

'make gendoc' continue to generate all documentation.

The following tools are required to generate documentation:
 - asciidoc
 - mscgen
 - asciidoc mscgen plugin
 - pygments
 - xmlstarlet

trafic class/classifer API improvements and documentation

- removed dead functions in header files
- deprecated rtnl_class_foreach_*() functions due to their missing
  handling possibility of OOM situations
- improved API documentation

Qdisc API improvements and documentation

Deprecated the functions rtnl_qdisc_change() and
rtnl_qdisc_build_change_request() for their lack of being able to
specify flags. The new functions rtnl_qdisc_update() and
rtnl_qdisc_build_update_request() may be used instead. The old
functions are still available though. However, rtnl_qdisc_update()
no longer implies NLM_F_REPLACE, it has to specified implicitely
to allow updating a qdisc without risking to replace another qdisc.

Included detailed documentation of qdisc addition/update/deletion.

Introduced APPBUG() macro to let application developer know of
API abuse.