Thank you for reporting this bug and helping to make Ubuntu better.

I apologize for the delay. This isn't really my area of expertise so
could you please include some more information so I can test and verify
whether a fix works:

1. the specific steps or actions you took that caused you to encounter the 
problem,
2. the behavior you expected, and
3. the behavior you actually encountered (in as much detail as possible).

** Changed in: gnome-session (Ubuntu)
       Status: New => Incomplete

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-session in Ubuntu.
https://bugs.launchpad.net/bugs/1101154

Title:
  all signal handlers have been removed in error

Status in “gnome-session” package in Ubuntu:
  Incomplete

Bug description:
  gnome-session/main.c has disabled all signal handling, but intention
  was seemingly to only disable SIGSEGV signal handing:

  10:43 <jodh> chrisccoulson: hi - I'm looking at how gnome-session currently
               shuts down and trying to understand why the following is
               commented out: http://paste.ubuntu.com/1544640/
  10:44 <chrisccoulson> jodh, IIRC it was to stop gnome-session from swallowing
                        those signals so we got crash reports with apport
  10:44 <chrisccoulson> but this was a long time ago ;)
  10:49 <jodh> chrisccoulson: ok, thanks.
  10:49 <chrisccoulson> jodh, technically, i think only the call to
                        gdm_signal_handler_add_fatal() needs to be commented out
  10:49 <chrisccoulson> i can't remember why we disabled all signal handling
  10:50 <chrisccoulson> i've slept and drank beer since then ;)
  10:50 <pitti> chrisccoulson: or perhaps a merge error? in the original patch
                the commented out section might only have been for SIGSEGV, and
                in newer upstream versions the SIGTERMs got added?
  10:52 <chrisccoulson> pitti, yeah, you're right
  10:52 <chrisccoulson> the original patch was
                        
http://bazaar.launchpad.net/~ubuntu-desktop/gnome-session/ubuntu/revision/18
  10:52 <chrisccoulson> jodh ^^
  10:53 <chrisccoulson> it got changed here by seb128:
                        
http://bazaar.launchpad.net/~ubuntu-desktop/gnome-session/ubuntu/revision/233
  10:53 <chrisccoulson> not sure why ;)
  10:53 <pitti> that doesn't look intended
  10:54 <chrisccoulson> jodh, so, the short summary is - the patch is only meant
                        to disable the SEGV handler ;)

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-session/+bug/1101154/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to