Open Bug 1514734 Opened 1 year ago Updated 1 year ago

Crash in mozilla::dom::ipc::SharedStringMap::SharedStringMap

Categories

(Core :: IPC, defect, P3, critical)

Unspecified
All
defect

Tracking

()

People

(Reporter: gsvelto, Unassigned)

Details

(Keywords: crash)

Crash Data

This bug was filed from the Socorro interface and is
report bp-f14f97ca-7d11-4c76-9e78-da9130181215.
=============================================================

Top 10 frames of crashing thread:

0 libxul.so mozilla::dom::ipc::SharedStringMap::SharedStringMap dom/ipc/SharedStringMap.cpp:42
1 libxul.so  intl/strres/nsStringBundle.cpp:519
2 libxul.so  intl/strres/nsStringBundle.cpp:564
3 libxul.so nsStringBundleBase::GetStringFromName intl/strres/nsStringBundle.cpp:552
4 libxul.so mozilla::widget::WidgetUtils::GetBrandShortName widget/WidgetUtils.cpp:144
5 libxul.so nsAppShell::Init widget/gtk/nsAppShell.cpp:173
6 libxul.so nsAppShellInit widget/nsAppShellSingleton.h:42
7 libxul.so nsFactoryEntry::GetFactory xpcom/components/nsComponentManager.cpp:757
8 libxul.so nsComponentManagerImpl::CreateInstance xpcom/components/nsComponentManager.cpp:977
9 libxul.so nsComponentManagerImpl::GetService xpcom/components/nsComponentManager.cpp:1217

=============================================================

We're hitting this assertion:

https://hg.mozilla.org/mozilla-central/annotate/5b0b8a39d09e49efe283b94deba6853e223edad3/dom/ipc/SharedStringMap.cpp#l42

MOZ_RELEASE_ASSERT(result.isOk())
Any ideas, Kris?
Flags: needinfo?(kmaglione+bmo)
This is most likely just an OOM crash.

Although, given that these are almost all on Linux, it might be some weird configuration that limits how much /dev/shm storage we can allocate.
Flags: needinfo?(kmaglione+bmo)
(In reply to Kris Maglione [:kmag] from comment #2)
> This is most likely just an OOM crash.
> 
> Although, given that these are almost all on Linux, it might be some weird
> configuration that limits how much /dev/shm storage we can allocate.

These are almost all startup crashes, OOM seems somewhat less likely.
(In reply to Eric Rahm [:erahm] (ni? for review in phab) from comment #3)
> (In reply to Kris Maglione [:kmag] from comment #2)
> > This is most likely just an OOM crash.
> > 
> > Although, given that these are almost all on Linux, it might be some weird
> > configuration that limits how much /dev/shm storage we can allocate.
> 
> These are almost all startup crashes, OOM seems somewhat less likely.

They'll always be startup crashes, since we allocate all of these shared string bundles at startup.

Unfortunately, none of these crash reports has system memory info, but I've seen similar Windows startup crashes for the shared pref map that were definitely OOM crashes.
Priority: -- → P3
Duplicate of this bug: 1539188

FWIW I am seeing the assertion in question when I try to run a wpt on GeckoView and Android emulator x86_64.

(In reply to Hiroyuki Ikezoe (:hiro) from comment #6)

FWIW I am seeing the assertion in question when I try to run a wpt on GeckoView and Android emulator x86_64.

That's likely a separate issue. Do you have a link to the crash? Or is it only happening locally?

Yeah, it's on a local build with GeckoView's TestRunnerActivity, and I saw a permission error to access a temporary directory which didn't exist at that time, creating the directory by hand solved the assertion, so presumably it's an issue in the TestRunnerActivity or relevant stuff.

You need to log in before you can comment on or make changes to this bug.