Fix RPC calls: ATTRIBUTE buf not null but length 0

Let's add a support for cases when the buffer != NULL but the
length is 0. According to Oasis, buffer = NULL and length = 0
means a query of the length so the subsequent calls with a
buffer != NULL should fill then buffer when length is long enough.
If not, according to Oasis, one should get a CKR_BUFFER_TOO_SMALL.

See the previous commit for IN_ULONG_BUFFER(). This patch is
follow a similar design pattern.

Fix: issue #257

Fix RPC calls: BYTE buffer not null and length 0

Let's add a support for cases when the buffer != NULL but the
length is 0. According to Oasis, buffer = NULL and length = 0
means a query of the length so the subsequent calls with a
buffer != NULL should fill buffer when length is long enough.
If not, according to Oasis, one should get a CKR_BUFFER_TOO_SMALL.

This current fix is for IN_BYTE_BUFFER(), same
for IN_ATTRIBUTE_BUFFER().

See the previous commit for IN_ULONG_BUFFER(). This patch is
strictly using the same design pattern.

Fix: issue #257

Suggested-by: Daiki Ueno <dueno@redhat.com>

Fix C_GetSlotList() when length is 0

Let's add a support for cases when the buffer != NULL but the
length is 0. According to Oasis, buffer = NULL and length = 0
means a query of the length so the subsequent calls with a
buffer != NULL should fill buffer when length is long enough.
If not, according to Oasis, one should get a CKR_BUFFER_TOO_SMALL.

This current fix is for IN_ULONG_BUFFER(), same
should be applied for IN_BYTE_BUFFER() and for IN_ATTRIBUTE_BUFFER().

Include a test_no_slots()

Fix: issue #257

Suggested-by: Daiki Ueno <dueno@redhat.com>

test-proxy: Implement reproducer for bad prefix list matching

modules: Implement correct search in list

The current version of matching was failing, when the list contained
also a searched string with some suffix, for example, when we ran from
p11-kit and the p11-kit-proxy was first in the list and p11-kit later,
it was not matched, because the test did not find a separator after
the first match, decided that it does not match and did not try further.

example program p11-kit
example enable-in: p11-kit-proxy,p11-kit

autotools: Fix bash-completion installation

Install bash completion in the CI

Add simple bash completion for provided commands

configure: Fix typo to avoid errors during configure

common: add Russian PKCS#11 extensions to pkcs11x.h header

Add values defined to support Russian GOST cryptography to pkcs11x.h
header.

Signed-off-by: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>

autotools: Add more files from meson build in distribution

autotools: Add more files from meson build in distribution

build: Fix 'make distcheck'

Release 0.23.18

rpc: Allow empty CK_DATE value

Unlike other data types, CK_DATE value may be empty (and that is the
default).  Treat it as a valid value and serialize/deserialize
accordingly.

Reported by Vincent JARDIN in:
https://github.com/p11-glue/p11-kit/issues/244

build: Fix undefined behavior in left shift

Spotted by UBSan.

autotools: Use symbol versioning if possible

The meson build already using it for:
https://github.com/mesonbuild/meson/issues/3047

Suggested by Jan Alexander Steffens

build: Import ld-version-script.m4 from gnulib

build: Add meson files in autotools distribution

travis: Run ninja install as root

This is to avoid meson calling pkexec, which doesn't work inside docker.

meson: Add missing prefix to system path defines

meson: Install p11-kit-server units

meson: Build and install man pages

meson: Don't prefix p11_user_config

meson: PKCS#11 modules should only export C_GetFunctionList

meson: Install p11-kit-client module

meson: Fix dictionary syntax

Reported by Jan Alexander Steffens.

Release 0.23.17

uri: Supress cppcheck false-positive

uri: Check return value of insert_attribute

meson: Update project version

build: Fix typo in checking program_invocation_short_name decl

The decl should be in errno.h, not error.h.

common: Fix uClibc-ng compilation

program_invocation_short_name is const under uClibc-ng.

trust: do not allow daylight to invalidate date validation

Issue: 235
Signed-off-by: Alon Bar-Lev <alon.barlev@gmail.com>

