Remove assert that breaks things

The assertion was misplaced, because it should not be fatal in some
circumstances to fail to load the conf file.  For example, in the client
library, having the socket specified either via env or command line
should be the only requirement.  This removes the assert, which in
addition to being wrong also made tests failed with asserts are
enabled.  The author regrets the error.

Signed-off-by: Michael Leibowitz <michael.leibowitz@intel.com>
db/gdbm: Set a dummy label for wire protocol transit

This fixes an assertion that should not be reached when list_keys
does not have a label set for each BuxtonData.

Signed-off-by: Michael Ikey Doherty <michael.i.doherty@intel.com>
db/gdbm: Ensure transit length includes null terminator

Signed-off-by: Michael Ikey Doherty <michael.i.doherty@intel.com>
Link gdbm.so to libbuxton

As counter intuitive as this may seem, gdbm currently needs access
to libbuxton specific functionality, namely buxton_get_name to split
a key's name from the key:group identifier.

In the near future the present make system will be modified to remove
cross-linking between existing components, and shared accessors such
as buxton_get_name will be split into a static library.

Signed-off-by: Michael Ikey Doherty <michael.i.doherty@intel.com>
configurator: Use const char* for internal members

The strings returned by the public configurator methods are owned
by configurator itself, and should not be modified or free'd by
any external API.

This commit makes this requirement clearer by forcing the use of
const char* for returned strings,

Signed-off-by: Michael Ikey Doherty <michael.i.doherty@intel.com>