2009-05-21 NetBSD src/usr.sbin/inetd
この 変更 の
@@ -649,6 +649,7 @@
char buf[NI_MAXSERV];
struct servtab *s;
#ifdef LIBWRAP
+ char abuf[BUFSIZ];
struct request_info req;
int denied;
char *service = NULL; /* XXX gcc */
@@ -672,17 +673,19 @@
ntohs(sep->se_ctrladdr_in.sin_port));
}
service = buf;
+ sockaddr_snprintf(abuf, sizeof(abuf), "%a",
+ &sep->se_ctrladdr);
}
if (denied) {
syslog(deny_severity,
- "refused connection from %.500s, service %s (%s)",
- eval_client(&req), service, sep->se_proto);
+ "refused connection from %.500s(%s), service %s (%s)",
+ eval_client(&req), abuf, service, sep->se_proto);
goto reject;
}
if (lflag) {
syslog(allow_severity,
- "connection from %.500s, service %s (%s)",
- eval_client(&req), service, sep->se_proto);
+ "connection from %.500s(%s), service %s (%s)",
+ eval_client(&req), abuf, service, sep->se_proto);
}
}
#endif /* LIBWRAP */
…の本来の意図は、↓のように
@@ -672,7 +672,7 @@
}
service = buf;
sockaddr_snprintf(abuf, sizeof(abuf), "%a",
- &sep->se_ctrladdr);
+ req.client->sin);
}
if (denied) {
syslog(deny_severity,
こうしたかったのではなかろうか。
bindしてるアドレスを表示したいんじゃなくて、クライアントのアドレスを表示したいんだよなぁ。どう考えても。
追記:
ちゃんと元のPRまで追って見たらやっぱりその意図だった。http://mail-index.netbsd.org/netbsd-bugs/2009/01/07/msg008213.html というか何で誰も気(r
誰か(
追記(2):
がんばってメールした。
EOF