apport should not report SIGXCPU and SIGXFSZ "crashes"

Bug #498074 reported by Kees Cook
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: apport

Brian Murray pointed out bug 418297 to me since I touched odt2txt last. As it turns out, these crashes are from tracker setting soft RLIMITs on CPU time (line 143, src/libtracker-common/tracker-os-dependant-unix.c), and odt2txt was hitting them. This kind of thing should probably be considered "normal" behavior, and doesn't really need a crash report. Either these signals should be ignored, or a new kind of crash should be reported that includes the RLIMIT details as well as a process ancestry to find who set the limits (and this doesn't seem like a general-purpose action really).

Branch attached to ignore the signals. Simple reproducer (note that bash ignores SIGXCPU by default):

$ bash
$ ulimit -St 5
$ trap "echo XCPU >&2; exit 1" 24
$ while :; do echo monkey >/dev/null; done
XCPU
$ echo $?
1

ProblemType: Bug
ApportLog:

Architecture: amd64
CrashReports:
 600:0:0:10419:2009-12-16 19:28:18.000000000 -0800:2009-12-16 19:28:19.000000000 -0800:/var/crash/_usr_share_gnome-app-install_update-app-install.0.crash
 600:501:501:70302:2009-12-16 19:44:30.000000000 -0800:2009-12-16 19:44:31.000000000 -0800:/var/crash/_usr_share_terminator_terminator.501.crash
Date: Thu Dec 17 20:44:21 2009
DistroRelease: Ubuntu 10.04
Package: apport 1.9.6-0ubuntu1
PackageArchitecture: all
ProcEnviron:
 LANGUAGE=en_US.UTF-8
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.32-7.10-generic
SourcePackage: apport
Tags: lucid
Uname: Linux 2.6.32-7-generic x86_64

Revision history for this message
Kees Cook (kees) wrote :
Revision history for this message
Martin Pitt (pitti) wrote :

trunk r1659, thanks Kees!

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 - 1.10-0ubuntu1

---------------
apport (1.10-0ubuntu1) lucid; urgency=low

  * New upstream release:
    - Add a mode for updating an existing problem report to ui.py
      (-u/--update). This is similar to the Ubuntu specific "apport-collect"
      tool, but implemented the right way now: In particular, this has access
      to the UI and thus can use interactive hooks (LP: #385811) and show you
      what is being sent for confirmation/cancelling (LP: #371827)

    - apport-bug: If invoked as "apport-collect" or "apport-update-bug" (i. e.
      through a symlink), run apport in update mode (-u <number>). This
      provides a convenient no-options command line program. Please note that
      setup.py does not currently install such a symlink. Update the
      apport-bug manpage accordingly.

    - launchpad.py: Use new login_with() to clean up code, and specify allowed
      access levels (WRITE_PRIVATE is the only sensible one anyway).
      (LP: #410205)

    - New hookutils functions:
      + xsession_errors (match lines from ~/.xsession-errors)
      + shared_libraries (determine which libraries a binary links with)
      + links_with_shared_library (test if a binary links with a particular
        library)

    - New CrashDatabase API: get_affected_packages(), can_update(), is_reporter()

    - Rename CrashDatabase.update() to update_traces().

    - Add CrashDatabase.update() for adding all new fields of a report. This is
      primarily useful for collecting local standard and package hook data for an
      already existing bug report which was not filed through Apport. This checks
      can_update()/is_reporter() if the user is eligible for updating that
      particular bug. (LP: #485880)

    - Ignore SIGXCPU and SIGXFSZ; thanks to Kees Cook. (LP: #498074)

    - launchpad.py: Do not mark non-Ubuntu bugs as needs-retrace, since there is
      no retracer right now. (LP: #489794)

    - packaging-apt-dpkg.py, install_retracing_packages(): Do not crash on
      malformed Dependencies.txt lines. (LP: #441709)

    - use-local: Fix for new source tree location of "apport" binary.

  * Drop debian/local/apport-collect{,.1} and install symlinks for apport-bug
    instead.
  * data/general-hooks/ubuntu.py: Do not report "corrupted filesystem tarfile"
    package errors. (LP: #320743)
  * data/general-hooks/ubuntu.py: Report "package ... is already installed and
    configured" errors against dpkg, not the package that failed. (LP: #467688)
 -- Martin Pitt <email address hidden> Sat, 19 Dec 2009 16:48:48 +0100

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.