diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2010-01-17 16:48:56 -0500 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2010-01-17 16:48:56 -0500 |
| commit | e94001c0bb9ee7847a2ef7a3b436acd74acb9fd6 (patch) | |
| tree | 091a507a3d6583c7e775df60f5069eb03ef48e03 /utils/statd/statd.c | |
| parent | 4bc0dbaed1ba2f58beea2cdb0595b461e0c88b8e (diff) | |
| download | nfs-utils-e94001c0bb9ee7847a2ef7a3b436acd74acb9fd6.tar.gz nfs-utils-e94001c0bb9ee7847a2ef7a3b436acd74acb9fd6.tar.xz nfs-utils-e94001c0bb9ee7847a2ef7a3b436acd74acb9fd6.zip | |
tcp_wrapper: Clean up logit()
Eliminate these compiler warnings:
tcpwrapper.c: In function logit
tcpwrapper.c:225: warning: unused parameter procnum
tcpwrapper.c:225: warning: unused parameter prognum
Actually, @procnum is not used anywhere in our tcpwrapper.c, so
let's just get rid of it.
Since there is only one logit() call site in tcpwrapper.c, the macro
wrapper just adds needless clutter. Let's get rid of that too.
Finally, both mountd and statd now use xlog(), which adds an
appropriate program name prefix to every message. Replace the
open-coded syslog(2) call with an xlog() call in order to
consistently identify the RPC service reporting the intrusion.
Since logit() no longer references "deny_severity" and no nfs-utils
caller sets either allow_severity or deny_severity, we remove them.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'utils/statd/statd.c')
| -rw-r--r-- | utils/statd/statd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/utils/statd/statd.c b/utils/statd/statd.c index 7be6454..fa3c6d5 100644 --- a/utils/statd/statd.c +++ b/utils/statd/statd.c @@ -79,7 +79,7 @@ sm_prog_1_wrapper (struct svc_req *rqstp, register SVCXPRT *transp) /* remote host authorization check */ if (sin->sin_family == AF_INET && - !check_default("statd", sin, rqstp->rq_proc, SM_PROG)) { + !check_default("statd", sin, SM_PROG)) { svcerr_auth (transp, AUTH_FAILED); return; } |