commit 48889844299e036515e567aa5f992df18ccdf2d7
from: Alexander Barton <alex@barton.de>
date: Wed Jan 25 16:11:00 2012 UTC

Client_SetHostname(): Code cleanup, more debug logging

commit - 44bb22d23ec6841457db41732caa6f5f9129b615
commit + 48889844299e036515e567aa5f992df18ccdf2d7
blob - 1aaf687ce6995e8f958daf7d4b98860084234d9a
blob + 078449c0b1c882cd2eeab92ca11409c686a7d46e
--- src/ngircd/client.c
+++ src/ngircd/client.c
@@ -313,16 +313,29 @@ Client_Destroy( CLIENT *Client, const char *LogMsg, co
 } /* Client_Destroy */
 
 
+/**
+ * Set client hostname.
+ *
+ * If global hostname cloaking is in effect, don't set the real hostname
+ * but the configured one.
+ *
+ * @param Client The client of which the hostname should be set.
+ * @param Hostname The new hostname.
+ */
 GLOBAL void
 Client_SetHostname( CLIENT *Client, const char *Hostname )
 {
-	assert( Client != NULL );
-	assert( Hostname != NULL );
+	assert(Client != NULL);
+	assert(Hostname != NULL);
 
 	if (strlen(Conf_CloakHost)) {
-		strlcpy( Client->host, Conf_CloakHost, sizeof( Client->host ));
+		LogDebug("Updating hostname of \"%s\": \"%s\" -> \"%s\"",
+			 Client_ID(Client), Client->host, Conf_CloakHost);
+		strlcpy(Client->host, Conf_CloakHost, sizeof(Client->host));
 	} else {
-		strlcpy( Client->host, Hostname, sizeof( Client->host ));
+		LogDebug("Updating hostname of \"%s\": \"%s\" -> \"%s\"",
+			 Client_ID(Client), Client->host, Hostname);
+		strlcpy(Client->host, Hostname, sizeof(Client->host));
 	}
 } /* Client_SetHostname */