dovecot - Jul 2010 - 2.2rc1 doveadm segfault

[root at sbh16 ~]# doveadm help
Segmentation fault (core dumped)
[root at sbh16 ~]# gdb /usr/local/bin/doveadm core.28104
[...]
(gdb) bt full
#0  0x461a2da8 in strcmp () from /lib/libc.so.6
No symbol table info available.
#1  0x009aed70 in module_dir_load_missing (old_modules=0x0,
    dir=0x805fc23 "/usr/local/lib/dovecot", module_names=0x805d3a8
"",
    set=0xbfac058c) at module-dir.c:263
        arr = <value optimized out>
        _data_stack_cur_id = 2
        modules = <value optimized out>
#2  0x08053b08 in doveadm_mail_init () at doveadm-mail.c:554
        mod_set = {version = 0x9bfa0c "2.0.rc1", require_init_funcs =
1,
  debug = 0, ignore_dlopen_errors = 0}
        i = <value optimized out>
#3  0x0804e79b in main (argc=2, argv=Cannot access memory at address
0x6f6c5f70) at doveadm.c:343
        cmd_name = 0x8c7e1d4 "help"
        i = <value optimized out>
        quick_init = false
        c = <value optimized out>
(gdb)

-- 
Mark Sapiro <mark at msapiro.net>        The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan

Mark Sapiro wrote:
> [root at sbh16 ~]# doveadm help
> Segmentation fault (core dumped)
> [root at sbh16 ~]# gdb /usr/local/bin/doveadm core.28104
> [...]
> (gdb) bt full
This patch fixes things for me:

diff -r 4a4f7add34d9 src/lib/module-dir.c
--- a/src/lib/module-dir.c      Sat Jul 03 15:53:54 2010 +0100
+++ b/src/lib/module-dir.c      Sun Jul 04 17:58:59 2010 +0200
@@ -252,6 +252,9 @@
  {
         unsigned int i, j;

+       if ( module_names[0] == NULL )
+               return;
+
         /* allow giving the module names also in non-base form.
            convert them in here. */
         for (i = 0; module_names[i] != NULL; i++)


Regards,

Stephan.

Stephan Bosch wrote:
>Mark Sapiro wrote:
>> [root at sbh16 ~]# doveadm help
>> Segmentation fault (core dumped)
>> [root at sbh16 ~]# gdb /usr/local/bin/doveadm core.28104
>> [...]
>> (gdb) bt full
>
>This patch fixes things for me:
>
>diff -r 4a4f7add34d9 src/lib/module-dir.c
>--- a/src/lib/module-dir.c      Sat Jul 03 15:53:54 2010 +0100
>+++ b/src/lib/module-dir.c      Sun Jul 04 17:58:59 2010 +0200
>@@ -252,6 +252,9 @@
>  {
>         unsigned int i, j;
>
>+       if ( module_names[0] == NULL )
>+               return;
>+
>         /* allow giving the module names also in non-base form.
>            convert them in here. */
>         for (i = 0; module_names[i] != NULL; i++)
>

Thank you. It works for me too. (sorry about the typo in the original
subject).

-- 
Mark Sapiro <mark at msapiro.net>        The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan