Fix uninitialized variable bug in KEYRING ccache

Commit 5f4a4d7d357fedac5feadc65c09ecf487ff98db8 removed the only
unconditional assignment of ret in get_time_offsets, causing the
function to return an uninitialized value if nothing goes wrong.
Initialize ret at declaration time to fix this.
Make tcl_kadm5.c work with Tcl 8.6

Directly accessing the result field of Tcl_Interp has been deprecated
for a long time, requires a special define in Tcl 8.6, and will be
impossible in Tcl 9.  Use Tcl_SetResult instead.  The new error
messages are less helpful than the old ones, but this is just support
infrastructure for old tests, so it isn't important.

ticket: 7924
Correct documentation of 'x' in kadm5.acl

It really is all privileges, including setting keys and
disallowing the propagation of the principal database.
get_time_offsets() should return success at the end.

This fixes the compile error that rc might be used uninitialized!

Signed-off-by: Andreas Schneider <asn@samba.org>
Add a famlily independent bindresvport_sa().

This functions allows you to pass IPv4 and IPv6 addresses. It will
determine the family by checking the socket with getsockname().

Signed-off-by: Andreas Schneider <asn@samba.org>
Just call bindresvport_sa() in bindresvport().

This avoids code duplication.

Signed-off-by: Andreas Schneider <asn@samba.org>
Use the new binresvport_sa() function.

This function can deal with IPv4 and IPv6 sockets. It allows us to
correctly check the return values and react on errors.

Signed-off-by: Andreas Schneider <asn@samba.org>