apport hookutils.attach_drm assumes unicode, crashes

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

Bug Description

apport hookutils.attach_drm open()s files as if they are text, and then processes the results as if they're unicode. Both are trouble.

Related branches

Chad Miller (cmiller)
Changed in apport:
assignee: nobody → Chad Miller (cmiller)
Revision history for this message
Martin Pitt (pitti) wrote :
Changed in apport:
status: New → Fix Released
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.16.2-0ubuntu1

---------------
apport (2.16.2-0ubuntu1) vivid; urgency=medium

  * New upstream bug fix release:
    - ProblemReport: Set a timestamp of 0 in gzip compressed fields; they are
      meaningless and cause unnecessary jitter in the output.
    - launchpad backend: Fix unclosed file in upload().
    - launchpad backend: Fix wrong use of filter() with Python 3.
    - launchpad backend download(): Try to convert textual values from byte
      arrays into into strings.
    - ui.py, collect_info(): Fix crash on bug pattern checking with broken
      gzipped values. (LP: #1345653)
    - hookutils, attach_drm_info(): Avoid UnicodeDecodeErrors in Python 3 when
      reading binary files. Thanks Chad Miller. (LP: #1425254)
    - apport-gtk: Update legacy icon names to modern GTK ones, to fix icons
      under GNOME. Thanks Scott Sanbar. (LP: #1422176)
    - Move backend_apt_dpkg testsuite to use Ubuntu 14.04 LTS.
    - hookutils, attach_dmesg(): Only attach dmesg as CurrentDmesg, drop
      BootDmesg as /var/log/dmesg is upstart specific and thus not reliably
      correct any more.
    - hookutils, recent_syslog(): Read system logs from the journal when
      running under systemd, and fall back to /var/log/syslog if not.
    - hookutils, attach_mac_events(): Read kernel violation messages from
      dmesg instead of /var/log/kern.log, as that's specific to rsyslog and
      its configuration.
 -- Martin Pitt <email address hidden> Mon, 02 Mar 2015 11:37:32 +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.