diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2019-01-04 01:47:03 -0500 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2019-02-06 15:37:14 -0500 |
commit | 1c3da4452d185e4be663e76a1b9842184d8f9c4c (patch) | |
tree | 1736f3e7f164c68fdf7d9cd01c2d6340da4994a6 | |
parent | e248aa7be86e8179f20ac0931774ecd746f3f5bf (diff) |
nfsd: fix an IS_ERR() vs NULL check
The get_backchannel_cred() used to return error pointers on error but
now it returns NULL pointers.
Fixes: 97f68c6b02e0 ("SUNRPC: add 'struct cred *' to auth_cred and rpc_cre")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
-rw-r--r-- | fs/nfsd/nfs4callback.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/nfsd/nfs4callback.c b/fs/nfsd/nfs4callback.c index c74e4538d0eb..9b38dab1c21b 100644 --- a/fs/nfsd/nfs4callback.c +++ b/fs/nfsd/nfs4callback.c | |||
@@ -913,9 +913,9 @@ static int setup_callback_client(struct nfs4_client *clp, struct nfs4_cb_conn *c | |||
913 | return PTR_ERR(client); | 913 | return PTR_ERR(client); |
914 | } | 914 | } |
915 | cred = get_backchannel_cred(clp, client, ses); | 915 | cred = get_backchannel_cred(clp, client, ses); |
916 | if (IS_ERR(cred)) { | 916 | if (!cred) { |
917 | rpc_shutdown_client(client); | 917 | rpc_shutdown_client(client); |
918 | return PTR_ERR(cred); | 918 | return -ENOMEM; |
919 | } | 919 | } |
920 | clp->cl_cb_client = client; | 920 | clp->cl_cb_client = client; |
921 | clp->cl_cb_cred = cred; | 921 | clp->cl_cb_cred = cred; |