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.
>> 
>> 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

Oh, right. Fixed: http://hg.dovecot.org/dovecot-2.1/rev/17b20880453c

Reply via email to