Bug 10256

Summary:	oxygen-gtk3 makes apps to segfault with broadway backend
Product:	Mageia	Reporter:	Thierry Vignaud <thierry.vignaud>
Component:	Backports	Assignee:	Mageia Bug Squad <bugsquad>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Normal	CC:	fundawang, hugo.pereira, juan.baptiste, olav
Version:	Cauldron
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	oxygen-gtk3-1.1.3-1.mga4	CVE:
Status comment:
Attachments:	GDB trace with symbols fixes the issue. Apply against gtk3-1.1.3 or 1.1.4 New patch. this one without the change to the version number in CMakeLists.txt

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