telepathy-butterfly crashed with ValueError in __parse()

[William Grant]

Binary package hint: telepathy-butterfly

This happens every time I receive a message from a particular contact at the moment. Said contact is using eBuddy right now.

This is an example message:

<<< MSG <redacted email> <redacted name> 160

 X-MMS-IM-Format: 'FN=Arial; EF=; CO=000000; CS=0; PF=00; RL=0;'\r\n
 Content-Type: 'text/plain; charset=UTF-8'\r\n
 MIME-Version: '1.0'\r\n
 \r\n
 <redacted plaintext message body>

ProblemType: Crash
DistroRelease: Ubuntu 10.04
Package: telepathy-butterfly 0.5.8-1ubuntu1
ProcVersionSignature: Ubuntu 2.6.32-20.30-generic-pae 2.6.32.11+drm33.2
Uname: Linux 2.6.32-20-generic-pae i686
Architecture: i386
Date: Wed Apr 14 22:57:04 2010
EcryptfsInUse: Yes
ExecutablePath: /usr/lib/telepathy/telepathy-butterfly
InstallationMedia: Error: [Errno 13] Permission denied: '/var/log/installer/media-info'
InterpreterPath: /usr/bin/python2.6
PackageArchitecture: all
ProcCmdline: /usr/bin/python /usr/lib/telepathy/telepathy-butterfly
ProcEnviron:
 SHELL=/bin/bash
 LANG=en_AU.utf8
PythonArgs: ['/usr/lib/telepathy/telepathy-butterfly']
SourcePackage: telepathy-butterfly
Title: telepathy-butterfly crashed with ValueError in __parse()
UserGroups: adm admin cdrom dialout kvm libvirtd lpadmin plugdev sambashare

[William Grant (wgrant)]

Is this due to the trailing ';' in the 'X-MMS-IM-Format'?

[Apport retracing service (apport)]

Thank you for taking the time to report this crash and helping to make Ubuntu better. This particular crash has already been reported and is a duplicate of bug #558493, so is being marked as such. Please look at the other bug report to see if there is any missing information that you can provide, or to see if there is a workaround for the bug. Additionally, any further discussion regarding the bug should occur in the other report. Please continue to report any other bugs you may find.

[William Grant (wgrant)]

Adding an emptiness check like this fixes it:

    def __parse(self, format):
        for property in format.split(';'):
            if not property:
                continue
            key, value = [p.strip(' \t|').upper() \
                    for p in property.split('=', 1)]