tcp_wrapper: Clean up logit()
authorChuck Lever <chuck.lever@oracle.com>
Sun, 17 Jan 2010 21:48:56 +0000 (16:48 -0500)
committerSteve Dickson <steved@redhat.com>
Sun, 17 Jan 2010 21:48:56 +0000 (16:48 -0500)
commite94001c0bb9ee7847a2ef7a3b436acd74acb9fd6
tree091a507a3d6583c7e775df60f5069eb03ef48e03
parent4bc0dbaed1ba2f58beea2cdb0595b461e0c88b8e
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>
support/include/tcpwrapper.h
support/misc/tcpwrapper.c
utils/mountd/mount_dispatch.c
utils/statd/statd.c