Mercurial > hg > dhcpcd

changeset 2257:5d6ccc5a3b11 draft

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Improve authentication logging.
author Roy Marples <roy@marples.name>
date Thu, 30 Jan 2014 14:20:24 +0000
parents bab1e1a8f94e
children 83f7b828306e
files dhcp.c dhcp6.c
diffstat 2 files changed, 27 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/dhcp.c	Thu Jan 30 13:27:49 2014 +0000
+++ b/dhcp.c	Thu Jan 30 14:20:24 2014 +0000
@@ -2046,9 +2046,9 @@
 }
 
 static void
-log_dhcp(int lvl, const char *msg,
+log_dhcp1(int lvl, const char *msg,
     const struct interface *iface, const struct dhcp_message *dhcp,
-    const struct in_addr *from)
+    const struct in_addr *from, int ad)
 {
 	const char *tfrom;
 	char *a;
@@ -2057,7 +2057,7 @@
 
 	if (strcmp(msg, "NAK:") == 0)
 		a = get_option_string(dhcp, DHO_MESSAGE);
-	else if (dhcp->yiaddr != 0) {
+	else if (ad && dhcp->yiaddr != 0) {
 		addr.s_addr = dhcp->yiaddr;
 		a = strdup(inet_ntoa(addr));
 		if (a == NULL) {
@@ -2069,10 +2069,14 @@
 
 	tfrom = "from";
 	r = get_option_addr(&addr, dhcp, DHO_SERVERID);
-	if (dhcp->servername[0] && r == 0)
-		syslog(lvl, "%s: %s %s %s %s `%s'", iface->name, msg, a,
-		    tfrom, inet_ntoa(addr), dhcp->servername);
-	else {
+	if (dhcp->servername[0] && r == 0) {
+		if (a == NULL)
+			syslog(lvl, "%s: %s %s %s `%s'", iface->name, msg,
+			    tfrom, inet_ntoa(addr), dhcp->servername);
+		else
+			syslog(lvl, "%s: %s %s %s %s `%s'", iface->name, msg, a,
+			    tfrom, inet_ntoa(addr), dhcp->servername);
+	} else {
 		if (r != 0) {
 			tfrom = "via";
 			addr = *from;
@@ -2087,6 +2091,15 @@
 	free(a);
 }
 
+static void
+log_dhcp(int lvl, const char *msg,
+    const struct interface *iface, const struct dhcp_message *dhcp,
+    const struct in_addr *from)
+{
+
+	log_dhcp1(lvl, msg, iface, dhcp, from, 1);
+}
+
 static int
 blacklisted_ip(const struct if_options *ifo, in_addr_t addr)
 {
@@ -2138,18 +2151,18 @@
 		{
 			syslog(LOG_DEBUG, "%s: dhcp_auth_validate: %m",
 			    iface->name);
-			log_dhcp(LOG_ERR, "authentication failed",
-			    iface, dhcp, from);
+			log_dhcp1(LOG_ERR, "authentication failed",
+			    iface, dhcp, from, 0);
 			return;
 		}
 		syslog(LOG_DEBUG, "%s: validated using 0x%08" PRIu32,
 		    iface->name, state->auth.token->secretid);
 	} else if (ifo->auth.options & DHCPCD_AUTH_REQUIRE) {
-		log_dhcp(LOG_ERR, "missing authentiation", iface, dhcp, from);
+		log_dhcp1(LOG_ERR, "no authentication", iface, dhcp, from, 0);
 		return;
 	} else if (ifo->auth.options & DHCPCD_AUTH_SEND)
-		log_dhcp(LOG_WARNING, "missing authentiation",
-		    iface, dhcp, from);
+		log_dhcp1(LOG_WARNING, "no authentication",
+		    iface, dhcp, from, 0);
 
 	/* reset the message counter */
 	state->interval = 0;
--- a/dhcp6.c	Thu Jan 30 13:27:49 2014 +0000
+++ b/dhcp6.c	Thu Jan 30 14:20:24 2014 +0000
@@ -2169,12 +2169,12 @@
 		syslog(LOG_DEBUG, "%s: validated using 0x%08" PRIu32,
 		    ifp->name, state->auth.token->secretid);
 	} else if (ifo->auth.options & DHCPCD_AUTH_REQUIRE) {
-		syslog(LOG_ERR, "%s: missing authentiation from %s",
+		syslog(LOG_ERR, "%s: no authentication from %s",
 		    ifp->name, sfrom);
 		return;
 	} else if (ifo->auth.options & DHCPCD_AUTH_SEND)
 		syslog(LOG_WARNING,
-		    "%s: missing authentiation from %s",
+		    "%s: no authentication from %s",
 		    ifp->name, sfrom);
 
 	op = dhcp6_get_op(r->type);