git: 1512579adcf3 - main - nfscl: Make coverity happy
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Mon, 27 Mar 2023 23:54:39 UTC
The branch main has been updated by rmacklem:
URL: https://cgit.FreeBSD.org/src/commit/?id=1512579adcf3a2dee3f13efd702204c1390ccea7
commit 1512579adcf3a2dee3f13efd702204c1390ccea7
Author: Rick Macklem <rmacklem@FreeBSD.org>
AuthorDate: 2023-03-27 23:53:30 +0000
Commit: Rick Macklem <rmacklem@FreeBSD.org>
CommitDate: 2023-03-27 23:53:30 +0000
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.
Reported by: emaste
MFC after: 3 months
---
sys/fs/nfsclient/nfs_clrpcops.c | 2 +-
sys/fs/nfsclient/nfs_clvnops.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sys/fs/nfsclient/nfs_clrpcops.c b/sys/fs/nfsclient/nfs_clrpcops.c
index 559bf8646625..9a8e7436f620 100644
--- a/sys/fs/nfsclient/nfs_clrpcops.c
+++ b/sys/fs/nfsclient/nfs_clrpcops.c
@@ -9245,7 +9245,7 @@ nfscl_statfs(struct vnode *vp, struct ucred *cred, NFSPROC_T *td)
error = nfsrpc_statfs(vp, &sb, &fs, &lease, cred, td, &nfsva,
&attrflag);
if (attrflag != 0)
- nfscl_loadattrcache(&vp, &nfsva, NULL, 0, 1);
+ (void) nfscl_loadattrcache(&vp, &nfsva, NULL, 0, 1);
if (error == 0) {
NFSLOCKCLSTATE();
if (nmp->nm_clp != NULL)
diff --git a/sys/fs/nfsclient/nfs_clvnops.c b/sys/fs/nfsclient/nfs_clvnops.c
index 39409852e86f..ea493e668dc2 100644
--- a/sys/fs/nfsclient/nfs_clvnops.c
+++ b/sys/fs/nfsclient/nfs_clvnops.c
@@ -4542,7 +4542,7 @@ nfs_pathconf(struct vop_pathconf_args *ap)
NFSV4CONTENT_HOLE, td->td_ucred, &nfsva,
&attrflag);
if (attrflag != 0)
- nfscl_loadattrcache(&vp, &nfsva,
+ (void) nfscl_loadattrcache(&vp, &nfsva,
NULL, 0, 1);
mtx_lock(&nmp->nm_mtx);
if (error == NFSERR_NOTSUPP)