Re: Squatter crash with statusdb

2013-06-24 Thread Jeroen van Meeuwen (Kolab Systems)
On 2013-06-24 13:24, Simon Matter wrote: >> Hi Andy, could you file a bug for this? Then it will not be >> forgotten... > > Or, could you check this bug here > http://bugzilla.cyrusimap.org/show_bug.cgi?id=3757 > > The patch below was the fix, could you verify if it also fixes your > issue? >

Re: Squatter crash with statusdb

2013-06-24 Thread Andy Fiddaman
On Mon, 24 Jun 2013, Simon Matter wrote: ; ; Or, could you check this bug here ; http://bugzilla.cyrusimap.org/show_bug.cgi?id=3757 ; ; The patch below was the fix, could you verify if it also fixes your issue? Yes that patch fixes my issue too. I did search bugzilla but didn't see that one becau

Re: Squatter crash with statusdb

2013-06-24 Thread Simon Matter
> Hi Andy, could you file a bug for this? Then it will not be forgotten... Or, could you check this bug here http://bugzilla.cyrusimap.org/show_bug.cgi?id=3757 The patch below was the fix, could you verify if it also fixes your issue? Thanks, Simon >From 1661683d453ea444aae5832b4a2cb7fd5448967

Re: Squatter crash with statusdb

2013-06-24 Thread Rudy Gevaert
Hi Andy, could you file a bug for this? Then it will not be forgotten... Quoting Andy Fiddaman , Mon, 24 Jun 2013: > FWIW, this gets it working again: > > --- cyrus-imapd-2.4.17.dist/imap/statuscache_db.c 2013-06-24 > 10:10:08.219203100 + > +++ cyrus-imapd-2.4.17/imap/statuscache_db.c

Re: Squatter crash with statusdb

2013-06-24 Thread Andy Fiddaman
FWIW, this gets it working again: --- cyrus-imapd-2.4.17.dist/imap/statuscache_db.c 2013-06-24 10:10:08.219203100 + +++ cyrus-imapd-2.4.17/imap/statuscache_db.c2013-06-24 10:10:20.537711377 + @@ -152,7 +152,7 @@ len = strlcpy(key, mailboxname, sizeof(key)); key[len++]

Squatter crash with statusdb

2013-06-24 Thread Andy Fiddaman
Hi, I've just upgraded my Cyrus installation to 2.4.17 and squatter is crashing in statuscache_buildkey() because userid is NULL. I'm not sure what the best fix for this is. Should squatter even be using the statuscache or should it populate "cyrus" as the username when initialising the index, o