Closed Bug 1427088 Opened 2 years ago Closed 2 years ago

Crash in mozilla::gl::GLContext::InitWithPrefixImpl


(Core :: Graphics, defect, critical)

57 Branch
Not set



Tracking Status
firefox-esr52 --- unaffected
firefox57 --- wontfix
firefox58 --- wontfix
firefox59 --- fixed


(Reporter: philipp, Assigned: jgilbert)



(Keywords: crash, regression, Whiteboard: gfx-noted)

Crash Data


(1 file)

This bug was filed from the Socorro interface and is
report bp-5bf45d47-9d8f-41e2-b7af-4bb7a0171223.

Top 10 frames of crashing thread:

0 mozilla::gl::GLContext::InitWithPrefixImpl gfx/gl/GLContext.cpp:867
1 mozilla::gl::GLContext::InitWithPrefix gfx/gl/GLContext.cpp:347
2 mozilla::gl::GLContextEGL::Init gfx/gl/GLContextProviderEGL.cpp:284
3 mozilla::gl::GLContextEGL::CreateGLContext gfx/gl/GLContextProviderEGL.cpp:541
4 mozilla::gl::GLContextEGL::CreateEGLPBufferOffscreenContext gfx/gl/GLContextProviderEGL.cpp:942
5 mozilla::gl::GLContextProviderEGL::CreateHeadless gfx/gl/GLContextProviderEGL.cpp:960
6 mozilla::widget::GfxInfo::GLStrings::EnsureInitialized widget/android/GfxInfo.cpp:76
7 mozilla::widget::GfxInfo::EnsureInitialized widget/android/GfxInfo.cpp:37
8 mozilla::widget::GfxInfo::GetFeatureStatusImpl widget/android/GfxInfo.cpp:403
9 mozilla::widget::GfxInfoBase::GetFeatureStatus widget/GfxInfoBase.cpp:650


this fennec crash signature is regressing in volume in firefox 57 (but has been around before that already).
most of the crash reports come with MOZ_RELEASE_ASSERT(!err) added in bug 1339256.
Please help advise
Flags: needinfo?(bwu)
Per comment 0, is this a regression from bug 1339256?
Flags: needinfo?(bwu) → needinfo?(jgilbert)
Uh, yeah, but that should indeed never ever be crashing.
Assignee: nobody → jgilbert
Component: General → Graphics
Flags: needinfo?(jgilbert)
Product: Firefox for Android → Core
Whiteboard: gfx-noted
Version: Firefox 57 → 57 Branch
This is the assert:
We can remove it, but the only time I can see it being hit is:
* Sudden context loss
* A bug we're not picking up on
So at least the ff57 reports have 0x501 in r0, which is INVALID_VALUE. (ff58 has r0:0x0, but that's probably a (bad) optimization)
My guess is that we're querying GL_NUM_EXTENSIONS because we see we have fGetStringi, but that entrypoint isn't actually supposed to be used for some reason. (es2 contexts?) This would cause INVALID_VALUE.
Comment on attachment 8940854 [details]
Bug 1427088 - Don't rely on glGetStringi just because it exists. -

Attachment #8940854 - Flags: review?(dmu) → review+
Pushed by
Don't rely on glGetStringi just because it exists. - r=daoshengmu
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
it is something we want to uplift to 58?
Flags: needinfo?(jgilbert)
(In reply to Sylvestre Ledru [:sylvestre] from comment #10)
> it is something we want to uplift to 58?

This seems pretty low-volume, android-only, so probably not. If I'm reading the crash volume wrong, I'm willing to uplift it into late beta, but otherwise let's just let this already-on-release bug ride out 59.
Flags: needinfo?(jgilbert)
You need to log in before you can comment on or make changes to this bug.