After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 500655 - GDM crashes when trying to bring up chooser
GDM crashes when trying to bring up chooser
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
unspecified
Other Windows
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2007-11-30 15:05 UTC by Ray Strode [halfline]
Modified: 2007-11-30 18:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch that made crash go away (426 bytes, patch)
2007-11-30 15:13 UTC, Ray Strode [halfline]
reviewed Details | Review

Description Ray Strode [halfline] 2007-11-30 15:05:09 UTC
(from https://bugzilla.redhat.com/show_bug.cgi?id=403961)

Philippe Troin noticed a crash when using the XDMCP button in the circles theme.

1. Set up gdm with a theme that display the remote login button (eg. circles) as
the default theme is very limited (ahem).
2. Click on "Remote login via XDMCP"
3. Pick a host that runs XDMCP
4. See gdm crash
5. The following is logged:
kernel: gdm-binary[10567]: segfault at 000004d0 eip 00885328 esp bf922a80 error 4
gdm-binary[10350]: WARNING: gdm_cleanup_children: child 10567 crashed of signal 11
gdm-binary[10350]: WARNING: gdm_cleanup_children: Slave crashed, killing its
children

He attached a patch that fixed the problem for him.
Comment 1 Ray Strode [halfline] 2007-11-30 15:13:40 UTC
Created attachment 99908 [details] [review]
patch that made crash go away
Comment 2 Ray Strode [halfline] 2007-11-30 15:19:19 UTC
Philippe, does this patch make XDMCP work or just stop the crash?

I'm a little confused as to why display is NULL in the gdm_get_current_vtnum () call. 

We only make that call once, in slave.c, and at that point, the only way d->dsp could be NULL is if XOpenDisplay failed, but we already do

gdm_slave_quick_exit (DISPLAY_ABORT);

if d->dsp is NULL earlier.
Comment 3 Brian Cameron 2007-11-30 18:15:54 UTC
Ray, note that this has already been fixed in 2.20.2.  I wrote the exact same change on 11-13-2007.
Comment 4 Ray Strode [halfline] 2007-11-30 18:23:42 UTC
oh okay, I'll just push a 2.20.2 update to F8.
Comment 5 Philippe Troin 2007-11-30 18:42:28 UTC
Ray,
Since you're pushing 2.20.2 to F8, would you mind adding the Suspend patch from
RH Bugzilla #387001 (Gnome Bugzilla #500362)?

Phil.
Comment 6 Ray Strode [halfline] 2007-11-30 18:59:06 UTC
yup, sure.