aboutsummaryrefslogtreecommitdiffstats
path: root/driver.c
diff options
context:
space:
mode:
Diffstat (limited to 'driver.c')
-rw-r--r--driver.c15
1 files changed, 10 insertions, 5 deletions
diff --git a/driver.c b/driver.c
index d33acfd1..56f358b6 100644
--- a/driver.c
+++ b/driver.c
@@ -1808,14 +1808,19 @@ const int maxfetch; /* maximum number of messages to fetch */
stage = STAGE_GETAUTH;
if (protocol->getauth)
{
- if (protocol->password_canonify)
- (protocol->password_canonify)(shroud, ctl->password, PASSWORDLEN);
- else
- strcpy(shroud, ctl->password);
+ /*
+ * We want to restrict shrouding as much as possible -- it
+ * might actually leak information by splatting out revealing
+ * pieces of a message.
+ */
+ if (ctl->server.authenticate == A_PASSWORD)
+ if (protocol->password_canonify)
+ (protocol->password_canonify)(shroud, ctl->password, PASSWORDLEN);
+ else
+ strcpy(shroud, ctl->password);
ok = (protocol->getauth)(mailserver_socket, ctl, buf);
- /* prevent shrouding later on -- it might backfire */
shroud[0] = '\0';
if (ok != 0)