[logging] fix log target of recently added API calls

spotted during sctp testing

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] add more extensive test for links_acl_ip

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

acl: Fix English in commments

[access lists] add public API tests

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] improve checks on various data types

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] test implicit access lists management for UDP, SCTP and LOOPBACK

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] add external API calls to manage access lists

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] add documentation for enable_access_list

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[transports / access list] add internal API to gather which fd to use for access lists given a certain link struct

this is required for the external API that has to be transport indepedent

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] rename ip1/2 to ss1/2 to keep it more generic

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] use arrays to access per-protocol functions

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] use better name for fd_tracker structure

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] confine access lists data structs within the protocol itself

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] add errno around and start using them

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] fix build on BSD and add some include files around

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] make internal API consistent

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[links] rename tranport_type to transport to avoid confusion (part 2)

complements be9d053efafc822cabd696914d53b5dfe25fb4fd due to early
cherry-pick of 7033ddab505a0cf3655115fe5037579b7c882a8c

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[links] rename transport_type to transport to avoid confusion

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] remove 2 unnecessary wrappers

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] cleanup API a bit

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] more use of generic wrappers and remove duplicate code

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] move access lists structs and data types to links_acl.*

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] move all acl wrappers to links_acl* and split links_acl_ip to their own files

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] fix build on freebsd

don't use malloc.h, obsoleted by stdlib.h
define s6_addr32 that's only available in kernel space

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] add access lists support to sctp

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] enable generic access lists only for protocols that use them

protocols such as SCTP that use their own access list tracking will
need to setup access lists in transport_link_set/clear_config

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] enable access lists for GENERIC_ACL protocols (udp for example)

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] allow knet_bench to enable/disable access lists

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] add tests for default access lists

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] automatically add and remove point to point access lists

those are not used just yet.

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[handle] properly initialize fd tracker buffers

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[access lists] make code more generic to accept more than IP protocol and start to bind it to each fd

access lists are unique per file descriptor, each fd can have its own protocol and list.

remane around ipcheck* with check* to be more generic.

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[transports] add information about the nature of the transport and supported access lists

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[acl] add knet_handle_enable_access_lists api call

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[acl] move poc-code into libknet dir and rename to links_acl.*

code is not integrated yet and test suite can´t run standalone

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

Merge pull request #211 from kronosnet/stable1-proposed

stable1-proposed

[udp] improve error message decoding from ICMP errors

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[udp] use defines vs hardcoded numbers

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[man] fix libknet.h for errors detected by newly added test

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[tests] add man page check to verify doxy header order and definitions

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[man] fix knet_host_set_policy parameters order

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

man: Tidy more man pages

Followup to previous 'tidy'

