Open Bug 730498 Opened 11 years ago Updated 7 years ago

##!!! ASSERTION: nsCOMArrayEnumerator not thread-safe: '_mOwningThread.GetThread() == PR_GetCurrentThread()', file mozilla/xpcom/build/nsArrayEnumerator.cpp, line 161

Categories

(MailNews Core :: Import, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

People

(Reporter: hiro, Unassigned)

References

Details

(Whiteboard: [patchlove])

Attachments

(1 file)

The test in bug 700920 comment #3 causes this assertion.
We need to fix this.
Blocks: 700920
Attached patch Proposed fixSplinter Review
nsISupportsArray which is an array of nsIImportMailboxDescriptor is generated in importing thread, so we don't have to dispatch to the main thread.
Attachment #600569 - Flags: review?(dbienvenu)
Comment on attachment 600569 [details] [diff] [review]
Proposed fix

the problem is that these classes really aren't thread-safe, so making the isupports thread-safe isn't sufficient, other than to quiet the warnings which are really legitimate warnings.

Fixing the thread-safety assertions might well involve rewriting the import code not to run on its own thread. Fixing this particular assertion probably involves figuring out a way of getting a thread-safe array or enumerator out of the mailnews backend.
Attachment #600569 - Flags: review?(dbienvenu) → review-
comment 2 is currently our basic problem with import
Whiteboard: [patchlove]
You need to log in before you can comment on or make changes to this bug.