iwlwifi error dumps: not enabled in distro, could contain private information

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

Bug Description

We've been advised that the dumps which are collected from the kernel for debugging iwlwifi kernel errors may contain data which could be considered private, so reports which contain this data should be private.

As luck would have it, we haven't been collecting the dumps anyway. The script to collect them is present in the version of apport shipped in Trusty, but it's not getting installed. When releasing this fix the packaging should get updated to install iwlwifi_error_dump along with a udev rule to trigger the reports. I'm attaching the udev rule that I'm using to test.

FIXES:
http://bazaar.launchpad.net/~apport-hackers/apport/trunk/revision/2793
http://bazaar.launchpad.net/~ubuntu-core-dev/ubuntu/utopic/apport/ubuntu/revision/2308

Revision history for this message
Seth Forshee (sforshee) wrote :
Revision history for this message
Martin Pitt (pitti) wrote :

Privacy marking fix merged upstream. Upstream already has udev/50-apport.rules from your original MP and setup.py installs it, but apparently we were missing the debian/*.install bits for it.

Changed in apport (Ubuntu):
status: New → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :
description: updated
summary: - iwlwifi error dumps could contain private information
+ iwlwifi error dumps: not enabled in distro, could contain private
+ information
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
apport (2.14.2-0ubuntu1) utopic; urgency=medium

  * New upstream release:
    - Move error handling for invalid .crash files into collect_info(), so
      that it also applies when using the "Show Details..." button in the UI.
      Otherwise the UI just hangs eternally at this point when encountering
      broken core dumps. (LP: #1282349)
    - apt/dpkg install_packages(): Try to install the requested package
      version instead of always picking the most recent one. This should
      improve retracing results for older crash reports. Thanks to Brian
      Murray for inspiring this.
    - sandboxutils.py, make_sandbox(): When determining packages to install
      from ProcMaps, look up and use the package versions from the reporter,
      to improve retracing results. Thanks Brian Murray for the initial patch!
    - iwlwifi_error_dump: Make reports private, and subscribe
      canonical-kernel-team. Thanks Seth Forshee. (LP: #1313818)
    - signal_crashes test: Time out after 5 seconds if the test process does
      not terminate on the specified signal, to avoid eternal hangs.
    - signal_crashes test: Ensure that we don't inherit an ignored SIGQUIT
      from the caller.
  * Switch Vcs-* to utopic branch.
  * Add build-essential test dependency, some tests call gcc.
  * Install iwl_wifi_error_dump and corresponding udev rules. (Second part of
    LP: #1313818)
 -- Martin Pitt <email address hidden> Wed, 30 Apr 2014 14:07:17 +0200

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.