! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Xfdesktop segfault when setting the window background
Status:
CLOSED: FIXED
Product:
Xfdesktop
Component:
General

Comments

Description Nick Schermer editbugs 2006-07-21 16:26:27 CEST
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.4) Gecko/20060602 Firefox/1.5.0.4 (Swiftfox)
Build Identifier: Trunk Rev 22504

Xfdesktop crashes directly on startup, when it tries to run the style_set_cb function. Not every PC seems to have this problem, but it happend on my laptop.

Reproducible: Always

Steps to Reproduce:
Comment 1 Nick Schermer editbugs 2006-07-21 16:28:32 CEST
Created attachment 669 
Only update the background when really needed.

This might even speed up startup a bit, because the function is called 2 times when you start xfdesktop.
Comment 2 Brian J. Tarricone (not reading bugmail) 2006-07-21 16:55:37 CEST
That's not going to fix the crash, though.  Can I get a backtrace?

And the second patch band is unncessary.
Comment 3 Nick Schermer editbugs 2006-07-21 19:10:00 CEST
Well it solved the problem here, tho. Anyway bt is attached.
Comment 4 Nick Schermer editbugs 2006-07-21 19:10:41 CEST
Created attachment 670 
Bracktrace 1
Comment 5 Brian J. Tarricone (not reading bugmail) 2006-07-21 19:17:07 CEST
You fixed a symptom, not the cause...  though the bt seems to show that gtk is setting the style before realising the window, which is kinda weird.  Either way, your patch plus another check for GTK_WIDGET_REALIZED() should be sufficient.
Comment 6 Brian J. Tarricone (not reading bugmail) 2006-07-21 20:06:23 CEST
Actually, it occurs to me that your patch is a bad idea.  We *have* to set the back pixmap each time, regardless if the theme has actually changed or not, because whenever a style is applied that has its own back pixmap, it'll override ours.

The crash should be fixed now though.
Comment 7 Nick Schermer editbugs 2006-07-23 10:32:00 CEST
Yep, everything works fine now.

Bug #2053

Reported by:
Nick Schermer
Reported on: 2006-07-21
Last modified on: 2009-07-14

People

Assignee:
Brian J. Tarricone (not reading bugmail)
CC List:
0 users

Version

Attachments

Only update the background when really needed. (931 bytes, patch)
2006-07-21 16:28 CEST , Nick Schermer
no flags
Bracktrace 1 (1.62 KB, text/plain)
2006-07-21 19:10 CEST , Nick Schermer
no flags

Additional information