Ubuntu
screen-resolution-extra package

policyui.py crashed with TypeError in setVirtual()

Bug #262906 reported by Andreas Schildbach on 2008-08-30

Affects		Status	Importance	Assigned to	Milestone
	screen-resolution-extra (Ubuntu)	Fix Released	High	Unassigned
Nominated for Intrepid by Daniel Sargeant

Bug Description

Binary package hint: screen-resolution-extra

I was trying out the new automatic setting of Virtual. I opened the screen resolution applet, unchecked "mirrored" and put the two monitor aside of each other. The bug occured after I pressed "apply" and said "yes" to the confirmation dialog.

ProblemType: Crash
Architecture: i386
DistroRelease: Ubuntu 8.10
ExecutablePath: /usr/share/screen-resolution-extra/policyui.py
InterpreterPath: /usr/bin/python2.5
Package: screen-resolution-extra 0.1
PackageArchitecture: all
ProcAttrCurrent: unconfined
ProcCmdline: python /usr/share/screen-resolution-extra/policyui.py 1280,0:1280x1024 0,0:1280x768
ProcEnviron:
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
LANG=en_US.UTF-8
SHELL=/bin/bash
PythonArgs: ['/usr/share/screen-resolution-extra/policyui.py', '1280,0:1280x1024', '0,0:1280x768']
SourcePackage: screen-resolution-extra
Title: policyui.py crashed with TypeError in setVirtual()
Uname: Linux 2.6.27-2-generic i686
UserGroups: adm admin audio cdrom dialout fuse lpadmin plugdev sambashare

Tags:

Related branches

lp:~james-w/screen-resolution-extra/intrepid

lp:ubuntu/karmic/screen-resolution-extra

Revision history for this message

Andreas Schildbach (schildbach) wrote on 2008-08-30:

original xorg.conf Edit (1.2 KiB, text/plain)
Dependencies.txt Edit (880 bytes, text/plain; charset="utf-8")
ProcMaps.txt Edit (24.9 KiB, text/plain; charset="utf-8")
ProcStatus.txt Edit (777 bytes, text/plain; charset="utf-8")
Traceback.txt Edit (932 bytes, text/plain; charset="utf-8")

Revision history for this message

Thomas Novin (thomasn80) wrote on 2008-09-04:

_usr_share_screen-resolution-extra_policyui.py.1000.crash Edit (30.1 KiB, text/plain)

+1, had the same crash. Adding my .crash-file as attachment.

Revision history for this message

Thomas Novin (thomasn80) wrote on 2008-09-04:

Running it from command line (looked at a ps for details).

$ python /usr/share/screen-resolution-extra/policyui.py -1,-1:-1x-1 0,0:1680x1050 1680,0:1024x768
Traceback (most recent call last):
  File "/usr/share/screen-resolution-extra/policyui.py", line 190, in on_button1_clicked
    status = self.conf.setVirtual(self.resolution)
  File "/var/lib/python-support/python2.5/dbus/proxies.py", line 68, in __call__
    return self._proxy_method(*args, **keywords)
  File "/var/lib/python-support/python2.5/dbus/proxies.py", line 140, in __call__
    **keywords)
  File "/var/lib/python-support/python2.5/dbus/connection.py", line 607, in call_blocking
    message, timeout)
dbus.exceptions.DBusException: org.freedesktop.DBus.Python.TypeError: Traceback (most recent call last):
  File "/var/lib/python-support/python2.5/dbus/service.py", line 696, in _message_cb
    retval = candidate_method(self, *args, **keywords)
  File "/usr/share/screen-resolution-extra/screenresolution-mechanism.py", line 259, in setVirtual
    backup = source + time.time()
TypeError: cannot concatenate 'str' and 'float' objects

Revision history for this message

lilbudda (cory-meisch-gmail) wrote on 2008-09-08:

confirmed. Happened to me 9/8/08. Clicked on the monitor applet, changed the second screen resolution and hit apply. It prompted to change the virtual settings. After giving password, it crashed.

screen-resolution-extra:
  Installed: 0.1
  Candidate: 0.1
  Version table:
*** 0.1 0
        500 http://us.archive.ubuntu.com intrepid/main Packages
        100 /var/lib/dpkg/status

ran in console and got same output as above.

Revision history for this message

Daniel Sargeant (dsargeant) wrote on 2008-09-09:

screenresolution-mechanism.py.diff Edit (439 bytes, text/plain)

I fixed it on my machine by using time.time().__str__() instead of time.time() in screen-resolution-mechanism.py (line 259). I've attached the patch.

Changed in screen-resolution-extra:
status:	New → Confirmed

Revision history for this message

James Westby (james-w) wrote on 2008-09-12:

Hi,

This is fixed upstream already, and should be in the next update.

Thanks,

James

Changed in screen-resolution-extra:
importance:	Undecided → High
status:	Confirmed → Triaged

Revision history for this message

Launchpad Janitor (janitor) wrote on 2008-09-12:

This bug was fixed in the package screen-resolution-extra - 0.1.1

---------------
screen-resolution-extra (0.1.1) intrepid; urgency=low

  * Don't try and concatenate a string and a float (LP: #262906)
    - Also use strftime() to give a more understandable date suffix to
      the backup name.

-- James Westby <email address hidden> Fri, 12 Sep 2008 19:31:54 +0100

Changed in screen-resolution-extra:
status:	Triaged → Fix Released

Revision history for this message

Andreas Schildbach (schildbach) wrote on 2008-09-12:

Fixed for me. Thanks!

Revision history for this message

Typhoe (spamistrash) wrote on 2008-09-12:

Sorry, problem still here for me with v0.1.1

I still get a crash when trying to set virtual desktop

Typhoe

Revision history for this message

James Westby (james-w) wrote on 2008-09-12: Re: [Bug 262906] Re: policyui.py crashed with TypeError in setVirtual()

#10

On Fri, 2008-09-12 at 22:31 +0000, Typhoe wrote:
> Sorry, problem still here for me with v0.1.1
>
> I still get a crash when trying to set virtual desktop

Hi,

This bug has been confirmed fixed for other people, so you may
have a different issue. Could you please let apport report a new
bug for you, and then we will work on that issue?

Thanks,

James

Revision history for this message

Typhoe (spamistrash) wrote on 2008-09-13:

#11

Hi,
new LP report #269757
https://bugs.launchpad.net/ubuntu/+source/screen-resolution-extra/+bug/269757

Thanks too