man: Tidy manpages (#215)

* man: Tidy manpages for libnozzle

doxygen works in mysterious ways, adding a blank line before
@brief makes the lines following that much tidier.

So now instead of

nozzle_close nozzle - pointer to the nozzle struct to destroy

we get:

nozzle_close

       nozzle - pointer to the nozzle struct to destroy

* doxyxml: Cope with pointers-to-pointers passed as params

Double pointers showed as ' * *name' when they should be '  **name'.

Also tidy STRUCTURES display so that they are not indented too much,

* man: Similar @brief fixes for libknet.h

* doxyxml: Tidy descriptions of functions as parameters

If a complex function pointer was passed as a parameter then doxyxml
tryied to line up all the other parameters with it - making a mess
by having lots of blank space between the type and the name.

Now we enforce a maximum type length (a line-ish) so that shorter
tyopes will line up OK and the really long ones will be left to their
own devices.

[crypto] remove libnss 3des support

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

reduce minimum crypto key size to 1024bit

Since the key is used for AES/3DES and HMAC operations only, this is
safe. AES/3DES use keys in the 128- to 256-bit range, HMAC with
MD5/SHA1/SHA2 should use keys with a minimum of 128- to 512-bit (in both
cases, depending on the actual algorithm used).

This reduction also keeps knet compatible with existing Corosync 2.x
keyfiles, which are 1024-bit.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

[build] add another exception to valgrind nss combo

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

Merge pull request #199 from kronosnet/stable1-proposed

stable1-proposed

[global] update copyright across the board

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

link: Check address families on a link always match

We can't create a link with a source address of a different
address family than the destination as all sends will fail.

[transports] fix usage after free of transport info memory

spotted while implementing functional testing for rekey.

when configuring multiple hosts (3+) with 1 link sharing
the same listener, but NOT enabling the link, if the user
attempts to shutdown knet_h, the first call to knet_link_clear_config
will not recognize that the listener is still in use by another
link and will release all resources.
At the time of invoking clear_config on the second link, transport_clear_config
will access already freed memory, or alternatively, a call to
link_enable would cause a crash because the listener does no
longer exist.

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[tests] test rpm should match currently installed version of libknet

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

manpages: Document enums (#206)

And also fix a bug in structure printing that caused it to print the wrong name for a struct.

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[tests] allow shipping of the test suite

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[host] increase log level for currently used links

bug reported on IRC by yuanren

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[transport] don´t specify destination address for connection oriented protocols

some implementations of sendto/sendmsg expects dst_addr / msg_name to be
empty for connection oriented protocols such as SCTP (spotted on FreeBSD).

Make sure to clear or not pass those information on connection oriented protocol.

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[man] fix man page build dependency for BSD Make

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[tests] add ability to create ipv6 sockets

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[man] fix libnozzle man page generation target

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[tests] fix knet_bench help text

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

doxyxml: Don't crash if there's no doxygen stanza for a function

[man] add test to make sure all generated man pages are being shipped / installed

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

Merge pull request #183 from kronosnet/stable1-proposed

Stable1 proposed

[common] fix not reporting a true dlinfo error cause

This was automatically caught with GCC 9
("'%s' directive argument is null").

Signed-off-by: Jan Pokorný <jpokorny@redhat.com>

[libknet] allow better handling of internal threads

while playing around adding some dynamic internal threads, i did
hit a limitation with allocating a dynamic THREAD_MAX value based
on OS availability of certain features.

this patch changes from dynamic to static allocation of threads, that
can always be increased if necessary without breaking anything.

simplify thread numbering and conditionals on OS features

improve debugging of thread status by adding a 2 phase thread registration
process:

- register a thread before starting it with THREAD_REGISTERED before pthread_create
- set a thread status as started with THREAD_STARTED

any failure in between can now be detected in debugging logs.

Note for developers: all threads *MUST* be registered/started before
wait_all_threads_status check in handle.c.

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[build] enable all compilers warnings at once

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libknet] expose API version in libknet.h to make it easier for applications to add conditional builds

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

Skip SCTP event initialization if the protocol is not supported

We may not want to actually use SCTP after all, and if we do, we won't
get to use the uninitialized subscribe struct anyway.  Thus aborting
handle initialization is premature here.

Fix nozzle man dates to SOURCE_EPOCH, just like we fix the libknet ones

[rpm] ship libnozzle man pages

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

Fix typos

Install the nozzle man pages

Merge pull request #162 from kronosnet/stable1-proposed

stable1 proposed

[knet] fix build with older openssl versions

spotted on freebsd-devel

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[global] update notes to package maintainers

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

Tabify

Use the kernel struct size for SCTP event subscription

Signed-off-by: Ferenc Wágner <wferi@debian.org>

[libnozzle] fix variable assignment order

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[global] fix copyright across

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add nozzle_get_ips tests and kill nozzle_test

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add nozzle_del_ip tests

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] fix del_ip removal of Ipv6 addresses when MTU < 1280

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add nozzle_add_ip tests

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add api_nozzle_run_updown tests

and cleanup lots of craft around related to the old testing method
for the same API

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add internal execute_bin_sh_command test

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] cleanup tests Makefile.am

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add nozzle_get_fd test

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add nozzle_get_name_by_handle test

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add nozzle_get_handle_by_name tests

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add nozzle_get_mac / nozzle_set_mac / nozzle_reset_mac tests

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add nozzle_get_mtu / nozzle_set_mtu / nozzle_reset_mtu tests

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] move multi nozzle device test to api_nozzle_open

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] clean up code in set_up test

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] improve set_down test

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add api_nozzle_set_(up|down) tests

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] drop unnecessary indirection to call set_down

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] drop redundant check

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add api_nozzle_close_test

also, share more code in test-common, fix another BSD build issue

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] fix test suite build on BSD

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] add api_nozzle_open test

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>

[libnozzle] prepare Makefile.am to explode with API tests

also fix nozzle_test for using global compiler flags

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>