Fix dovecot with empty 334 challenge.
authorPhil Pennock <pdp@exim.org>
Mon, 30 Sep 2013 04:57:07 +0000 (00:57 -0400)
committerPhil Pennock <pdp@exim.org>
Mon, 30 Sep 2013 04:57:07 +0000 (00:57 -0400)
commit970ba64f07bf5523c7098235664f2ce02962a128
treea63d0f9bd021f18747deafefde8b3ad216fa9c57
parent12d0043db4d843869ed6e85dcb1c87c17bc8b82e
Fix dovecot with empty 334 challenge.

Thomas Morper reported, with 4.82RC1, that he saw "334 NULL" as the
challenge when using AUTH PLAIN to Dovecot when the client does not send
an initial response.  I could replicate.

This was caused by commit 3f1df0e3 on 2012-11-19 (PP/13 of 4.82); I was
too cautious in the robustness fixes; the clue came in this line of
debug output:

    76430 dovecot: warning: ignoring trailing tab

This change removes that check, and documents in a comment that this
input is acceptable protocol-wise, and why.

With this fix:

    AUTH PLAIN
    334
    AGZyZWRlcmljAGh1bXB0eS1kdW1wdHk=
    235 Authentication succeeded
src/src/auths/dovecot.c