[Dovecot] xexec and dovecot 1.1

[Marc Perkel]

Stephan Bosch wrote:
> Alex wrote:
>   
>> Stephan Bosch wrote:
>>     
>>> subscriber at viliar.net.ru schreef:
>>>       
>>>> subscriber at viliar.net.ru schreef:
>>>> Thank you, Stephan, for your help. Patch applies without any problem
>>>> and
>>>> compilation was successful. According maillog, Dovecot load it without
>>>> problems too:
>>>>
>>>> Mar 19 09:06:37 mx3 dovecot: IMAP(alx at skymail.alx.in): Module loaded:
>>>> /usr/lib/dovecot/imap/lib10_xexec_plugin.so
>>>> I have activate it in config as for dovecot 1.0. But "sub commands"
>>>> does
>>>> not started:
>>>>
>>>> ...
>>>>
>>>> [pid  4093] --- SIGSEGV (Segmentation fault) @ 0 (0) ---
>>>> Process 4093 detached
>>>> <... select resumed> )                  = 2 (in [8 10], left {2147482,
>>>> 996000})
>>>> --- SIGCHLD (Child exited) @ 0 (0) --
>>>>         
>>> Found it, apparently dovecot-1.1 does not like a NULL parameter to
>>> the array_get function anymore:
>>>
>>> -        execvp(*(char **)array_idx(&command, 0), array_get(&command,
>>> NULL));
>>> +        execvp(*(char **)array_idx(&command, 0), array_idx(&command,
>>> 0));
>>>
>>> With this additional change, the attached full patch manages to
>>> update the XEXEC plugin to dovecot-1.1. This was tested with the
>>> 'average' example and that worked flawlessly.
>>>
>>> Regards,
>>>
>>> Stephan
>>>
>>>
>>>       
>> Thank you again, Stephan. Now all works perfectly :-) Probably at
>> http://wiki.dovecot.org/Plugins/xexec it is good to have link on
>> patch/new version of module, created by you. Or attach to page file. I
>> can do it, if it's ok. What you think?
>>     
> Fine with me, but I think you should ask the author of this module. He
> can also release a proper new version.
>
> Regards,
>
> Stephan.
>
>   

Go ahead and post it as a new released version. That's fine with me. I'm 
glad someone is using this. What are you all doing with it.