1148069 - crash in js::PutEscapedStringImpl

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect)

Description

[Trevor Rowbotham [:rowbot]]

While trying to profile a performance issue with playing 4K videos on YouTube, all tabs would crash shortly after clicking Analyze in the Gecko Profiler.  This crash happened about 75% of the time.  I am using the latest 64-bit nightly on Windows 7.  e10s is enabled.

STR:
1) Install Gecko Profiler[1] addon
2) Play a YouTube video at 4K using the HTML5 player (I was profiling [2])
3) Profile during playback for at least 10-15 seconds
4) Click Analyze in the Gecko Profiler addon

Actual results:
All tabs crashed.

Expected results:
Profile should be loaded in cleopatra.

bp-f7c38532-253f-47d8-b8d7-253842150326
bp-c56f0787-6257-457d-b855-3d3cc2150326
bp-911feb79-c66f-4fee-98b2-a8b992150326

[1] https://developer.mozilla.org/en-US/docs/Mozilla/Performance/Profiling_with_the_Built-in_Profiler
[2] https://www.youtube.com/watch?v=s0MDY9fl-IA

Comment 2

[Alice0775 White]

Sorry spam,
please ignore comment#1

Comment 3

[Alice0775 White]

[Tracking Requested - why for this release]: Crash since Nightly39

I think the following regression range is probably correct.

Regression window(m-c):
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2a404169de2d&tochange=f949be6cd23e

Regression window(m-i):
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=8d14c6661f00&tochange=8bd7b499e291

Comment 4

[Shu-yu Guo [:shu]]

Attachment: Set SyncProfiles' buffers to an invalid generation.

Comment 5

[Shu-yu Guo [:shu]]

Alice or Trevor, could you confirm that the patch fixes the crash for you?

Comment 6

[Alice0775 White]

(In reply to Shu-yu Guo [:shu] from comment #5)
> Alice or Trevor, could you confirm that the patch fixes the crash for you?

Build from m-c tip 67f8d63b2cad : crashes
Build from m-c tip 67f8d63b2cad + applied attachment 8584825 [details] [diff] [review] : no crash (attempted 5 times)

So, the patch seems to fix the crash problem on Windows7.

Comment 7

[Trevor Rowbotham [:rowbot]]

Clearing ni? since Alice answered already.  I'm not setup to build Firefox right now, but I'll be sure to test it out when it lands in nightly.

Comment 8

[Phil Ringnalda (:philor)]

Backed out both https://hg.mozilla.org/integration/mozilla-inbound/rev/c68a6ebe6083 and https://hg.mozilla.org/integration/mozilla-inbound/rev/d1e9b787eaeb for causing, and then failing to fix, devtools-2 leaks.

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/1715529498c8

Comment 10

[Trevor Rowbotham [:rowbot]]

This appears to be fixed.  Thanks!

Comment 11

[Liz Henry (:lizzard)]

Sounds like we don't need to track this, since it's fixed on 39, but please nominate it again if there's any problem.