[nozzle] move set up and down to the stable API section

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

[nozzle] decouple running pre-up.d/up.d/down.d/post-down.d from interface status (part 4)

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

[nozzle] decouple running pre-up.d/up.d/down.d/post-down.d from interface status (part 3)

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

[nozzle] decouple running pre-up.d/up.d/down.d/post-down.d from interface status (part 2)

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

[nozzle] decouple running pre-up.d/up.d/down.d/post-down.d from interface status (part 1)

provide the facility to do it via nozzle_run_updown but delegate the task to
the application.

This has the benefit of much better fine grained control over errors during those
code paths.

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

[nozzle] move find_ip to internals

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

[nozzle] move ipv4 broadcast to internals

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

[nozzle] cleanup mac address functions

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

[nozzle] move code around to reflect public API and internal functions

cleanup _get_mtu in the process

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

[nozzle] move code around to reflect completion and drop copy of code from internals.c

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

[nozzle] rename _check to is_valid_nozzle, make it more useful and fixup error code checking

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

[nozzle] be more explicit on function naming (part 2)

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

[nozzle] be more explicit on function naming

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

[nozzle] start cleaning internal function names

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

[nozzle] split code around to separate libnozzle and testsuite

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

[build] split tests, man pages and build dirs

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

[nozzle] rename lib_mutex to config_mutex

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

[build] fix build on BSD

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

[cleanup] document nozzle_iface struct, drop unnecessary ifr struct and cleanup names

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

[cleanup] rename ip_addr to ipaddr

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

[cleanup] rename sockfd to ioctlfd

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

[cleanup] rename struct _ip to struct nozzle_ip and cleanup related defines

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

[nozzle] rename _config to nozzle_lib_config

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

[cleanup] start splitting the code across multiple files

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

[nozzle] cleanup and document nozzle_get_fd

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

[nozzle] cleanup and document nozzle_get_name_by_handle

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

[nozzle] cleanup and document nozzle_get_handle_by_name

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

[nozzle] cleanup and document nozzle_(re)set_mac

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

[nozzle] cleanup and document nozzle_get_mac

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

Tidy some English

[nozzle] cleanup and document nozzle_set_mtu and nozzle_reset_mtu

couldn't split those 2 because they are stricly related

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

[nozzle] cleanup and document nozzle_get_mtu

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

[nozzle] fix test on BSD

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

[nozzle] cleanup and document nozzle_get_ips (part 1)

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

[nozzle] cleanup and document nozzle_del_ip

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

[nozzle] cleanup and document nozzle_add_ip

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

[nozzle] re-order functions in order of importance

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

[nozzle] cleanup and document nozzle_set_down

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

[nozzle] cleanup and document nozzle_set_up

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

[nozzle] start to cleanup nozzle_close to be more informative and effective

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

[nozzle] cleanup nozzle_open coding style and return codes

also update the header file for Doxygen man pages

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

[nozzle] rename last tap bits to nozzle

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

[nozzle] rename API from tap_ to nozzle_

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

[nozzle] fix pkg-config to point to nozzle

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

[nozzle] rename struct _iface to nozzle_iface (looks better on public headers)

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

[nozzle] rename tap_t to nozzle_t

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

[nozzle] include net/if.h to define IFNAMSIZ instead of redefine

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

[nozzle] add Doxygen header

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

[libnozzle] rename libtap to libnozzle (part 1)

Our fearless Debian maintainer wferi made us aware that TAP is an overloaded
term already and it would be very confusing to add yet another one.

nozzle is a good pick from http://www.thesaurus.com/browse/tap

I am sure lib*cock* would have cause havoc :-)

This commit only rename files and allow builds. The API needs to be renamed too,
but that will come over time with major code cleanup as documented here:
https://trello.com/c/pyEKd4UH/396-libnozzle-make-it-production-ga-ready

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

[libtap] BSD: switch to ioctl to destroy tap devices

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

[libtap] define IFNAMSIZ for applications that don't need/want to include net/if.h

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

[libtap] update README for BSD

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

[libtap] port to freebsd and fix a bunch of small bugs

- tap_open: do mutex lock only after malloc to avoid deadlock
- fix test suite to increase randomness when testing named tap devices
- remove hardcoded paths to /bin/true /bin/false and /bin/grep

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

[libtap] tests: add better IPv6 private address space randomization

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

[libtap] tests: allow random localhost IPs for testing

- avoids clashing with machine routing
- allows parallel testing

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

UDP: Cope with EPERM returned from sendmsg/sendto

IPtables can cause -EPERM to be returned from socket sends, so
we need to be able to handle it without going into a spin.

SCTP seems to be unaffected.

Signed-off-by: Christine Caulfield <ccaulfie@redhat.com>

[misc] fix a few typos/stylistics

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

[tests] update error message in api_knet_{addrtostr,strtoaddr}.c

Signed-off-by: Chen Jingpiao <jchen@suse.com>

[tests] fix check condition in api_knet_{addrtostr,strtoaddr}.c

Signed-off-by: Chen Jingpiao <jchen@suse.com>

clear the errno in each exported API when the call success

Signed-off-by: Chen Jingpiao <jchen@suse.com>

[openssl] fix return type for pthread_self

spotted on freebsd-devel

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

[tx] fix tx_uncompressed_packets stats collect in _parse_recv_from_sock

Signed-off-by: Chen Jingpiao <jchen@suse.com>

[rx] fix incorrect packet length in _parse_recv_from_links

Signed-off-by: Chen Jingpiao <jchen@suse.com>

