svn commit: r325740 - head/sbin/dhclient
Mariusz Zaborski
oshogbo at FreeBSD.org
Sun Nov 12 08:42:45 UTC 2017
Author: oshogbo
Date: Sun Nov 12 08:42:43 2017
New Revision: 325740
URL: https://svnweb.freebsd.org/changeset/base/325740
Log:
Use syslog service in dhclient(8).
dhclient(8) is failing during boot to connect to the syslog service, because
syslog daemon is started after dhclient(8). This can be reproduced by stooping
syslog daemon and ktrace the dhclient or use kern.trap_enotcap sysctl and boot
the machine. Using the Casper syslog service fix the problem.
Reviewed by: bapt@
Differential Revision: https://reviews.freebsd.org/D12825
Modified:
head/sbin/dhclient/Makefile
head/sbin/dhclient/dhclient.c
head/sbin/dhclient/dhcpd.h
head/sbin/dhclient/dispatch.c
head/sbin/dhclient/errwarn.c
Modified: head/sbin/dhclient/Makefile
==============================================================================
--- head/sbin/dhclient/Makefile Sun Nov 12 08:34:25 2017 (r325739)
+++ head/sbin/dhclient/Makefile Sun Nov 12 08:42:43 2017 (r325740)
@@ -44,6 +44,12 @@ MAN= dhclient.8 dhclient.conf.5 dhclient.leases.5 dhcp
dhclient-script.8
LIBADD= util
+.if ${MK_CASPER} != "no" && !defined(RESCUE)
+LIBADD+= casper
+LIBADD+= cap_syslog
+CFLAGS+=-DWITH_CASPER
+.endif
+
WARNS?= 2
HAS_TESTS=
Modified: head/sbin/dhclient/dhclient.c
==============================================================================
--- head/sbin/dhclient/dhclient.c Sun Nov 12 08:34:25 2017 (r325739)
+++ head/sbin/dhclient/dhclient.c Sun Nov 12 08:42:43 2017 (r325740)
@@ -84,6 +84,8 @@ __FBSDID("$FreeBSD$");
#define CLIENT_PATH "PATH=/usr/bin:/usr/sbin:/bin:/sbin"
+cap_channel_t *capsyslog;
+
time_t cur_time;
time_t default_lease_time = 43200; /* 12 hours... */
@@ -345,6 +347,21 @@ die:
exit(1);
}
+static void
+init_casper(void)
+{
+ cap_channel_t *casper;
+
+ casper = cap_init();
+ if (casper == NULL)
+ error("unable to start casper");
+
+ capsyslog = cap_service_open(casper, "system.syslog");
+ cap_close(casper);
+ if (capsyslog == NULL)
+ error("unable to open system.syslog service");
+}
+
int
main(int argc, char *argv[])
{
@@ -356,9 +373,11 @@ main(int argc, char *argv[])
pid_t otherpid;
cap_rights_t rights;
+ init_casper();
+
/* Initially, log errors to stderr as well as to syslogd. */
- openlog(__progname, LOG_PID | LOG_NDELAY, DHCPD_LOG_FACILITY);
- setlogmask(LOG_UPTO(LOG_DEBUG));
+ cap_openlog(capsyslog, __progname, LOG_PID | LOG_NDELAY, DHCPD_LOG_FACILITY);
+ cap_setlogmask(capsyslog, LOG_UPTO(LOG_DEBUG));
while ((ch = getopt(argc, argv, "bc:dl:p:qu")) != -1)
switch (ch) {
@@ -518,7 +537,7 @@ main(int argc, char *argv[])
setproctitle("%s", ifi->name);
- if (cap_enter() < 0 && errno != ENOSYS)
+ if (CASPER_SUPPORT && cap_enter() < 0 && errno != ENOSYS)
error("can't enter capability mode: %m");
if (immediate_daemon)
Modified: head/sbin/dhclient/dhcpd.h
==============================================================================
--- head/sbin/dhclient/dhcpd.h Sun Nov 12 08:34:25 2017 (r325739)
+++ head/sbin/dhclient/dhcpd.h Sun Nov 12 08:42:43 2017 (r325740)
@@ -73,6 +73,9 @@
#include <time.h>
#include <unistd.h>
+#include <libcasper.h>
+#include <casper/cap_syslog.h>
+
#include "dhcp.h"
#include "tree.h"
@@ -352,6 +355,7 @@ int addr_eq(struct iaddr, struct iaddr);
char *piaddr(struct iaddr);
/* dhclient.c */
+extern cap_channel_t *capsyslog;
extern char *path_dhclient_conf;
extern char *path_dhclient_db;
extern time_t cur_time;
Modified: head/sbin/dhclient/dispatch.c
==============================================================================
--- head/sbin/dhclient/dispatch.c Sun Nov 12 08:34:25 2017 (r325739)
+++ head/sbin/dhclient/dispatch.c Sun Nov 12 08:42:43 2017 (r325740)
@@ -298,7 +298,8 @@ interface_status(struct interface_info *ifinfo)
memset(&ifr, 0, sizeof(ifr));
strlcpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name));
if (ioctl(ifsock, SIOCGIFFLAGS, &ifr) < 0) {
- syslog(LOG_ERR, "ioctl(SIOCGIFFLAGS) on %s: %m", ifname);
+ cap_syslog(capsyslog, LOG_ERR, "ioctl(SIOCGIFFLAGS) on %s: %m",
+ ifname);
goto inactive;
}
@@ -316,9 +317,8 @@ interface_status(struct interface_info *ifinfo)
strlcpy(ifmr.ifm_name, ifname, sizeof(ifmr.ifm_name));
if (ioctl(ifsock, SIOCGIFMEDIA, (caddr_t)&ifmr) < 0) {
if (errno != EINVAL) {
- syslog(LOG_DEBUG, "ioctl(SIOCGIFMEDIA) on %s: %m",
- ifname);
-
+ cap_syslog(capsyslog, LOG_DEBUG,
+ "ioctl(SIOCGIFMEDIA) on %s: %m", ifname);
ifinfo->noifmedia = 1;
goto active;
}
@@ -479,8 +479,8 @@ interface_link_status(char *ifname)
if (ioctl(sock, SIOCGIFMEDIA, (caddr_t)&ifmr) == -1) {
/* EINVAL -> link state unknown. treat as active */
if (errno != EINVAL)
- syslog(LOG_DEBUG, "ioctl(SIOCGIFMEDIA) on %s: %m",
- ifname);
+ cap_syslog(capsyslog, LOG_DEBUG,
+ "ioctl(SIOCGIFMEDIA) on %s: %m", ifname);
close(sock);
return (1);
}
Modified: head/sbin/dhclient/errwarn.c
==============================================================================
--- head/sbin/dhclient/errwarn.c Sun Nov 12 08:34:25 2017 (r325739)
+++ head/sbin/dhclient/errwarn.c Sun Nov 12 08:42:43 2017 (r325740)
@@ -69,7 +69,7 @@ error(char *fmt, ...)
va_end(list);
#ifndef DEBUG
- syslog(log_priority | LOG_ERR, "%s", mbuf);
+ cap_syslog(capsyslog, log_priority | LOG_ERR, "%s", mbuf);
#endif
/* Also log it to stderr? */
@@ -78,7 +78,7 @@ error(char *fmt, ...)
write(2, "\n", 1);
}
- syslog(LOG_CRIT, "exiting.");
+ cap_syslog(capsyslog, LOG_CRIT, "exiting.");
if (log_perror) {
fprintf(stderr, "exiting.\n");
fflush(stderr);
@@ -103,7 +103,7 @@ warning(char *fmt, ...)
va_end(list);
#ifndef DEBUG
- syslog(log_priority | LOG_ERR, "%s", mbuf);
+ cap_syslog(capsyslog, log_priority | LOG_ERR, "%s", mbuf);
#endif
if (log_perror) {
@@ -129,7 +129,7 @@ note(char *fmt, ...)
va_end(list);
#ifndef DEBUG
- syslog(log_priority | LOG_INFO, "%s", mbuf);
+ cap_syslog(capsyslog, log_priority | LOG_INFO, "%s", mbuf);
#endif
if (log_perror) {
@@ -155,7 +155,7 @@ debug(char *fmt, ...)
va_end(list);
#ifndef DEBUG
- syslog(log_priority | LOG_DEBUG, "%s", mbuf);
+ cap_syslog(capsyslog, log_priority | LOG_DEBUG, "%s", mbuf);
#endif
if (log_perror) {
@@ -217,10 +217,10 @@ parse_warn(char *fmt, ...)
va_end(list);
#ifndef DEBUG
- syslog(log_priority | LOG_ERR, "%s", mbuf);
- syslog(log_priority | LOG_ERR, "%s", token_line);
+ cap_syslog(capsyslog, log_priority | LOG_ERR, "%s", mbuf);
+ cap_syslog(capsyslog, log_priority | LOG_ERR, "%s", token_line);
if (lexline < 81)
- syslog(log_priority | LOG_ERR,
+ cap_syslog(capsyslog, log_priority | LOG_ERR,
"%s^", &spaces[sizeof(spaces) - lexchar]);
#endif
More information about the svn-src-all
mailing list