lightyears crashed with AttributeError in Set()

[Jan Girlich]

Binary package hint: lightyears

Just during playing the game building some pipes it crashed as in like stopped working and I saw my desktop again. Sometime in level 6.

ProblemType: Crash
Architecture: i386
Date: Fri Feb 22 00:57:21 2008
DistroRelease: Ubuntu 8.04
ExecutablePath: /usr/games/lightyears
InterpreterPath: /usr/bin/python2.5
Package: lightyears 1.3a-2
PackageArchitecture: all
ProcCmdline: /usr/bin/python /usr/games/lightyears
ProcCwd: /home/jan
ProcEnviron:
 LANGUAGE=de_DE:de:en_GB:en
 PATH=/home/jan/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
 LANG=de_DE.UTF-8
 SHELL=/bin/bash
PythonArgs: ['/usr/games/lightyears']
SourcePackage: lightyears
Title: lightyears crashed with AttributeError in Set()
Uname: Linux jan-x61 2.6.24-8-generic #1 SMP Thu Feb 14 20:40:45 UTC 2008 i686 GNU/Linux
UserGroups: adm admin audio cdrom dialout dip floppy lpadmin netdev plugdev powerdev sambashare scanner video

[James Westby (james-w)]

Hi,

A quick look at the code shows that the None comes from

  self.schan = self.sobj.play()

and the doc of the self.sobj.play() method says:

    /*DOC*/ "Sound.play([loops, [maxtime]]) -> Channel\n"
    /*DOC*/ "play sound\n"
    /*DOC*/ "\n"
    /*DOC*/ "Starts playing a song on an available channel. If no channels are\n"
    /*DOC*/ "available, it will not play and return None.

so I guess that is the case that was hit, and the None value
is not tested for.

Thanks,

James

[Siegfried Gevatter (rainct)]

This should be fixed in the next lightyears revision.

[Launchpad Janitor (janitor)]

This bug was fixed in the package lightyears - 1.3a-4

---------------
lightyears (1.3a-4) unstable; urgency=low

  [ Siegfried-Angel Gevatter Pujals ]
  * debian/control:
    - Update the Homepage field to point to the new location.
    - Delete the comment from the Maintainer field.
    - Bump Standards-Version to 3.8.0.
    - Add python-psyco as a Suggests.
  * debian/copyright:
    - Change the download location to the new one.
  * debian/menu:
    - Change the formatting.
  * debian/patches/only_show_valid_resolutions.patch:
    - Only show resolutions supported by the system in the menu (LP: #219512).
  * debian/patches/fix_crash_in_sound_py.patch:
    - Avoid a crash which could happen when no sound channels were
      available; thanks to James Westby (LP: #194183).

  [ Sandro Tosi ]
  * debian/watch:
    - Fixed the watch file, as the download location changed.

 -- Siegfried Gevatter (RainCT) <email address hidden> Wed, 30 Jul 2008 21:08:09 +0100