Merge pull request #62 from chrissie-c/new-apis2

libknet: Add APIs to query crypto & compression models

libknet: Add APIs to query crypto & compression models

Also add API call to clear the handle and link stats

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

Merge pull request #60 from wferi/autotools

Autotools

configure: initialize pkg-config with its standard macro

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

configure: remove some superfluous checks

We've got no C++ code in here.
No need for ranlib when using libtool.

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

Merge pull request #59 from kronosnet/ci-fixes

Ci fixes

[build] enable valgrind automatic supressions generation during make check-*

this is useful to create exception when false errors are found on CI run

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

[tests] add exception for lzma on Ubuntu 17.10 i386

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

Merge pull request #58 from liu4480/run-autogen

add "runautogen" option for compiling

add "runautogen" option for compiling

Merge pull request #56 from kronosnet/fabbione-cleanup

[testing] drop travis support as we are moving all builders to https:…

[testing] drop travis support as we are moving all builders to https://ci.kronosnet.org/

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

Merge pull request #57 from kronosnet/fabbione-udp-fixes

[udp] fix incorrect return code

[udp] fix incorrect return code

UDP TX is tricky and racy and in theory TX never fails, as UDP is allowed
to drop packets internally anyway.

In some race condition situations, we can endup in a situation where:
- thread X attempts to send a packet
- socket receives an ICMP back (for whatever reasons) generated
  by some other random thread/packet combo
- thread X detects an error in TX and decode the error (correct)

pre patch:
- thread X would treat that as error and drop the packet
post patch:
- thread X should simply resend the packet

simply also the ordering of parsing socket error code to make it easier to read

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

Merge pull request #53 from digimer/master

Added 'gcc' to .spec BuildRequires

* Added a requirement for 'gcc' as that is not installed on fedora minimal by default, breaking rpmbuild's build attempt.

Signed-off-by: Digimer <digimer@alteeve.ca>

[global] use memmove instead of memcpy

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

[build] more casting fixes

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

[build] more signed/unsigned fixes

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

[build] fix some signed/unsigned declarations

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

[build] allow make check-memcheck and check-hellgrind only if valgrind is avalable

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

[build] bump minimum requirements for automake

this is required for distclean + subdir-object to work properly

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

[tests] fix build and cast aligment on Debian/sparc64 unstable (gcc7)

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

[build] cleanup whitespace

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

[build] disable poc code by default

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

[dlopen] drop support for unload of libraries

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

[openssl] check for errors when initializing openssl1.1

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

[build] fix build on freebsd11 (and others)

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

[tests] fix build with no nss

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

[tests] move dump_all to callback section

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

[tests] add exception for lzma on Debian Unstable

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

[test] drop useless library count check

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

[crypto] add support for openssl1.0.2 and openssl1.1.0 to accomodate Debian unstable

Please note that openssl1.1.0 needs more testing

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

[host] Never mark loopback-connected hosts as unreachable

There's no point in checking the links of a host with a LOOPBACK
link - it's always up and reachable.

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

[openssl] update libknet.h to document openssl support

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

[openssl] extend lib load/unload to cover for openssl

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

[openssl] filter "none" from *getbyname openssl functions

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

[openssl] drop unrequired OPENSSL_config

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

[openssl] revert debugging code

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

[openssl] go back to the original minimal keysize

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

[openssl] move to dlopen model

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

[nss] protect shutdown from segfaults in case we cannot resolve all syms

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

[openssl] add api_knet_handle_crypto test

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

[openssl] drop incorrect keylen management code

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

[crypto] add openssl support (first cut)

still needs dlopen and memcheck love + sync with nss key size

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

[cryto] nss: add namespace to internal functions

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

[build] fix casting with gcc-7 (debian unstable)

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

[debian] fix tests/ linking with debian unstable

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

[debian] fix build deps (pass 3 or 4)

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

[debian] be more specific about builddeps

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

[debian] fix build dependency in control file

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

[test] fix conditional build of int_lib_load_unload

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

[global] bump lib soname to 1.0.0

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

