serialize: remove useless check

since len is unsigned, the test can never be true

Signed-off-by: Michael Leibowitz <michael.leibowitz@intel.com>
protocol: insurance against null ptr deref

Flagged by the static analyzer

Signed-off-by: Michael Leibowitz <michael.leibowitz@intel.com>
gdbm: fix use after free in error

Signed-off-by: Michael Leibowitz <michael.leibowitz@intel.com>
client: leak fixes

Also add static function to avoid passing NULL to printf %s.  Although
glibc handles it properly, the static analyzer flags it and it is
undefined behavior per standard.

Signed-off-by: Michael Leibowitz <michael.leibowitz@intel.com>
lbuxton: fix a leak on error

flagged by static analyzer

Signed-off-by: Michael Leibowitz <michael.leibowitz@intel.com>
daemon: free out_list array in all cases

move free to later to make sure it is freed on error case

Signed-off-by: Michael Leibowitz <michael.leibowitz@intel.com>