[PATCH] closedir missing in timsieved

[Thomas Cataldo]

On Tue, May 26, 2009 at 1:47 AM, Thomas Cataldo
<thomas.cataldo at aliasource.fr> wrote:
> Hello,
>
> While writing a sieve client lib for the minig (minig.org) webmail
> project, I triggered a bug in the LISTSCRIPTS implementation in cyrus
> 2.2 from debian etch. The directory seems to remain unclosed and this
> is a problem when prefork > 0 is set on timsieved.
>
> The attached patch should fix it.
>

Same patch needed on 2.3 cvs :

Index: actions.c
===================================================================
RCS file: /cvs/src/cyrus/timsieved/actions.c,v
retrieving revision 1.46
diff -u -r1.46 actions.c
--- actions.c   14 Jan 2009 15:50:47 -0000      1.46
+++ actions.c   26 May 2009 00:09:09 -0000
@@ -555,6 +555,8 @@
        }
     }

+    closedir(dp);
+
     prot_printf(conn,"OK\r\n");

     return TIMSIEVE_OK;