[tx] remove goto statement to improve readability

Signed-off-by: Chen Jingpiao <jchen@suse.com>

Add static qualifier to {compress, crypto, transport}_modules_cmds

Signed-off-by: Chen Jingpiao <jchen@suse.com>

[transports] fix transport_modules_cmd array out of range

Fixes: 18ab4888
Signed-off-by: Chen Jingpiao <jchen@suse.com>

[build] fix supported compiler warning detection

move from AC_PREPROC_IFELSE (strongly discouraged) to AC_COMPILE_IFELSE

our detection system was very weak and recent versions of clang did
show that PREPROC_IFELFE (cpp) would enable warning options that
the compiler does not support (clang).

use a full compilation test to detect what works and what doesn't.

Also expand the warning list to include new / renamed clang options
of equivalents already enabled for older versions of clang and gcc.

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

tests: fix output message

Fixes: 04537c57 ("[transports] cleanup API calls")
Signed-off-by: Chen Jingpiao <chenjingpiao@gmail.com>

fix typo

Signed-off-by: Chen Jingpiao <chenjingpiao@gmail.com>

man: update XML directory in doxyxml.c

Fixes: 31945b18 ("[man] move libknet man pages and man pages build tools to top level dir")
Signed-off-by: Chen Jingpiao <chenjingpiao@gmail.com>

misc: keep covscan happy

covscan complains about is ignoring returns from pthread_mutex_lock()
but in those 2 circumstances we don't have much choice.

This patch just keeps coverity quiet so we don't miss any real bugs it
might find.

Signed-off-by: Christine Caulfield <ccaulfie@redhat.com>

Merge pull request #155 from kronosnet/stable1-proposed

Stable1 proposed

links: Don't close loopback link

knet_link_clear_config would incorrectly clear the local
loopback link if the last 'real' link to an external node
was closed. We didn't check the nodeid in the code, just
the link number when clearing the loopback flag.

Signed-off-by: Christine Caulfield <ccaulfie@redhat.com>

Import key with length not dividable by wrap key block size

Wrapping of the key is standard crypto operation which needs data
aligned to cipher block size, otherwise it fails.

Possible solution is to use a zero filled buffer with size aligned to
required wrap key block size. Private key is copied to the beginning of
the buffer and unwrap operation keeps using only required private key
size.

Signed-off-by: Jan Friesse <jfriesse@redhat.com>

[tests] clear errno to avoid incorrect propagation from void fn call

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

[tests] fix select(2) handling (mostly affects slow machines)

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

[threads] make it easier to debug thread startup / stop operations with pretty names

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

Use gitarchive-version for spec file

Signed-off-by: Jan Friesse <jfriesse@redhat.com>

git-version-gen fail on UNKNOWN version

Signed-off-by: Jan Friesse <jfriesse@redhat.com>

Do not match lightweight tags in git-version-gen

Fix leftover from 175b517fd71d999150aa701ec847510490a18a6e which added
matching of lightweight git tags.

Signed-off-by: Jan Friesse <jfriesse@redhat.com>

Support for git archive stored tags

Attempt to solve problem with git archive generated tarballs
(used for example by github when release is downloaded) which are no
longer git tree and (in contrast to officially released tarballs) also
doesn't contain .tarball-version file so git-version-gen script simply
cannot obtain valid version info.

Solution is based on using gitattributes which is instructs git to
replace string in the .gitarchivever file by known ref names.
git-version-gen is enhanced to support this file and tries to parse
any string which looks like "tag: v[0-9][^,)]*". If such string
is found it's used as a version. This file is used as a last attempt and
other methods (.tarball-version, git abbrev) have precedence.

Based on idea stated by Jan Pokorný <jpokorny@redhat.com>.

Signed-off-by: Jan Friesse <jfriesse@redhat.com>

[tests] drop arm exceptions as they have been fixed in glibc

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

[tests] cleanup nss memcheck supports

add one more exception for newer nss (spotted on fedora rawhide)
drop FreeBSD excetions as we don't support running under valgrind on BSD

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

Merge pull request #150 from kronosnet/stable1-proposed

stable1-proposed

[threads] use internal array to track status of threads (running/stopped) vs a random sleep()

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

[tests] don't redefine symbols from the main library

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

[tests] adds more overrides for fedora28/arm/glibc

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

[handle] reduce usage of handle_config_mutex to wrap only global lib init

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

[logging] report error in case logging buffer is bigger than PIPE_BUF

See https://github.com/kronosnet/kronosnet/pull/148

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

[handle] drop init_lock_done as it was used only by locked version of log_msg

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

[logging] drop locking inside log_msg (part 1)

looking at the code, there are only few calls to log_msg happening outside
of locking context (error reports when failing to grab the lock).
Everything else appears to be safe.

drop the attempt to lock inside log_msg for now, part 2 will address those
few calls that can happen outside of locking context.

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

[handle] drop call to log_msg that would have no effect

logging is not setup yet at this stage and we can't log just yet.

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

[tests] ignore nss internal memory leak (3.38+) non recurring

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

[tests] reference correct library in memcheck comments

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

[tests] add another lzma exception

spotted on Debian Experimental, the latest toolchain appears to add
full path to objects and valgrind would generate an error.

add a generic catch-all path exception.

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

coding style: add parentheses after the sizeof operator

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

log_msg: no point in putting NL after the final NUL

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

knet_log_msg: fix documentation: the closing newline is added by flush_logs

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

_logthread: while loop instead of goto

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

_logthread: select() returns the number of set bits, as an int

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