nfscl: Make coverity happy
Coverity does not like code that checks a function's return value sometimes. Add "(void)" in front of the function when the return value does not matter to try and make it happy. A recent commit deleted "(void)"s in front of nfsm_fhtom(). This commit puts them back in. Reported by: emaste MFC after: 3 months
This commit is contained in:
parent
35105900c6
commit
695d87bae1
@ -437,7 +437,7 @@ nfscl_reqstart(struct nfsrv_descript *nd, int procnum, struct nfsmount *nmp,
|
||||
if (nfsv4_opflag[nfsv4_opmap[procnum].op].needscfh > 0) {
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
nfsm_fhtom(nmp, nd, nfhp, fhlen, 0);
|
||||
(void)nfsm_fhtom(nmp, nd, nfhp, fhlen, 0);
|
||||
if (nfsv4_opflag[nfsv4_opmap[procnum].op].needscfh
|
||||
== 2 && procnum != NFSPROC_WRITEDS &&
|
||||
procnum != NFSPROC_COMMITDS) {
|
||||
@ -468,7 +468,7 @@ nfscl_reqstart(struct nfsrv_descript *nd, int procnum, struct nfsmount *nmp,
|
||||
*tl = txdr_unsigned(nfsv4_opmap[procnum].op);
|
||||
}
|
||||
} else {
|
||||
nfsm_fhtom(NULL, nd, nfhp, fhlen, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, nfhp, fhlen, 0);
|
||||
}
|
||||
if (procnum < NFSV42_NPROCS)
|
||||
NFSINCRGLOBAL(nfsstatsv1.rpccnt[procnum]);
|
||||
|
@ -2585,7 +2585,7 @@ nfsrpc_createv4(vnode_t dvp, char *name, int namelen, struct vattr *vap,
|
||||
/* Get the directory's post-op attributes. */
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh, np->n_fhp->nfh_len, 0);
|
||||
(void)nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh, np->n_fhp->nfh_len, 0);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_GETATTR);
|
||||
(void) nfsrv_putattrbit(nd, &attrbits);
|
||||
@ -2785,7 +2785,7 @@ nfsrpc_remove(vnode_t dvp, char *name, int namelen, vnode_t vp,
|
||||
*tl++ = dstateid.other[2];
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
np = VTONFS(dvp);
|
||||
nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh,
|
||||
(void)nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh,
|
||||
np->n_fhp->nfh_len, 0);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_REMOVE);
|
||||
@ -2873,7 +2873,7 @@ nfsrpc_rename(vnode_t fdvp, vnode_t fvp, char *fnameptr, int fnamelen,
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
np = VTONFS(tvp);
|
||||
nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh,
|
||||
(void)nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh,
|
||||
np->n_fhp->nfh_len, 0);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_DELEGRETURN);
|
||||
@ -2893,7 +2893,7 @@ nfsrpc_rename(vnode_t fdvp, vnode_t fvp, char *fnameptr, int fnamelen,
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
np = VTONFS(fdvp);
|
||||
nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh,
|
||||
(void)nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh,
|
||||
np->n_fhp->nfh_len, 0);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_SAVEFH);
|
||||
@ -2910,7 +2910,7 @@ nfsrpc_rename(vnode_t fdvp, vnode_t fvp, char *fnameptr, int fnamelen,
|
||||
(void) nfsrv_putattrbit(nd, &attrbits);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
nfsm_fhtom(nmp, nd, VTONFS(tdvp)->n_fhp->nfh_fh,
|
||||
(void)nfsm_fhtom(nmp, nd, VTONFS(tdvp)->n_fhp->nfh_fh,
|
||||
VTONFS(tdvp)->n_fhp->nfh_len, 0);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_GETATTR);
|
||||
@ -2921,7 +2921,7 @@ nfsrpc_rename(vnode_t fdvp, vnode_t fvp, char *fnameptr, int fnamelen,
|
||||
}
|
||||
(void) nfsm_strtom(nd, fnameptr, fnamelen);
|
||||
if (!(nd->nd_flag & ND_NFSV4))
|
||||
nfsm_fhtom(nmp, nd, VTONFS(tdvp)->n_fhp->nfh_fh,
|
||||
(void)nfsm_fhtom(nmp, nd, VTONFS(tdvp)->n_fhp->nfh_fh,
|
||||
VTONFS(tdvp)->n_fhp->nfh_len, 0);
|
||||
(void) nfsm_strtom(nd, tnameptr, tnamelen);
|
||||
error = nfscl_request(nd, fdvp, p, cred);
|
||||
@ -3006,7 +3006,7 @@ nfsrpc_link(vnode_t dvp, vnode_t vp, char *name, int namelen,
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
}
|
||||
nfsm_fhtom(VFSTONFS(dvp->v_mount), nd, VTONFS(dvp)->n_fhp->nfh_fh,
|
||||
(void)nfsm_fhtom(VFSTONFS(dvp->v_mount), nd, VTONFS(dvp)->n_fhp->nfh_fh,
|
||||
VTONFS(dvp)->n_fhp->nfh_len, 0);
|
||||
if (nd->nd_flag & ND_NFSV4) {
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
@ -3146,7 +3146,7 @@ nfsrpc_mkdir(vnode_t dvp, char *name, int namelen, struct vattr *vap,
|
||||
(void) nfsrv_putattrbit(nd, &attrbits);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
nfsm_fhtom(nmp, nd, fhp->nfh_fh, fhp->nfh_len, 0);
|
||||
(void)nfsm_fhtom(nmp, nd, fhp->nfh_fh, fhp->nfh_len, 0);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_GETATTR);
|
||||
(void) nfsrv_putattrbit(nd, &attrbits);
|
||||
@ -8230,7 +8230,7 @@ nfsrpc_createlayout(vnode_t dvp, char *name, int namelen, struct vattr *vap,
|
||||
/* Get the directory's post-op attributes. */
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh, np->n_fhp->nfh_len, 0);
|
||||
(void)nfsm_fhtom(nmp, nd, np->n_fhp->nfh_fh, np->n_fhp->nfh_len, 0);
|
||||
NFSM_BUILD(tl, u_int32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_GETATTR);
|
||||
nfsrv_putattrbit(nd, &attrbits);
|
||||
@ -8635,7 +8635,7 @@ nfsrpc_copyrpc(vnode_t invp, off_t inoff, vnode_t outvp, off_t outoff,
|
||||
nfsrv_putattrbit(nd, &attrbits);
|
||||
NFSM_BUILD(tl, uint32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_PUTFH);
|
||||
nfsm_fhtom(nmp, nd, VTONFS(outvp)->n_fhp->nfh_fh,
|
||||
(void)nfsm_fhtom(nmp, nd, VTONFS(outvp)->n_fhp->nfh_fh,
|
||||
VTONFS(outvp)->n_fhp->nfh_len, 0);
|
||||
NFSM_BUILD(tl, uint32_t *, NFSX_UNSIGNED);
|
||||
*tl = txdr_unsigned(NFSV4OP_COPY);
|
||||
|
@ -672,10 +672,10 @@ nfsrvd_lookup(struct nfsrv_descript *nd, __unused int isdgram,
|
||||
goto out;
|
||||
}
|
||||
if (nd->nd_flag & ND_NFSV2) {
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 0);
|
||||
nfsrv_fillattr(nd, &nva);
|
||||
} else if (nd->nd_flag & ND_NFSV3) {
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 0);
|
||||
nfsrv_postopattr(nd, 0, &nva);
|
||||
nfsrv_postopattr(nd, dattr_ret, &dattr);
|
||||
}
|
||||
@ -1282,7 +1282,7 @@ nfsrvd_create(struct nfsrv_descript *nd, __unused int isdgram,
|
||||
}
|
||||
if (nd->nd_flag & ND_NFSV2) {
|
||||
if (!nd->nd_repstat) {
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)&fh, 0, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)&fh, 0, 0);
|
||||
nfsrv_fillattr(nd, &nva);
|
||||
}
|
||||
} else {
|
||||
@ -1292,7 +1292,7 @@ nfsrvd_create(struct nfsrv_descript *nd, __unused int isdgram,
|
||||
diraft_ret = nfsvno_getattr(dirp, &diraft, nd, p, 0, NULL);
|
||||
vrele(dirp);
|
||||
if (!nd->nd_repstat) {
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)&fh, 0, 1);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)&fh, 0, 1);
|
||||
nfsrv_postopattr(nd, 0, &nva);
|
||||
}
|
||||
nfsrv_wcc(nd, dirfor_ret, &dirfor, diraft_ret, &diraft);
|
||||
@ -1492,7 +1492,7 @@ nfsrvd_mknod(struct nfsrv_descript *nd, __unused int isdgram,
|
||||
vrele(dirp);
|
||||
if (!nd->nd_repstat) {
|
||||
if (nd->nd_flag & ND_NFSV3) {
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 1);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 1);
|
||||
nfsrv_postopattr(nd, 0, &nva);
|
||||
} else {
|
||||
NFSM_BUILD(tl, u_int32_t *, 5 * NFSX_UNSIGNED);
|
||||
@ -1946,7 +1946,7 @@ nfsrvd_symlink(struct nfsrv_descript *nd, __unused int isdgram,
|
||||
|
||||
if (nd->nd_flag & ND_NFSV3) {
|
||||
if (!nd->nd_repstat) {
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 1);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 1);
|
||||
nfsrv_postopattr(nd, 0, &nva);
|
||||
}
|
||||
nfsrv_wcc(nd, dirfor_ret, &dirfor, diraft_ret, &diraft);
|
||||
@ -2070,12 +2070,12 @@ nfsrvd_mkdir(struct nfsrv_descript *nd, __unused int isdgram,
|
||||
|
||||
if (nd->nd_flag & ND_NFSV3) {
|
||||
if (!nd->nd_repstat) {
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 1);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 1);
|
||||
nfsrv_postopattr(nd, 0, &nva);
|
||||
}
|
||||
nfsrv_wcc(nd, dirfor_ret, &dirfor, diraft_ret, &diraft);
|
||||
} else if (!nd->nd_repstat) {
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, 0, 0);
|
||||
nfsrv_fillattr(nd, &nva);
|
||||
}
|
||||
|
||||
@ -3462,7 +3462,7 @@ nfsrvd_getfh(struct nfsrv_descript *nd, __unused int isdgram,
|
||||
nd->nd_repstat = nfsvno_getfh(vp, &fh, p);
|
||||
vput(vp);
|
||||
if (!nd->nd_repstat)
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)&fh, 0, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)&fh, 0, 0);
|
||||
NFSEXITCODE2(0, nd);
|
||||
return (0);
|
||||
}
|
||||
|
@ -4529,7 +4529,7 @@ nfsrv_docallback(struct nfsclient *clp, int procnum, nfsv4stateid_t *stateidp,
|
||||
m_freem(nd->nd_mreq);
|
||||
goto errout;
|
||||
}
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, NFSX_MYFH, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, NFSX_MYFH, 0);
|
||||
(void)nfsrv_putattrbit(nd, attrbitp);
|
||||
} else if (procnum == NFSV4OP_CBRECALL) {
|
||||
nd->nd_procnum = NFSV4PROC_CBCOMPOUND;
|
||||
@ -4548,7 +4548,7 @@ nfsrv_docallback(struct nfsclient *clp, int procnum, nfsv4stateid_t *stateidp,
|
||||
*tl = newnfs_true;
|
||||
else
|
||||
*tl = newnfs_false;
|
||||
nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, NFSX_MYFH, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, (u_int8_t *)fhp, NFSX_MYFH, 0);
|
||||
} else if (procnum == NFSV4OP_CBLAYOUTRECALL) {
|
||||
NFSD_DEBUG(4, "docallback layout recall\n");
|
||||
nd->nd_procnum = NFSV4PROC_CBCOMPOUND;
|
||||
@ -4567,7 +4567,7 @@ nfsrv_docallback(struct nfsclient *clp, int procnum, nfsv4stateid_t *stateidp,
|
||||
else
|
||||
*tl++ = newnfs_false;
|
||||
*tl = txdr_unsigned(NFSV4LAYOUTRET_FILE);
|
||||
nfsm_fhtom(NULL, nd, (uint8_t *)fhp, NFSX_MYFH, 0);
|
||||
(void)nfsm_fhtom(NULL, nd, (uint8_t *)fhp, NFSX_MYFH, 0);
|
||||
NFSM_BUILD(tl, u_int32_t *, 2 * NFSX_HYPER + NFSX_STATEID);
|
||||
tval = 0;
|
||||
txdr_hyper(tval, tl); tl += 2;
|
||||
|
Loading…
Reference in New Issue
Block a user