Remove nsISimpleEnumerator from import thread

RESOLVED FIXED in Thunderbird 30.0

Status

MailNews Core
Import
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: hiro, Assigned: hiro)

Tracking

Trunk
Thunderbird 30.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
otherwise thunderbird debug build causes a crash:

Hit MOZ_CRASH(nsCOMArrayEnumerator not thread-safe) at /home/zoe/hg/comm-central/mozilla/xpcom/glue/nsArrayEnumerator.cpp:128
(Assignee)

Comment 1

4 years ago
Created attachment 8370543 [details] [diff] [review]
bug968023.patch
Assignee: nobody → hiikezoe
Status: NEW → ASSIGNED
Attachment #8370543 - Flags: review?(neil)
(Assignee)

Updated

4 years ago
Blocks: 684455

Comment 2

4 years ago
Comment on attachment 8370543 [details] [diff] [review]
bug968023.patch

> NS_IMETHODIMP GetSubFoldersRunnable::Run()
> {
>-  return m_folder->GetSubFolders(m_enumerator);
>+  return m_folder->GetSubFolders(nullptr);
This confused me briefly until I noticed that passing nullptr is an existing pattern here (normally that's not allowed for XPCOM out parameters).
Attachment #8370543 - Flags: review?(neil) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/22a4aeca5215
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 30.0
You need to log in before you can comment on or make changes to this bug.