Re: [PATCH] Re: cyrus saslauthd error handling

2016-04-07 Thread Wietse Venema
Benning, Markus: > vstring_strcpy(reply, xsasl_cyrus_strerror(sasl_status)); > switch (sasl_status) { > + case SASL_FAIL: > + case SASL_NOMEM: > case SASL_TRYAGAIN: > case SASL_UNAVAIL: > return XSASL_AUTH_TEMP; Fixed in Postfix 3.2.

[PATCH] Re: cyrus saslauthd error handling

2016-04-07 Thread Benning, Markus
On 2016-04-06 16:19, Benning, Markus wrote: In sasl.h: #define SASL_FAIL -1 /* generic failure */ Could this one be added to the AUTH_TEMP case? I took a look at the cyrus-sasl code in lib/checkpw.c and most error cases there return SASL_FAIL. Wrong credentails return