1779980 - Crash in [@ mozilla::dom::workerinternals::loader::ScriptExecutorRunnable::WorkerRun]

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, defect)

Description

[Gabriele Svelto [:gsvelto]]

Crash report: https://crash-stats.mozilla.org/report/index/d3469697-e4af-4cec-b1f0-06b7a0220717

Reason: EXCEPTION_ACCESS_VIOLATION_READ

Top 10 frames of crashing thread:

0 xul.dll mozilla::dom::workerinternals::loader::ScriptExecutorRunnable::WorkerRun dom/workers/ScriptLoader.cpp:1062
1 xul.dll mozilla::dom::WorkerRunnable::Run dom/workers/WorkerRunnable.cpp:377
2 xul.dll nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:1199
3 xul.dll NS_ProcessNextEvent xpcom/threads/nsThreadUtils.cpp:465
4 xul.dll mozilla::dom::WorkerPrivate::DoRunLoop dom/workers/WorkerPrivate.cpp:3205
5 xul.dll mozilla::dom::workerinternals::`anonymous namespace'::WorkerThreadPrimaryRunnable::Run dom/workers/RuntimeService.cpp:2038
6 xul.dll nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:1199
7 xul.dll mozilla::ipc::MessagePumpForNonMainThreads::Run ipc/glue/MessagePump.cpp:300
8 xul.dll MessageLoop::RunHandler ipc/chromium/src/base/message_loop.cc:373
9 xul.dll MessageLoop::Run ipc/chromium/src/base/message_loop.cc:355

Only three crashes from three different users on nightly, looks like a potential regression.

Comment 1

[Jens Stutte [:jstutte]]

I'd suspect bug 1742438 to be related here?

Comment 2

[Yulia Startsev [:yulia] OOO until July 2026]

It may be. I will look into it shortly.

Comment 3

[Yulia Startsev [:yulia] OOO until July 2026]

It looks like this is no longer occurring.

Comment 4

[Jens Stutte [:jstutte]]

(In reply to Yulia Startsev [:yulia] from comment #3)

It looks like this is no longer occurring.

We just had some signature changes given the better support for inlined functions in crash-stats, this is still happening.

Comment 5

[Yulia Startsev [:yulia] OOO until July 2026]

With the new signature I am seeing this happening for 104? https://crash-stats.mozilla.org/signature/?signature=mozilla%3A%3Adom%3A%3Aworkerinternals%3A%3Aloader%3A%3AScriptExecutorRunnable%3A%3AWorkerRun&date=%3E%3D2022-09-23T07%3A03%3A00.000Z&date=%3C2022-09-30T07%3A03%3A00.000Z&_sort=-date#summary is that right or is there another one?

Comment 6

[Jens Stutte [:jstutte]]

It seems to me that this is the old signature? The new one looks like https://crash-stats.mozilla.org/report/index/4aa6d6a3-f95a-4964-8e51-f99aa0220930

Comment 7

[Yulia Startsev [:yulia] OOO until July 2026]

Ah, thanks!

Comment 8

[Jens Stutte [:jstutte]]

Can I assume you'll take this one, too?

Comment 9

[Yulia Startsev [:yulia] OOO until July 2026]

Yes, anything related to this path.

Comment 10

[Yulia Startsev [:yulia] OOO until July 2026]

This appears to be fixed in 108