Closed Bug 1439881 Opened 3 years ago Closed 3 years ago

Crash in nsNativeThemeGTK::DrawWidgetBackground

Categories

(Core :: Widget: Gtk, defect)

Unspecified
Linux
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox-esr52 --- unaffected
firefox58 --- unaffected
firefox59 --- unaffected
firefox60 --- fixed

People

(Reporter: linuxhippy, Assigned: jhorak)

References

Details

(Keywords: crash, regression)

Crash Data

Attachments

(1 file)

I experience this crash rather frequently on fedora 27, kde, 4k monitor with scaling=1.4 and the Adwaita GTK3 theme.

This bug was filed from the Socorro interface and is
report bp-4131e761-8916-4633-8a47-c89b10180221.
=============================================================

Top 10 frames of crashing thread:

0 libxul.so nsNativeThemeGTK::DrawWidgetBackground 
1 libxul.so nsDisplayThemedBackground::PaintInternal 
2 libxul.so mozilla::FrameLayerBuilder::PaintItems 
3 libxul.so mozilla::FrameLayerBuilder::DrawPaintedLayer 
4 libxul.so mozilla::layers::BasicPaintedLayer::PaintThebes 
5 libxul.so mozilla::layers::BasicLayerManager::PaintLayer 
6 libxul.so mozilla::layers::BasicLayerManager::PaintLayer 
7 libxul.so mozilla::layers::BasicLayerManager::EndTransactionInternal 
8 libxul.so nsDisplayList::PaintRoot 
9 libxul.so nsLayoutUtils::PaintFrame 

=============================================================
According to crash-stats this started in nightly build-id 20180220220044, with 2 installs crashing multiple times.
All of these are SIGFPE.  Martin, could any of your recent widget/gtk explain this?
Flags: needinfo?(stransky)
(In reply to Julien Cristau [:jcristau] from comment #2)
> All of these are SIGFPE.  Martin, could any of your recent widget/gtk
> explain this?

Yes, it's possible that scaleFactor at nsNativeThemeGTK::DrawWidgetBackground() is 0 which may cause the divide error. May be a regression from Bug 1431337. I see that crash comes from child process which uses pupet widget implementation. Adding Jan.

Clements, what do you mean by "scaling=1.4"? Do you have set devPixelsPerPx at about:config or Gtk+ font scale for whole system?
Flags: needinfo?(stransky)
Assignee: nobody → jhorak
Depends on: 1431337
Flags: needinfo?(linuxhippy)
> Clements, what do you mean by "scaling=1.4"? 
> Do you have set devPixelsPerPx at about:config or Gtk+ font scale for whole system?

I previously had devPixelsPerPx=1.4, but recentl I switched to devPixelsPerPx=-1 and have a GTK+ font scale of 1.4 system wide.
However, this change did not have any influence on the crash behaviour.
Flags: needinfo?(linuxhippy)
Comment on attachment 8953050 [details]
Bug 1439881 - Avoid returning monitor scale 0;

https://reviewboard.mozilla.org/r/222324/#review228224
Attachment #8953050 - Flags: review?(stransky) → review+
Hm, try looks strange:

[task 2018-02-22T14:21:47.149Z] 14:21:47    ERROR - GECKO(2904) | ==2999==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x7f5a181f9a53 bp 0x7f5a1426e300 sp 0x7f5a1426e2e0 T2
Flags: needinfo?(stransky)
(In reply to Martin Stránský [:stransky] from comment #7)
> Hm, try looks strange:
> 
> [task 2018-02-22T14:21:47.149Z] 14:21:47    ERROR - GECKO(2904) |
> ==2999==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc
> 0x7f5a181f9a53 bp 0x7f5a1426e300 sp 0x7f5a1426e2e0 T2

Seems to be Bug 1436383 not related to this one.
Flags: needinfo?(stransky)
Pushed by stransky@redhat.com:
https://hg.mozilla.org/integration/autoland/rev/31c59ada523a
Avoid returning monitor scale 0; r=stransky
https://hg.mozilla.org/mozilla-central/rev/31c59ada523a
Status: UNCONFIRMED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
I am seeing lots of warning on console;

[Parent 4530, Main Thread] WARNING: Invalid monitor scale: -1: file /home/ikezoe/stylo/widget/gtk/nsNativeThemeGTK.cpp, line 85

They are very noisy.  Can we void spewing the message?
Flags: needinfo?(jhorak)
(In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
> I am seeing lots of warning on console;
> 
> [Parent 4530, Main Thread] WARNING: Invalid monitor scale: -1: file
> /home/ikezoe/stylo/widget/gtk/nsNativeThemeGTK.cpp, line 85
> 
> They are very noisy.  Can we void spewing the message?

*avoid*
(In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
> I am seeing lots of warning on console;
> 
> [Parent 4530, Main Thread] WARNING: Invalid monitor scale: -1: file
> /home/ikezoe/stylo/widget/gtk/nsNativeThemeGTK.cpp, line 85
> 
> They are very noisy.  Can we void spewing the message?

Follow up Bug 1441108.
Flags: needinfo?(jhorak)
You need to log in before you can comment on or make changes to this bug.