Closed Bug 1408693 Opened 6 years ago Closed 6 years ago

Crash in std::_Tree<T>::find


(Core :: Audio/Video: Playback, defect, P2)

56 Branch



Tracking Status
firefox-esr52 --- unaffected
firefox56 --- unaffected
firefox57 --- fixed
firefox58 --- fixed


(Reporter: yoasif, Assigned: bechen)



(Keywords: crash, regression)

Crash Data


(2 files)

This bug was filed from the Socorro interface and is 
report bp-aa04e1e9-4617-4582-991c-a32fa0171013.

Only 17 crashes in the last week, but didn't see it logged in Bugzilla. Affects v57.
Component: General → Audio/Video
Blocks: 1393399
Flags: needinfo?(kaku)
Keywords: regression

This code isn’t thread safe. It could be accessed from multiple threads, erasing an element on one thread, to find in another is bound to create nastiness.
Kaku is on "P"TO. 
Can you help check it?
Component: Audio/Video → Audio/Video: Playback
Flags: needinfo?(kaku) → needinfo?(bechen)
Priority: -- → P2
(In reply to Blake Wu [:bwu][:blakewu] from comment #2)
> Kaku is on "P"TO. 
> Benjamin,
> Can you help check it?

Looks like we need a mutex lock to protect it.
Flags: needinfo?(bechen)
Comment on attachment 8918747 [details]
Bug 1408693 - Add a lock to protect the sGPUCrashDataMap.
Attachment #8918747 - Flags: review?(jwwang) → review+
Assignee: nobody → bechen
Keywords: checkin-needed
Pushed by
Add a lock to protect the sGPUCrashDataMap. r=jwwang
Keywords: checkin-needed
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Is this worth uplifting to 57?  Crash volume seems low, OTOH the bug sounds possibly nasty.
Flags: needinfo?(bechen)
I believe we need uplift this patch to 57, because the telemetry result is very important for "de-blacklist". Once the crash happened, we can not gather the GPU crash status due to "de-blacklist". That will mislead us to decide the blacklist should be exist or not.

Approval Request Comment
[Feature/Bug causing the regression]: 1393399
[User impact if declined]: Low possibility crash when trying to report telemetry.
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: No, very hard to reproduce.
[Needs manual test from QE? If yes, steps to reproduce]: No.
[List of other uplifts needed for the feature/fix]: None.
[Is the change risky?]: No.
[Why is the change risky/not risky?]: code logic very simple.
[String changes made/needed]: none
Flags: needinfo?(bechen)
Attachment #8919549 - Flags: approval-mozilla-beta?
Comment on attachment 8919549 [details] [diff] [review]

Better telemetry data, low risk, Beta57+
Attachment #8919549 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Hi Jean-Yves, fyi, I took this patch in beta57. If you believe this one is better off staying in 58 train, please let me know. Thanks!
Flags: needinfo?(jyavenard)
I think this one is pretty safe for uplifting...
Flags: needinfo?(jyavenard)
You need to log in before you can comment on or make changes to this bug.