On 05/04/2012 08:54 PM, Timo Sirainen wrote:
On 16.4.2012, at 12.55, Michal Hlavinka wrote:

I wrote simple patch that close the extra sockets. It's tested and works fine. 
You'll maybe want to move that function to different place and/or change 
wording of error messages.

I committed it to v2.1 now with a couple of changes. One is that it doesn't 
actually close the fd, but instead puts /dev/null into it. I think otherwise 
Dovecot might use that fd to something else and the check would later fail 
again and close the wrong fd.

http://hg.dovecot.org/dovecot-2.1/rev/4a3bf567da54

Thanks. I tested it, but it does not work because of the ret == 0 check before services_verify_systemd. ret is usually 1

Michal

Reply via email to