New name : gck_rpc_parse_host_port().

Resolve host/service to listen on using getaddrinfo().

I am a bit uncertain about this change, since both NULL and non-NULL
will make test cases fail within the SoftHSM test suite for example
(some SoftHSM C_ functions want the pointer to be NULL when there
are no attributes, and some allow (expect) it to be possible to pass a
non-NULL pointer with a zero count).

It seems that making it a NULL pointer when the count is 0 is the most
sensible thing though. How could the C_ function use the data pointed
to by the pointer, when the count says there is no data there? The
result would really be undefined.

A number of PKCS#11 functions take a pointer to a buffer size as
argument. To be a transparent proxy of PKCS#11 calls, it is necessary to
support invoking these functions with a NULL pointer. pkcs11-proxy used
to send the buffer size as an integer and create a pointer to the integer
on the server side, but this is different to the backend PKCS#11 module
in some cases.

E.g. the C_Encrypt call is specified to have side effectes (finalizing)
when called with a NULL encrypted data length. The softhsm test suite
exposed that these side effects never occured because the NULL data length
pointer was conveyed as a valid pointer to the integer zero.

Since an additional uint8_t was added to "byte buffers", this is an
backwards incompatible change. As such, the version number in the protocol
greeting was increased (GCK_RPC_HANDSHAKE).

Passing size_t to _get_uint32 might leave garbage in top 32 bits of size_t
on 64 bits platforms. While initializing the size_t to 0 would probably
work, using a temporary uint32_t and casting that is more explicit IMO.

Sending side gck_rpc_message_write_byte_array() adds length data
when valid == 0, so receiving code (this) need to parse that obviosly.

Necessary to close all ongoing sessions.

Best for the proxy to be as transparent as possible. This fixes a test
case with C_GenerateRandom of 0 bytes, found in pkcs11-tool --test.

Commit 09669a08 introduced ds->cs.sock along side ds->socket.
The code that closed the socket only set ds->cs.sock to -1, so
the cleanup code (that looked at ds->socket) never realized it
should free the message buffers.

This commit removes ds->socket completely, and makes the
cleanup code look if ds->cs.sock is -1. Memory usage now stays
constant when looping 'pkcs11-tool --test --slot 0' at least.

This is necessary to not get a SIGPIPE on write failures (easy DoS
of the pkcs11-daemon - client just need to close the connection at
the wrong moment).

- use syslog instead of stderr unless DEBUG_OUTPUT is 1

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

Signed-off-by: Corentin Chary <cchary@commonit.com>

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

Will be needed to fix the WaitForSlotEvent issue

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

When the port is not defined, the system will autogically pick
a free tcp port.

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>

Signed-off-by: Corentin Chary <cchary@commonit.com>

Signed-off-by: Corentin Chary <corentin.chary@gmail.com>