[openssl] fix return type for pthread_self

spotted on freebsd-devel

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

Merge pull request #171 from ppiao/fix-stats-collect-in-tx

[tx] fix tx_uncompressed_packets stats collect in _parse_recv_from_sock

[tx] fix tx_uncompressed_packets stats collect in _parse_recv_from_sock

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

Merge pull request #168 from ppiao/fix-packet-length-in-rx

[rx] fix incorrect packet length in _parse_recv_from_links

Merge branch 'master' into fix-packet-length-in-rx

Merge pull request #167 from ppiao/minor-changes

Minor changes: add static qualifier and remove goto statement

[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>

Merge pull request #166 from ppiao/fix-transports

[transports] fix transport_modules_cmd array out of range

[transports] fix transport_modules_cmd array out of range

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

Merge pull request #165 from kronosnet/fix-compiler-options

[build] fix supported compiler warning detection

[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>

Merge pull request #163 from ppiao/master

man: update XML directory in doxyxml.c

Merge branch 'master' into master

Merge pull request #164 from ppiao/fix-typo

Fix typo

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>

Merge pull request #161 from kronosnet/pthread-keep-covscan-quiet

misc: keep covscan happy

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 #160 from kronosnet/dont-close-loopback

links: Don't close loopback link

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>

Merge pull request #159 from jfriesse/nss-misalligned-private-key

Import key with length not dividable by wrap key block size

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>

Merge pull request #158 from kronosnet/fix-select

Fix select

[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>

Merge pull request #157 from kronosnet/threads

[threads] make it easier to debug thread startup / stop operations wi…

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

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

Merge pull request #156 from jfriesse/gitarchivever

Use git archive stored version tag

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>

Merge pull request #154 from kronosnet/nss-fixes

small ci / tests fixes

[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 #153 from kronosnet/locking-cleanup

Locking cleanup

[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>

Merge pull request #152 from kronosnet/ci-fixes

Ci fixes

[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>

Merge pull request #151 from kronosnet/deb-exp

[tests] add another lzma exception

[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>

Merge pull request #149 from kronosnet/log-cleanup

Log cleanup

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>

flush_logs: don't ever print past the message buffer

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

flush_logs: while loop instead of goto

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

flush_logs: a for loop is more straightforward here

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

flush_logs: don't repeat the type of msg

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

flush_logs: the reads will overwrite the entire msg structure

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

flush_logs: remove redundant condition

The read loop is always entered, resetting len and returning from the
function if it isn't greater than 0.

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

Merge pull request #146 from kronosnet/extra-checks

[crypto] NSS_NoDB_Init: the parameter is reserved, must be NULL

[crypto] NSS_NoDB_Init: the parameter is reserved, must be NULL

Thanks to Feri for spotting it in corosync

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

Merge pull request #144 from kronosnet/extra-checks

[crypto] improve checks around RX packet size

[crypto] improve checks around RX packet size

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

Merge pull request #143 from kronosnet/source_epoch

Source epoch and version check

build: look for source_epoch in srcdir

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

build: check for SOURCE_DATE_EPOCH before turning to Git for an epoch

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

[build] fail and provide proper error if source tree version is unknown

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>
Signed-off-by: Ferenc Wágner <wferi@debian.org>

build: git -C is a relatively new option, avoid it

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

build: accomodate to the actual date utility

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

spec: use the source epoch here as well

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

man: use new doxyxml date options to build man pages

Based on Fabio's original patch.

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>
Signed-off-by: Ferenc Wágner <wferi@debian.org>

man: Add date & year options to doxyxml

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

build: distribute the source_epoch file

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

build: determine SOURCE_EPOCH during configuration

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

Merge pull request #139 from kronosnet/new-compress-level-checks

[compress] delegate compress_level validation to compress libraries

[compress] extend check to compare end buffer data

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

[compress] update valgrind exception file to match new lzma entry points

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

[compress] delegate compress_level validation to compress libraries

- make val_level internal api call optional
- keep lzo2 val_level around due to the specific nature of compress_level values
- add internal compress_lib_test to do a round-robin (compress/decompress) check with provided values
- drop unnecessary val_level checks around
- update internal compress API include file
- adjust api_knet_handle_compress test to use a known bad value for zlib validation

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

Merge pull request #138 from kronosnet/host-filter-fix

[tx/rx] improve error checking when running external code

[tx/rx] improve error checking when running external code

libknet cannot guarantee that the dst_host_filter is sane nor that it cannot
be exploited to return garbage to knet.

there is an infinitesimal possibility that, if the plugin returns total crap,
knet could crash by trying to access out-of-bound memory.

Prevent that by adding specific checks both in TX/RX.

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

Merge pull request #137 from kronosnet/fix-log-gen

[build] fix ChangeLog generation in release tarball

[build] fix ChangeLog generation in release tarball

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

Merge pull request #124 from kronosnet/test_handle

Unprivileged handles

Indicate new API call in libtool version info

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

build: provide -version-info directly instead of -version-number

The info components carry the real OS-independent library version
information, from which the version number is computed differently
on different systems. According to the libtool manual "new projects
should use the -version-info flag instead".

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

New API function: knet_handle_new_ex() taking handle flags

For now, the only supported flag is KNET_HANDLE_FLAG_PRIVILEGED,
which is also the default in knet_handle_new().

The tests are adapted to switch this flag off, which makes them pass
in default environments without privileges.

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

Merge pull request #136 from chrissie-c/fix-udp-einval

UDP: Don't keep retrying send if we get EINVAL

UDP: Don't keep retrying send if we get EINVAL

UDP can return EINVAL if the route is down or not configured, this can
cause knet to 'spin' (well, ish, there is a usleep in there).

With this patch in place knet doesn't hog the CPU anywhere near so much
(and it does recover if the link returns).

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

Merge pull request #134 from kronosnet/move-man

Move man

doxyxml: Use overridden XML_DIR for reading structures

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

[man] fix path to stamp file

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

[man] move kronosnetd man pages

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

[man] move libknet man pages and man pages build tools to top level dir

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

Merge pull request #127 from kronosnet/configure-hardening2

Configure hardening

[build] simplify Doxygen test based on Feri's input

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