signal crashes fail when the user has been removed prior to crash

Bug #1163453 reported by Evan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

oem-config removes the temporary user account, but its binaries will outlive this. In a recent case, indicator-session was crashing and in turn causing apport to crash while it attempted to look up the ignore file for that user:

http://paste.ubuntu.com/5670896/

ERROR: apport (pid 19818) Tue Apr 2 16:28:06 2013: Unhandled exception:
Traceback (most recent call last):
  File "/usr/share/apport/apport", line 369, in <module>
    if info.check_ignored():
  File "/usr/lib/python3/dist-packages/apport/report.py", line 955, in check_ignored
    dom = self._get_ignore_dom()
  File "/usr/lib/python3/dist-packages/apport/report.py", line 892, in _get_ignore_dom
    ifpath = os.path.expanduser(_ignore_file)
  File "/usr/lib/python3.3/posixpath.py", line 261, in expanduser
    userhome = pwd.getpwuid(os.getuid()).pw_dir

Related branches

Martin Pitt (pitti)
Changed in apport (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apport - 2.10-0ubuntu1

---------------
apport (2.10-0ubuntu1) saucy; urgency=low

  [ Martin Pitt ]
  * New upstream release. Changes since our previous snapshot:
    - Support retracing foreign powerpc reports, thanks Steve Langasek.
    - apport/report.py: Generate a crash signature for suspend/resume failures.
    - hookutils.py: Add attach_upstart_logs(), thanks Steve Langasek.
    - hookutils.py, in_session_of_problem(): Port from ConsoleKit to logind.
    - apport/report.py: Handle the case where the user has been removed from
      the system, but one of its still-running binaries crashes (LP: #1163453).
    - Fix anonymization of user/GECOS fields with regexp control characters
      like '+'. (LP: #985049)
    - Run tests under LC_CTYPE=C and unset LANG as well, to discover
      assumptions about UTF-8 locales. Fix the two failing tests.
    - Fix UnicodeDecodeError when apport encounters non-ASCII environment
      variables. (LP: #1172638)
  * debian/control: Update Vcs-* for saucy branch.
  * data/general-hooks/cloud_archive.py: Fix PEP-8 error.

  [ Steve Langasek ]
  * data/general-hooks/ubuntu.py: Call attach_upstart_logs(), to attach the
    logs for any upstart user jobs shipped by the binary package.
 -- Martin Pitt <email address hidden> Tue, 30 Apr 2013 14:49:42 -0700

Changed in apport (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.