From: Simon Tatham <anakin@pobox.com>
Date: Sun, 7 Jun 2015 11:46:33 +0000 (+0100)
Subject: Make log messages look slightly nicer.
X-Git-Tag: 0.68~523
X-Git-Url: https://asedeno.scripts.mit.edu/gitweb/?a=commitdiff_plain;ds=sidebyside;h=d0c74a115a80c0dfdd4f2ecb057f24afbdbaae97;p=PuTTY.git

Make log messages look slightly nicer.

I'd rather see the cipher and MAC named separately, with a hint that
the two are linked together in some way, than see the cipher called by
a name including the MAC and the MAC init message have an ugly
'<implicit>' in it.
---

diff --git a/ssh.c b/ssh.c
index d89dfa38..c5650596 100644
--- a/ssh.c
+++ b/ssh.c
@@ -7139,9 +7139,10 @@ static void do_ssh2_transport(Ssh ssh, const void *vin, int inlen,
 
     logeventf(ssh, "Initialised %.200s client->server encryption",
 	      ssh->cscipher->text_name);
-    logeventf(ssh, "Initialised %.200s client->server MAC algorithm%s",
+    logeventf(ssh, "Initialised %.200s client->server MAC algorithm%s%s",
 	      ssh->csmac->text_name,
-              ssh->csmac_etm ? " (in ETM mode)" : "");
+              ssh->csmac_etm ? " (in ETM mode)" : "",
+              ssh->cscipher->required_mac ? " (required by cipher)" : "");
     if (ssh->cscomp->text_name)
 	logeventf(ssh, "Initialised %s compression",
 		  ssh->cscomp->text_name);
@@ -7206,9 +7207,10 @@ static void do_ssh2_transport(Ssh ssh, const void *vin, int inlen,
     }
     logeventf(ssh, "Initialised %.200s server->client encryption",
 	      ssh->sccipher->text_name);
-    logeventf(ssh, "Initialised %.200s server->client MAC algorithm%s",
+    logeventf(ssh, "Initialised %.200s server->client MAC algorithm%s%s",
 	      ssh->scmac->text_name,
-              ssh->scmac_etm ? " (in ETM mode)" : "");
+              ssh->scmac_etm ? " (in ETM mode)" : "",
+              ssh->sccipher->required_mac ? " (required by cipher)" : "");
     if (ssh->sccomp->text_name)
 	logeventf(ssh, "Initialised %s decompression",
 		  ssh->sccomp->text_name);
diff --git a/sshccp.c b/sshccp.c
index b38d257e..51a71450 100644
--- a/sshccp.c
+++ b/sshccp.c
@@ -438,7 +438,7 @@ static const struct ssh_mac ssh2_poly1305 = {
     poly_start, poly_bytes, poly_genresult, poly_verresult,
 
     "", "", /* Not selectable individually, just part of ChaCha20-Poly1305 */
-    16, "<implicit>"
+    16, "Poly1305"
 };
 
 static void *ccp_make_context(void)
@@ -527,7 +527,7 @@ static const struct ssh2_cipher ssh2_chacha20_poly1305 = {
     ccp_decrypt_length,
 
     "chacha20-poly1305@openssh.com",
-    1, 512, SSH_CIPHER_SEPARATE_LENGTH, "ChaCha20 Poly1305",
+    1, 512, SSH_CIPHER_SEPARATE_LENGTH, "ChaCha20",
 
     &ssh2_poly1305
 };