rpc.statd: Clean up: replace "if (!(foo = rtnl))".

Static code checkers flag this kind of thing because it's easy to
confuse with "if (!(foo == rtnl))".  In one of these cases, the
combination of evaluation and assignment isn't even necessary.

While we are in the neighborhood, remove an extra argument to note() that is
not called for in the passed-in format string.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Steve Dickson <steved@redhat.com>

diff --git a/utils/statd/callback.c b/utils/statd/callback.c
index d5d036e9fc28357b3ef703f3da3005f7cee0ad06..505fdb3c742112d09ea25230ee5588c4ff6e9915 100644 (file)
--- a/utils/statd/callback.c
+++ b/utils/statd/callback.c
@@ -36,11 +36,9 @@ sm_notify_1_svc(struct stat_chge *argp, struct svc_req *rqstp)
                                argp->mon_name, argp->state);
 
        /* quick check - don't bother if we're not monitoring anyone */
-       /* LH - this was != MULL, meaning that if anyone _was_ in our RTNL,
-        * we'd never pass this point. */
-       if (!(lp = rtnl)) {
+       if (rtnl == NULL) {
                note(N_WARNING, "SM_NOTIFY from %s while not monitoring any hosts.",
-                               argp->mon_name, argp->state);
+                               argp->mon_name);
                return ((void *) &result);
        }
 

diff --git a/utils/statd/monitor.c b/utils/statd/monitor.c
index d300338503d92a1668ff32f7de3bc9214461794f..a6a1d9c225a4443de7a8a21904e6a4cf75748949 100644 (file)
--- a/utils/statd/monitor.c
+++ b/utils/statd/monitor.c
@@ -326,12 +326,13 @@ sm_unmon_1_svc(struct mon_id *argp, struct svc_req *rqstp)
 
 
        /* Check if we're monitoring anyone. */
-       if (!(clnt = rtnl)) {
+       if (rtnl == NULL) {
                note(N_WARNING,
                        "Received SM_UNMON request from %s for %s while not "
                        "monitoring any hosts.", my_name, argp->mon_name);
                return (&result);
        }
+       clnt = rtnl;
 
        /*
         * OK, we are.  Now look for appropriate entry in run-time list.
@@ -379,11 +380,12 @@ sm_unmon_all_1_svc(struct my_id *argp, struct svc_req *rqstp)
 
        result.state = MY_STATE;
 
-       if (!(clnt = rtnl)) {
+       if (rtnl == NULL) {
                note(N_WARNING, "Received SM_UNMON_ALL request from %s "
                        "while not monitoring any hosts", my_name);
                return (&result);
        }
+       clnt = rtnl;
 
        while ((clnt = nlist_gethost(clnt, my_name, 1))) {
                if (NL_MY_PROC(clnt) == argp->my_proc &&