SUMMARY: AddressSanitizer: heap-use-after-free /builds/worker/workspace/build/src/clang/bin/../lib/gcc/x86_64-unknown-linux-gnu/4.9.4/../../../../include/c++/4.9.4/bits/atomic_base.h:628:16 in fetch_sub
Categories
(Core :: WebRTC: Audio/Video, defect, P1)
Tracking
()
People
(Reporter: aryx, Assigned: bwc)
References
(Blocks 1 open bug)
Details
(Keywords: csectype-uaf, sec-high)
Updated•7 years ago
|
Comment 1•7 years ago
|
||
Comment 2•7 years ago
|
||
Updated•7 years ago
|
Comment 3•7 years ago
|
||
Comment 4•6 years ago
|
||
Reporter | ||
Comment 5•6 years ago
|
||
Comment 6•6 years ago
|
||
Comment 7•6 years ago
|
||
As I don't even see the EverySecondTelemetryTimer any more in our code I'm assuming this bug is no longer relevant. Nico what do you say?
Comment 8•6 years ago
|
||
I think this may still be an issue. I am doing a deeper dive today.
Comment 9•6 years ago
|
||
There haven't been any crashes in the last 6 months. I think this may have been resolved when :bwc refactored the GetStats to use promises in bug 1494312.
Comment 10•6 years ago
|
||
(In reply to Nico Grunbaum [:ng] from comment #9)
There haven't been any crashes in the last 6 months. I think this may have been resolved when :bwc refactored the GetStats to use promises in bug 1494312.
Since this only ever crashed on treeherder and bug 1494312 landed in 65 and thus this has been running for months on treeherder I would assume it's safe to call this fixed.
Comment 11•6 years ago
|
||
Is there anything we can realistically do for ESR60 here?
Updated•6 years ago
|
Assignee | ||
Comment 12•6 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #11)
Is there anything we can realistically do for ESR60 here?
This work is not going to be possible to port to 60 without bringing a lot of other stuff with it. It is not impossible to try to fix it in a more limited way, but that would basically be a new bug.
Updated•5 years ago
|
Updated•5 years ago
|
Updated•4 years ago
|
Description
•