Fix a spurious NULL check on entry_lookup

Also rearranged the code to make it more readable. glist_entry returned
pointer can't be null, so fixed that as well. Coverity CID# 30382

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Reviewed-by: Frank S. Filz <ffilzlnx@mindspring.com>
nfs4_utf8string2dynamic() returns nfsstat4.

nfs4_utf8string2dynamic does NOT return cache_inode_status!
Coverity CID 30329 and 30330

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Reviewed-by: Frank S. Filz <ffilzlnx@mindspring.com>
Removed dead code in get_delegation()

No functional change! Coverity CID# 30237

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Reviewed-by: Frank S. Filz <ffilzlnx@mindspring.com>