Hi,
Coverity found an unused variable in the krb5_child code. After reading
the code again after a week it was written, I think we should do some
changes for better readability.
Mostly, fail on error from get_tgt_times() and change the flow so the
main branch (ie topmost indent) will be the success branch. It makes
sense to shortcut on success the first time get_tgt_times() is called to
check existing FAST ccache, but I think the second time, we expect
success, otherwise the forked child failed.