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

Since proto_write_attribute_array() happily writes arrays with len == 0,
we should probably parse them. Tounge in cheek.

Necessary to close all ongoing sessions.

This harmonizes with the softhsm test suite.

This matches the PKCS#11 specification better, and also matches
the test cases for softhsm.

Fix zero sized buffers

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.

Helps find resource leaks, if nothing else.

softhsm 1.3.0-1build1
Ubuntu 12.04.1

Avoid DoS

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

SetPIN was using incorrect PIN length.

Remove references to /opt/commonit environment.