crash with ValueError in load()

Bug #205178 reported by Sebastien Bacher on 2008-03-22
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Martin Pitt

Bug Description

Binary package hint: apport

The retracer duplicate finder was crashing on bug #204316

"", 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

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
Daniel Hahler (blueyed) wrote :

Bug 211899 appears to be related.

Martin Pitt (pitti) wrote :

Fixed in bzr head, rolled out to the retracers.

Changed in apport:
assignee: nobody → pitti
status: Triaged → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apport - 0.110

apport (0.110) intrepid; urgency=low

  * apport/ 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/ 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/ Provide implementation for
    PackageInfo.enabled() for Debian/Ubuntu by evaluating /etc/default/apport.
    Add various test cases for different configuration files and absent files.
  * 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  Edit
Everyone can see this information.

Other bug subscribers