Remove unecessary retval test in vfs_get_root_handle (fixes CID24704)

Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>
call freeaddrinfo in add_client (fixes CID24706)

Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>
don't overwrite b_left in set_const_log_str (fixes CID24711 and CID24716)

Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>
Return CACHE_INODE_BAD_TYPE as error in cache_inode_setattr (fixes CID24721)

Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>
Simplify getting clientid_owner for LAYOUT ops (fixes CID24720)

The LAYOUT ops used to call get_clientid_owner to get the clientid owner
for the clientid associated with the current session. Since the ops MUST
be within a session, and thus the session and it's associated clientid
MUST be valid, we don't need to lookup, we can just look into the
associated clientid for it's owner.

This resolves any reference counting issues, and happens to clear a
Coverity bug with an ignored return result.

Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com>