[dlopen] compress: drop unnecessary tracker and use init array instead

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

[compress] move tracking update into a write lock context

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

[dlopen] compress: fix load/unload of libraries and add extensive test

each handle should only unload libraries that has loaded or referenced
on exit, attempt to close all libraries from that handle
on compress_fini close only the one that's open

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

[compress] make sure to unload resources before re-init the new config

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

[global] rename link.(c|h) to links.(c|h) as they conflict with global /usr/include/link.h

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

[compress] split check_lib_init into 2 functions

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

[crypto] simplify lib init code and fix error handling

also update the valgrind exception file

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

[dlopen] move all dlopen lib names at the top of the respective files

makes it easier to figure out library supported version

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

[dlopen] add suppress entries for internal NSPR/NSS memory leaks (part 2)

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

[docs] fix a typo

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

[dlopen] fix crypto/compress internal name space for similar functions

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

[dlopen] add suppress entries for internal NSPR/NSS memory leaks

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

[dlopen] use RTLD_NOW instead of RTLD_LAZY to avoid perf hit on fast path

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

[crypto] partially revert and partially fix b22ef0ca65d687dbc

nss/nspr still leaks approx 2kb on unload, but it's much cleaner
now than the many Kb before those fixes.

https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS/Memory_allocation

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

[handle] remove knet handle tracker

reload of shlib should be done on a per handle base to avoid complex
global functions.

For example, if you application is using 2 knet_h and you need to reload
libbz2, then switch both knet_h to use another compression method (or none)
and switch back to libbz2.

Similar can be done with nss/openssl once openssl support is added.

A global function to reload all library would a very complex error reporting
mechanism and could leave some handles improperly configured.

The application would have a better understanding of the needs at any given
time.

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

[crypto] fix nss unload/reload memory leak

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

[dlopen] if the library is a symlink, display the symlink info to the final binary

it cannot prevent symlink attacks but definetely helps to identify them

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

[common] don't hardcode max path size

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

[dlclose] add logging messages around unloading libraries

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

[common] use a standard fn to dlopen libraries and print load path info

might be useful to detect LD_PRELOAD attack

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

[crypto] nss: be more clear about shut down/unload situation

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

[tests] fix crypto test to deal with new nss dlopen space

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

[nss] clear error before dlopen

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

[nss] warn the users that library cannot be unloaded

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

[crypto] fix libnss horror show to avoid leaking memory

libnss simply can't be unloaded at runtime as it leaks memory.
switch back to the hold atexit code and don't allow unloading
of the library.

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

[crypto] handle properly errors reported from .init functions

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

[crypto] fix libnss unload crash and remove that atexit horror show

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

[crypto] fix libref count the same way as compress

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

[compress] centralize handling of libref in compress abstraction layer

the complexity come from a mix and match of those modules that
have/don't have .is_init/.init functions, and makes handling
more delicate.

This commit ensures that:
- library is loaded only once (makes load_lib mandatory for every modules)
- use knet_h->compress_int_data to track per handle init.
  modules providing .is_init,.init don't need set anything there
  unless they need to.
- modules don't need to track libref anymore but they must
  provide a unload_lib/load_lib.
- library are closed only once libref is 0.

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

[crypto] move libnss to a dlopen model (pass 1)

unload and exit will coredump for now

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

[dlopen] implement dlopen rate limit to avoid RX DoS

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>

[compress] fix internal ref count for library unload

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

[compress] switch to new load_lib / unload_lib API

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

[compress] fix error message to be more explicit

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

[dlopen] start to cleanup the compress module API

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

[build] consistent error message

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

[build] move hardcoding of linking libraries in one place and enforce better check

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

[compress] move bzip2 to dlopen model

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

[compress] cleanup dlerror before dlopen for lzma

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

[compress] cosmetic cleanup... (part 2)

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

[compress] cosmetic cleanup

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

[compress] move lzma to dlopen model

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

[compress] move lz4 to dlopen model

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

[compress] make sure lib handles are clean after dlclose

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

[compress] move zlib to dlopen model

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

[compress] fix library init to work for multiple knet_handle

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