From: Ken Raeburn <raeburn@mit.edu>
Date: Fri, 13 Feb 2009 22:23:24 +0000 (+0000)
Subject: Be less verbose about routing-socket messages
X-Git-Tag: krb5-1.8-alpha1~629
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=91fc077c96926dd60097a413ca37ebd3a70155c1;p=krb5.git

Be less verbose about routing-socket messages

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@22005 dc483132-0cff-0310-8789-dd5450dbe970
---

diff --git a/src/kdc/network.c b/src/kdc/network.c
index 4ebaf2b32..e3d8a4cad 100644
--- a/src/kdc/network.c
+++ b/src/kdc/network.c
@@ -809,7 +809,6 @@ static void process_routing_update(struct connection *conn, const char *prog,
     int n_read;
     struct rt_msghdr rtm;
 
-    krb5_klog_syslog(LOG_INFO, "routing socket readable");
     while ((n_read = read(conn->fd, &rtm, sizeof(rtm))) > 0) {
 	if (n_read < sizeof(rtm)) {
 	    /* Quick hack to figure out if the interesting
@@ -828,10 +827,12 @@ static void process_routing_update(struct connection *conn, const char *prog,
 		return;
 	    }
 	}
+#if 0
 	krb5_klog_syslog(LOG_INFO,
 			 "got routing msg type %d(%s) v%d",
 			 rtm.rtm_type, rtm_type_name(rtm.rtm_type),
 			 rtm.rtm_version);
+#endif
 	if (rtm.rtm_msglen > sizeof(rtm)) {
 	    /* It appears we get a partial message and the rest is
 	       thrown away?  */
@@ -848,7 +849,11 @@ static void process_routing_update(struct connection *conn, const char *prog,
 	case RTM_IFINFO:
 	case RTM_OLDADD:
 	case RTM_OLDDEL:
-	    krb5_klog_syslog(LOG_INFO, "reconfiguration needed");
+#if 0
+	    krb5_klog_syslog(LOG_DEBUG,
+			     "network reconfiguration message (%s) received",
+			     rtm_type_name(rtm.rtm_type));
+#endif
 	    network_reconfiguration_needed = 1;
 	    break;
 	case RTM_RESOLVE:
@@ -861,10 +866,14 @@ static void process_routing_update(struct connection *conn, const char *prog,
 	case RTM_LOSING:
 	case RTM_GET:
 	    /* Not interesting.  */
+#if 0
 	    krb5_klog_syslog(LOG_DEBUG, "routing msg not interesting");
+#endif
 	    break;
 	default:
-	    krb5_klog_syslog(LOG_INFO, "unhandled routing message type, will reconfigure just for the fun of it");
+	    krb5_klog_syslog(LOG_INFO,
+			     "unhandled routing message type %d, will reconfigure just for the fun of it",
+			     rtm.rtm_type);
 	    network_reconfiguration_needed = 1;
 	    break;
 	}
@@ -1610,7 +1619,9 @@ listen_and_process(const char *prog)
 	}
 
 	if (network_reconfiguration_needed) {
-	    krb5_klog_syslog(LOG_INFO, "network reconfiguration needed");
+	    /* No point in re-logging what we've just logged.  */
+	    if (netchanged == 0)
+		krb5_klog_syslog(LOG_INFO, "network reconfiguration needed");
 	    /* It might be tidier to add a timer-callback interface to
 	       the control loop here, but for this one use, it's not a
 	       big deal.  */