apport-gtk crashed with TypeError in handle_duplicate(): argument of type 'NoneType' is not iterable

Status changed to 'Confirmed' because the bug affects multiple users.

There are only two places where self.report is set back to none:

 * When load_report() fails. In all cases it returns False, in which case run_crash() returns. So it's not that.

 * When collect() encounters an IOError or zlib.error during info collection.

Did you get a message "This problem report is damaged and cannot be processed." before you encountered the crash?

Possibly ... Sorry but i don t remember precisely

:°)

I fixed a crash case when the report is broken in some way, and added a test case for it. For now, let's assume that this was the reason, I don't see any other. If it still happens with the next apport version, I'll re-investigate.

Fixed in trunk r2165.

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

---------------
apport (1.93-0ubuntu1) precise; urgency=low

  * New upstream bug fix release:
    - apport-gtk: Fix crash on nonexisting icon. Thanks Evan Dandrea.
      (LP: #937354)
    - ui.py, open_url(): Revert back to calling sudo instead of dropping
      privileges ourselves; with the latter, calling firefox as the sudo'ing
      user fails. (LP: #916810, #938128)
    - ui.py: Fix aborting with "AssertionError" if the report is already
      known, but without an URL. (LP: #938778)
    - launchpad.py: If a bug is already known, but the report is private, do
      not send the report. There is little sense piling up lots of duplicates.
      (LP: #938700)
    - test/crash: Fix regression of test_crash_apport(), consider $TERM a
      non-sensitive variable.
    - ui.py: Fix test failures for data collection progress, they are not
      expected to happen for "ProblemType: Crash" any more (happens in the
      background during sending, or if user clicks on "Show Details").
    - test/hooks: Use a package from Debian/Ubuntu main, so that this works
      better during package builds on build servers.
    - test/python: Do not assume that /var/crash/ exists. Use /var/tmp/ for
      the fake binaries instead.
    - data/general-hooks/parse_segv.py: Fix test case name.
    - ui.py: Fix crash on invalid core dumps. (LP: #937215)
    - launchpad.py: Fix crash on unicode report titles. (LP: #896626)
    - apport-gtk: Show the most interesting fields first in the details view.
    - do-release: Call pyflakes and abort on errors other than unused imports.
    - Move all test suites out of the code modules into test/test_<module>.py.
      This avoids having to load it every time the program runs, and also
      allows running the tests against the installed version of Apport.
    - Clean up the other executable test script in test/* and change them to
      the same structure as the module tests.
  * debian/control: Add python-mock and GTK gir build dependencies for the
    gtk/kde tests.
  * debian/control: Add procps dependency to avoid a "command not found" error
    for killall in the ubuntu.py general hook during the test suite.
  * debian/control: Add missing python-gi dependency to apport-gtk.
 -- Martin Pitt <email address hidden> Thu, 23 Feb 2012 16:22:37 +0100