On 06/25/2012 04:43 PM, Chuck Lever wrote: > Hi- > > nfs4_create_sec_client() and nfs4_lookup_root_sec() both invoke rpcauth_create(). They check to see that it returned something that wasn't NULL. > > As far as I can tell, rpcauth_create() reports an error by returning something that IS_ERR. It doesn't return NULL. nfs_init_server_rpcclient() appears to be correct. > > The fix looks simple, but since I don't have any unit tests for this area of code, I was hoping you could take it from here. > Sure, I'll take a look at it. You're right, checking for an error code instead of NULL sounds like an easy fix. - Bryan -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html