Ubuntu
apport package

attach_drm_info falls over with weird DRM connector names

Bug #597558 reported by Chris Halse Rogers
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
Medium
Martin Pitt

Bug Description

Binary package hint: apport

There's no guarantee that DRM connector names are valid apport keys. On my system, for example, one of my connectors is “card0-HDMI Type A-1”. The space causes attach_drm_info to die with an AssertionError in __setitem__.

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: apport 1.14-0ubuntu1
ProcVersionSignature: Ubuntu 2.6.35-4.5-generic 2.6.35-rc3
Uname: Linux 2.6.35-4-generic x86_64
Architecture: amd64
Date: Wed Jun 23 16:08:24 2010
PackageArchitecture: all
ProcEnviron:
 LANGUAGE=en_AU:en_GB:en
 PATH=(custom, user)
 LANG=en_AU.utf8
 SHELL=/bin/zsh
SourcePackage: apport

Revision history for this message
Chris Halse Rogers (raof) wrote :
Revision history for this message
Martin Pitt (pitti) wrote :

Nice catch, thanks! I committed a slightly different version to trunk r1780.

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

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

---------------
apport (1.14.1-0ubuntu1) maverick; urgency=low

  * New upstream bug fix release:
    - hookutils.py, attach_drm_info(): Sanitize connector names. Thanks Chris
      Halse Rogers. (LP: #597558)
    - bash completion: Complete all path names, apport-bug can be invoked with
      a path to a program. Thanks Philip Muskovac.
 -- Martin Pitt <email address hidden> Thu, 24 Jun 2010 15:37:19 +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.
Subscribing...

Other bug subscribers

Remote bug watches

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