build: Declare dependency chain between static libs in common

In common/ there are sub-libraries namely libp11-common.a,
libp11-library.a, libp11-tool.a, and libp11-test.a.

All the latter 3 libs use the symbols from libp11-common.a, it would
make sense to declare a dependency against it.

build: Make threads dependency more explicit

.travis.yml: Use meson for building

build: Add meson build support

This adds support for meson as an alternative build system.

trust: Fix mismatched return values

po: Remove en@{,bold}quot from LINGUAS

build: Don't hardcode module path

build: Move check_PROGRAMS into subdirectories

tests: Add tmpdir argument to p11_test_copy_setgid

To prevent BUILDDIR being embedded in the library.

tests: Fix memleaks in test-jks.c

tests: Skip tests calling getauxval(AT_SECURE) if binary is on /tmp

tests: Fix temp file permission before deleting

On mingw64/wine, unlink fails if the file has no write bit.

tests: Don't assume / is not writable on Windows in test-token.c

conf: Skip root UID check on Windows

virtual: Rename virtual-fixed.c to virtual-fixed-generated.h

Previously the generated .c file was included in another source file,
which is not supported in some build systems (e.g., meson).

common: Make issetugid check simpler

common: Fix vasprintf emulation

va_list must be saved when calling vsnprintf() in a loop.

rpc: On UNIX wait on condition variable instead of FD if header is for a different thread.

If rpc_socket_read() receives a header for a different thread, it tries to yield by
releasing the read mutex and waiting on the socket's read FD. On Linux systems, this has
been observed to cause a performance problem in cases where multiple threads are being
used. Threads expecting a different header can rapidly unlock and relock the read mutex,
as they resume when sock->read_code hasn't changed. This can result in contention on the
read mutex, which delays the thread that is expecting to consume the header.

This fix updates rpc_socket_read() on UNIX to wait on a condition variable instead of the
socket's read FD. The condition variable is signalled when sock->read_code changes. This
allows waiting threads to only resume once the header and payload have been consumed by
their target thread. This fix only targets UNIX platforms, as the Windows version that
p11-kit targets by default (Windows 2000) does not provide support for condition
variables.

Signed-off-by: Simon Haggett <simon.haggett@gmail.com>

tests: Avoid uninitialized value in test-proxy.c

build: Suppress compiler warning

Remove unused "global" variable.

doc: Add 'server' command in help

'server' is the last common command which is not in CLI help.
IMHO, adding this small documentation could help to promote usage of
pkcs11 forwarding.

Release 0.23.16

proxy: Support C_WaitForSlotEvent() if CKF_DONT_BLOCK is specified

While fully implementing C_WaitForSlotEvent() would require a separate
thread to monitor events, it is straightforward to implement the
function if the CKF_DONT_BLOCK flag is given.

Suggested by David Ward.

conf: Ignore user configuration if the program is running as root

Suggested by Bastien Nocera:
https://bugzilla.redhat.com/show_bug.cgi?id=1688583

proxy: Refresh slot list on every C_GetSlotList call

Previously, the proxy module calculated the slot list only once at the
C_Initialize() call.  That was causing a usability limitation when the
user attaches HSM after starting an application.

Suggested by David Ward.

modules: Fix index used in call to p11_dict_remove()

This fixes a call to p11_dict_remove() in managed_steal_sessions_inlock() to use
the correct index in the stolen array (i, rather than at). This avoids an
assert, which was encountered on a host serving a PKCS#11 module to a remote
Linux client.

Signed-off-by: Simon Haggett <simon.haggett@gmail.com>

Fix Win32 p11_dl_error crash

Caused by returning a buffer that wasn't allocated with malloc and
needed to be freed with LocalFree() instead. The fix is to strdup
msg_buf so what's returned can be free()d.

modules: check gl.modules before iterates on it when freeing

In some circumstances, as described in the BZ, can happen that
free_modules_when_no_refs_unlocked() is called multiple times
when the module destructor is invoked.
We should check gl.modules before iterates on it in the
free_modules_when_no_refs_unlocked() functions, to avoid
a SIGSEGV.

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1680963

