[PATCH] nfsd4: callback create rpc client returns
authorNeilBrown <neilb@cse.unsw.edu.au>
Sat, 16 Apr 2005 22:26:38 +0000 (15:26 -0700)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Sat, 16 Apr 2005 22:26:38 +0000 (15:26 -0700)
rpc_create_clnt and friends return errors, not NULL, on failure.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Neil Brown <neilb@cse.unsw.edu.au>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
fs/nfsd/nfs4callback.c

index c70de9c2af74658ab40cd5b180a5a1f201b112bc..1a55dfcb74bcb62b5bfb10fae788a3b56afc4f75 100644 (file)
@@ -405,7 +405,8 @@ nfsd4_probe_callback(struct nfs4_client *clp)
        timeparms.to_exponential = 1;
 
        /* Create RPC transport */
-       if (!(xprt = xprt_create_proto(IPPROTO_TCP, &addr, &timeparms))) {
+       xprt = xprt_create_proto(IPPROTO_TCP, &addr, &timeparms);
+       if (IS_ERR(xprt)) {
                dprintk("NFSD: couldn't create callback transport!\n");
                goto out_err;
        }
@@ -426,7 +427,8 @@ nfsd4_probe_callback(struct nfs4_client *clp)
         * XXX AUTH_UNIX only - need AUTH_GSS....
         */
        sprintf(hostname, "%u.%u.%u.%u", NIPQUAD(addr.sin_addr.s_addr));
-       if (!(clnt = rpc_create_client(xprt, hostname, program, 1, RPC_AUTH_UNIX))) {
+       clnt = rpc_create_client(xprt, hostname, program, 1, RPC_AUTH_UNIX);
+       if (IS_ERR(clnt)) {
                dprintk("NFSD: couldn't create callback client\n");
                goto out_xprt;
        }