AssertionError on .proc.driver.nvidia.gpus.0000:01:00.0

Bug #1566975 reported by Barry Warsaw on 2016-04-06
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Medium
Martin Pitt

Bug Description

I tried to use ubuntu-bug to report a problem with X and got an AssertionError because ProblemReport wasn't expecting a key with colons in it. I fixed this by using a regular expression.

Merge proposal coming up momentarily.

Related branches

Martin Pitt (pitti) wrote :

This needs to be fixed in xdiagnose's apport hook - ':' is an invalid character in key names.

affects: apport → xdiagnose (Ubuntu)
Changed in xdiagnose (Ubuntu):
status: New → Triaged
Martin Pitt (pitti) wrote :

Whoops, this is in apport after all -- xdiagnose's hook calls apport.hookutils.attach_file_if_exists().

Changed in xdiagnose (Ubuntu):
status: Triaged → In Progress
assignee: nobody → Martin Pitt (pitti)
affects: xdiagnose (Ubuntu) → apport (Ubuntu)
Changed in apport (Ubuntu):
assignee: Martin Pitt (pitti) → nobody
Martin Pitt (pitti) wrote :
Changed in apport (Ubuntu):
status: In Progress → Fix Committed
importance: Undecided → Medium
assignee: nobody → Martin Pitt (pitti)
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apport - 2.20.1-0ubuntu2

---------------
apport (2.20.1-0ubuntu2) xenial; urgency=medium

   * Merge fixes from trunk:
     - problem_report.py: Make assertion of invalid key names more verbose.
     - hookutils.py: Fix generation of valid report key names from arbitrary
       paths in attach_file() and related functions. This will now replace all
       invalid characters with dots, not just a few known invalid ones.
       (LP: #1566975)
     - problem_report.py: Instead of AssertionError, raise a ValueError for
       invalid key names and TypeError for invalid kinds of values. Thanks
       Barry Warsaw.
  * Disable Launchpad crash upload for final Ubuntu 16.04.

 -- Martin Pitt <email address hidden> Wed, 13 Apr 2016 23:53:46 +0200

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

Other bug subscribers