"ASSERTION: nsStandardURL not thread-safe" (nsJARInputThunk marked as threadsafe uses nsJARURI in a non threadsafe manner)

RESOLVED FIXED in mozilla1.9.2a1

Status

()

Core
Networking: JAR
--
major
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: Jesse Ruderman, Assigned: timeless)

Tracking

(Blocks: 1 bug, {assertion})

Trunk
mozilla1.9.2a1
assertion
Points:
---
Bug Flags:
blocking1.9 -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

10 years ago
Created attachment 285405 [details]
stack trace

I hit this assertion a few hours ago:

###!!! ASSERTION: nsStandardURL not thread-safe: '_mOwningThread.GetThread() == PR_GetCurrentThread()', file /Users/jruderman/trunk/mozilla/netwerk/base/src/nsStandardURL.cpp, line 931

I can't reproduce, but I do have a stack trace, thanks to XPCOM_DEBUG_BREAK=stack and fix-macosx-stack.pl.

I think the assertion was triggered by a missing-plugin bar appearing.  Maybe that required Firefox to load some extra script or the missing-plugin image from a jar file?
(Reporter)

Updated

10 years ago
Flags: blocking1.9?

Comment 1

10 years ago
Jesse - why did you nom this for blocking?
(Reporter)

Comment 2

10 years ago
Because thread safety violations scare me.  (Should they scare me?)

Comment 3

10 years ago
Minusing - if we get a reproducible test case and/or if we are security concerned please re-nom
Flags: blocking1.9? → blocking1.9-
(Assignee)

Updated

9 years ago
Summary: "ASSERTION: nsStandardURL not thread-safe" within nsJARURI::~nsJARURI → "ASSERTION: nsStandardURL not thread-safe" (nsJARInputThunk marked as threadsafe uses nsJARURI in a non threadsafe manner)
Target Milestone: --- → psm1.3
(Assignee)

Comment 4

9 years ago
Created attachment 336610 [details] [diff] [review]
don't hold uris they're for the main thread only
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #336610 - Flags: review?(cbiesinger)
Comment on attachment 336610 [details] [diff] [review]
don't hold uris they're for the main thread only

+        NS_ENSURE_STATE(mJarDirSpec.IsEmpty());

Don't you mean !mJarDirSpec.IsEmpty()?
Attachment #336610 - Flags: review?(cbiesinger) → review+
(Assignee)

Updated

9 years ago
Target Milestone: psm1.3 → mozilla2.0
timeless: one little review comment in comment 5 saying you got the patch backward ;) and you need to request sr.
Whiteboard: [timeless: need new patch][timeless: request sr]
(Assignee)

Comment 7

9 years ago
Created attachment 355311 [details] [diff] [review]
fixed
Attachment #285405 - Attachment is obsolete: true
Attachment #336610 - Attachment is obsolete: true
Attachment #355311 - Flags: review+
Attachment #355311 - Flags: superreview?(cbiesinger)
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: [timeless: need new patch][timeless: request sr] → [timeless: pending-sr]
Target Milestone: mozilla2.0 → mozilla1.9.2a1
Attachment #355311 - Flags: superreview?(cbiesinger) → superreview+
http://hg.mozilla.org/mozilla-central/rev/4c8363cef88b
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Whiteboard: [timeless: pending-sr]
You need to log in before you can comment on or make changes to this bug.