Bug 10256 - oxygen-gtk3 makes apps to segfault with broadway backend
Summary: oxygen-gtk3 makes apps to segfault with broadway backend
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: Backports (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-05-24 17:38 CEST by Thierry Vignaud
Modified: 2013-06-05 12:44 CEST (History)
4 users (show)

See Also:
Source RPM: oxygen-gtk3-1.1.3-1.mga4
CVE:
Status comment:


Attachments
GDB trace with symbols (4.14 KB, text/plain)
2013-05-24 17:38 CEST, Thierry Vignaud
Details
fixes the issue. Apply against gtk3-1.1.3 or 1.1.4 (5.41 KB, patch)
2013-06-04 09:36 CEST, Hugo Pereira Da Costa
Details | Diff
New patch. (5.92 KB, patch)
2013-06-05 11:13 CEST, Hugo Pereira Da Costa
Details | Diff
this one without the change to the version number in CMakeLists.txt (5.45 KB, patch)
2013-06-05 11:22 CEST, Hugo Pereira Da Costa
Details | Diff

Description Thierry Vignaud 2013-05-24 17:38:33 CEST
oxygen-gtk3 makes apps to segfault with broadway backend.
Tested with gedit, gcalctool, ...

Steps to Reproduce:
1. run "broadwayd"
2. open http://127.0.0.1:8080/ in a browser
3. run eg "GDK_BACKEND=broadway gedit"

It segfaults in libx11, either with current cauldron's one or with mga3's one.
But the real culprit is oxygen-gtk3. Using another theme make the segfault go away.

Reproducible: 

Steps to Reproduce:
Comment 1 Thierry Vignaud 2013-05-24 17:38:53 CEST
Created attachment 4043 [details]
GDB trace with symbols
Thierry Vignaud 2013-05-24 17:39:54 CEST

CC: (none) => fundawang, hugo, juan.baptiste, olav

Comment 2 Hugo Pereira Da Costa 2013-05-24 17:57:38 CEST
yep.
and all gtk3 applications will crash. 
Was not aware of this backend thing, and will need to add checks everywhere X11 specific code is used (mainly to communicate with window decoration, pass shadows to menus, etc.) working on it.

In principle the same is true also with oxygen-gtk2, but since backend is decided at compile time (gtk), and you can only have one backend at a time, it is less of an issue.

Will keep you posted. Thanks for reporting.
(Note: an actual bug report at bugs.kde.org would be very welcome, since not all oxygen-gtk devs are registered to mageia ;))
Comment 3 Hugo Pereira Da Costa 2013-05-24 18:04:05 CEST
Note that here, crash also happens with any other theme I could use (Raleigh, Adwaita), in libcanberra-gtk, so unrelated to the style. 

Will protect oxygen-gtk nonetheless, at least for all crashes that are hit before the one above.
Comment 4 Thierry Vignaud 2013-05-28 11:10:19 CEST
For me other themes works fine.
Anyway, please tell us which commit(s) fixes we whould backport until next oxygen-gtk release
Comment 5 Thierry Vignaud 2013-05-28 11:10:49 CEST
BTW, I'm testing Cauldron with gtk+-3.8, not Mageia 3 with gtk+-3.6
Comment 6 Hugo Pereira Da Costa 2013-06-04 09:36:00 CEST
Created attachment 4098 [details]
fixes the issue. Apply against gtk3-1.1.3 or 1.1.4
Comment 7 Hugo Pereira Da Costa 2013-06-05 11:13:47 CEST
Created attachment 4102 [details]
New patch.

Fixed (stupid) typo that would cause crash as soon as a menu is oppened (for normal X11 backend)

Many sorry for the trouble.
Comment 8 Hugo Pereira Da Costa 2013-06-05 11:22:10 CEST
Created attachment 4103 [details]
this one without the change to the version number in CMakeLists.txt
Thierry Vignaud 2013-06-05 12:43:59 CEST

Attachment 4102 is obsolete: 0 => 1

Thierry Vignaud 2013-06-05 12:44:01 CEST

Attachment 4098 is obsolete: 0 => 1

Comment 9 Thierry Vignaud 2013-06-05 12:44:19 CEST
Fixed

Status: NEW => RESOLVED
Resolution: (none) => FIXED


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