crash with ValueError in load()

Bug #205178 reported by Sebastien Bacher
4
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
Medium
Martin Pitt

Bug Description

Binary package hint: apport

The retracer duplicate finder was crashing on bug #204316

"problem_report.py", line 139, in load
    (key, value) = line.split(':', 1)
ValueError: need more than 1 value to unpack"

The functions seems to expect lines to start with a space or to be splitable, in this case the description had a copy of the apport informations without the space which created the confusion

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

Right, it should just ignore such broken bugs, since without leading spaces it's utterly hard to parse automatically.

Changed in apport:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Daniel Hahler (blueyed) wrote :

Bug 211899 appears to be related.

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

Fixed in bzr head, rolled out to the retracers.

Changed in apport:
assignee: nobody → pitti
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apport - 0.110

---------------
apport (0.110) intrepid; urgency=low

  * apport/chroot.py: In the test suite, copy some system binaries/libraries
    into a fakechroot and exercise a lot of standard shell commands (cp, ln
    -s, rm, rm -r, mkdir, echo, chmod, chown, etc.) with absolute/relative
    paths. This reproduces the total breakage of rm'ing, chmod'ing, and
    chown'ing absolute paths in hardy fakechroots.
  * bin/crash-digger: Intercept exceptions when downloading crash reports for
    duplicate checking, so that the retracer does not crash on malformed bug
    reports. (LP: #205178)
  * apport/packaging.py: Introduce a new function enabled() which reports
    whether Apport should create crash reports. Signal crashes are controlled
    by /proc/sys/kernel/core_pattern, but we need that to control whether
    reports for Python, package, or kernel crashes are generated.
  * backends/packaging-apt-dpkg.py: Provide implementation for
    PackageInfo.enabled() for Debian/Ubuntu by evaluating /etc/default/apport.
    Add various test cases for different configuration files and absent files.
  * apport_python_hook.py: Do not create reports if Apport is disabled (in
    /etc/default/apport). (LP: #222260)

 -- Martin Pitt <email address hidden> Sat, 17 May 2008 12:44:21 +0200

Changed in apport:
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.