Closed Bug 844817 Opened 11 years ago Closed 11 years ago

TSan: Thread data race in nsCOMPtr<nsIFactory>::get() vs. nsCOMPtr_base::assign_assuming_AddRef(nsISupports*)

Categories

(Core :: XPCOM, defect)

x86_64
Linux
defect
Not set
critical

Tracking

()

RESOLVED DUPLICATE of bug 684887

People

(Reporter: posidron, Unassigned)

References

Details

(Keywords: sec-want, Whiteboard: [tsan][tsan-test-blocker])

Attachments

(1 file)

Attached file trace
During Firefox start-up with ThreadSanitizer (LLVM version), we get a data race reported as described in the attached log. Trace was created on mozilla-central with changeset c233837cce08.

According to the TSan devs, most of the reported traces should be real data races, even though they can be "benign". We need to determine if the race can/should be fixed, or put on the ignore list. Even for benign races, TSan devs suggest to fix them (second priority), as they can also cause problems [1].

[1] http://software.intel.com/en-us/blogs/2013/01/06/benign-data-races-what-could-possibly-go-wrong
See Also: → 844794
This looks closely related to bug 833349.
Very cool, thanks :) We'll wait for the patch in bug 684887 to land then.
Depends on: 684887
Let's just mark the dup, this is definitely the same issue.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: