On 9.5.2012, at 14.49, Michal Hlavinka wrote:
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.
Thanks. I tested it, but it does not work because of the ret == 0 check before services_verify_systemd. ret is usually 1
Oh, right. Fixed: http://hg.dovecot.org/dovecot-2.1/rev/17b20880453c