cli: fix incorrect BuxtonKey memory free

Not every return path in cli_set_value frees key,
also just free call is not enough for BuxtonKey, due it's composite
structure. Callback free_buxton_key which are registered for
_cleanup_buxton_key_ releases complete BuxtonKey structure.

cli_get_value didn't release BuxtonKey's memory at all.

Signed-off-by: Alexey Perevalov <a.perevalov@samsung.com>
daemon: fix memmove in del_pollfd

Size of tail part was based on size of pointer type, not bool.

Signed-off-by: Alexey Perevalov <a.perevalov@samsung.com>
lbuxton: Add check for buxton socket path name length

Buxton's socket name could be obtained from environmental variables and
from config, end user could specify really long name. To prevent memory
corruption in case of missing /0 character and to prevent logic error,
it's better to add check for length of buxton's socket name in place of
its usage.

Signed-off-by: Alexey Perevalov <a.perevalov@samsung.com>