358441 - Busy Loop in nsMsgQuickSearchDBView::GetFirstMessageHdrToDisplayInThread

Status: RESOLVED DUPLICATE of bug 332883

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Kai Engert (:KaiE:)]

- I'm displaying a mailbox on my SSL IMAP server.
- I am in threaded display mode.
- I type into the "Subject or Sender contains" quicksearch edit field.

This results in a busy loop, and the application hangs.

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1) Gecko/20061027 SeaMonkey/1.1b

I used the debugger to identify where it loops:

#0  0x06010b15 in nsMsgQuickSearchDBView::GetFirstMessageHdrToDisplayInThread (this=0xaa26930, threadHdr=0xaa26bc0,
    result=0xbfebecf0) at /home/kaie/moz/18/mozilla/mailnews/base/src/nsMsgQuickSearchDBView.cpp:377

373             // count number of ancestors - that's our level
374             while (parentId != nsMsgKey_None)
375             {
376               rv = m_db->GetMsgHdrForKey(parentId, getter_AddRefs(parent));
377               if (parent)
378               {
379                 parent->GetThreadParent(&parentId);
380                 level++;
381               }
382             }

parent is NULL, level is zero, loops forever

I'll also attach a text file containing the full stack.

Comment 1

[Kai Engert (:KaiE:)]

Attachment: stack

Comment 2

[David :Bienvenu]

darn, I thought I fixed that. It's actually a core bug. Taking.

Comment 3

[Serge Gautherie (:sgautherie)]

Bug 332883 was fixed, on trunk and 1.8 branch, on 2006-09-19 16:04:37.
Gecko/20061027 should have had the new code.

Kai, could it be that you built with a non updated file ?
David, I suggest to resolve this bug as a duplicate.

Comment 4

[Kai Engert (:KaiE:)]

(In reply to comment #3)
> Kai, could it be that you built with a non updated file ?

Unlikely.

But FWIW, I think I haven't ran into such a deadlock with SeaMonkey 1.1 (1.8 branch) for a long time. (I use it for private email)

Comment 5

[Wayne Mery (:wsmwk)]

bienvenu in comment #2
> darn, I thought I fixed that. It's actually a core bug. Taking.

bienvenu, patch still needed?

Comment 6

[Wayne Mery (:wsmwk)]

duping per Serge comment 3