trust: Ignore unreadable content in anchors

This amends eb503f3a1467f21a5ecc9ae84ae23b216afc102f.  Instead of
failing C_FindObjectsInit, treat any errors internally and accumulates
the successfully loaded certificates.

Reported by Andrej Kvasnica in:
https://bugzilla.redhat.com/show_bug.cgi?id=1675441

extract-jks: Prefer _p11_extract_jks_timestamp to SOURCE_DATE_EPOCH

Give _p11_extract_jks_timestamp precedence over SOURCE_DATE_EPOCH so
that the test results are not affected by the envvar settings.

Release 0.23.15

pem: Fix assert condition

If the PEM header is "-----BEGIN -----", *type should be an empty
string and the parser shouldn't fail.  Reported by Han Han in:
https://bugzilla.redhat.com/show_bug.cgi?id=1665172

test: Add test that exercises duplicated certs in JKS

trust: Fix alias generation in JKS extractor

When there is a duplicate, the JKS extractor previously assigned
somewhat obscure name "-<digit>" (not "<name>-<digit>").

trust: Continue parsing if the file cannot be read as persist format

A corrupted file that contains "[p11-kit-object-v1]" can be a valid
PEM certs file.  Continue with the next format if it cannot be read as
a persistent format.

trust: p11_token_load: Treat parse error as failure

Those conditions can happen when the trust file is corrupted, so it
makes more sense to treat them as a failure instead of programmer
error.

trust: Fail if trust anchors are not loaded from a file

If the trust path is a file, treat parse error as fatal and abort the
C_FindObjectsInit call.

trust: Propagate library verbosity to module through init_args

Previously, even when the -v option is used with the 'trust' command,
the messages from p11-kit-trust.so module were suppressed because the
verbosity setting is not propagated to the module.

build: Fix typo spotted by codespell

doc: Make log-calls match the rest of the document style

build: Simplify assertions

Let it leak memory when we assert, so not to confuse static analyzers
as if this is the normal case.

travis: Add cppcheck profile

travis: Allow profile override

build: Suppress cppcheck errors

tests: Ensure p11_proxy_module_cleanup is called

Reported and suggested in #197.

url: Prefer upper-case letters in hex characters when encoding

This makes it more compliant with RFC 3986, where the use of
upper-case letters is recommended (as "SHOULD").

Suggested by Sumit Bose.

trust/extract-jks.c: also honor SOURCE_DATE_EPOCH time

For reproducible builds, accept a define timestamp for the java
keystore.

See https://reproducible-builds.org/docs/source-date-epoch/

build: Require pkg.m4 >= 0.29 at bootstrap

virtual: Prefer fixed closures to libffi closures

On some circumstances (such as when loading p11-kit-proxy from httpd),
it is known that creation of libffi closure always fails, due to
SELinux policy.  Although this is harmless, it pollutes the journal
and gives wrong hints when troubleshooting.  This patch changes the
order of preference of libffi vs pre-compiled closures to avoid that.

trust: Check index->buckets is allocated on cleanup

rpc-server: Check calloc failure

trust: Set umask before calling mkstemp

proxy: Fix null dereference when reusing slots

rpc-server: p11_kit_remote_serve_tokens: Fix memleak

build: Check return value of p11_rpc_buffer_get_uint64

build: Check return value of p11_dict_set

build: Free memory before return{,_val}_if_* macros

build: Call va_end() always when leaving the function

debug: Work around cppcheck false-positives

https://trac.cppcheck.net/ticket/8794

common: use /proc only on Linux

Non-Linux systems do not have /proc, so do not attempt to open it and
eliminate an unnecessary access() syscall on those systems.

pkcs11: Don't redefine CKM_CAMELLIA_KEY_GEN

Also reorder the CKM_CAMELLIA_* definitions.

Release 0.23.14

virtual: Tighten error handling when fixed closures are exhausted

virtual: Don't be too loud about recoverable failure

trust: Factor out module initialization into separate file

This prevents double call to p11_library_init() in test-module.c, once
from the ELF constructor, and secondly from the test itself.

common: Factor out common initializer code into a header