jockey-kde focus+crash (+FIX @Kubuntu 10.04 beta1)

Bug #552723 reported by ThorbjørnTux
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
jockey (Ubuntu)
Fix Released
Medium
Unassigned
Nominated for Lucid by ThorbjørnTux

Bug Description

Binary package hint: jockey-kde

2 bugs
1) If I close with X in upper right corner - jockey-kde will crash
2) The first item is selected but not set current. Activate will then 'lock up' the dialog.

I might later find some duplicates on theses issues.

Related branches

Revision history for this message
ThorbjørnTux (martsummsw) wrote :
Revision history for this message
ThorbjørnTux (martsummsw) wrote :
Revision history for this message
ThorbjørnTux (martsummsw) wrote :
Revision history for this message
ThorbjørnTux (martsummsw) wrote :

Would some with interest in KDE please answer here.

Revision history for this message
ThorbjørnTux (martsummsw) wrote :

It seems that this bug is related to the crash mentioned
https://bugs.launchpad.net/jockey/+bug/398912
(We are in the dtor of QMenu when the crash happens)

Revision history for this message
ThorbjørnTux (martsummsw) wrote :

Added the authors of jockey-kde to subscribers.
I hope this fix will reach the final Kubuntu 10.04.

Revision history for this message
Jonathan Thomas (echidnaman) wrote :

Could you post a patch based off jockey trunk please? https://code.edge.launchpad.net/~jockey-hackers/jockey/trunk
The file you posted is quite outdated, and I can't seem to find your changes out of all the changes in the past few months.

Revision history for this message
ThorbjørnTux (martsummsw) wrote :

I can see that one of the problems was fixed in
582.1.2 self.mw.treeview_drivers.setCurrentItem(cur_item)
(It was just select before)

My other fix is just a connect
(self.mw.connect(self.mw,SIGNAL('rejected()'),self.on_buttonBox_rejected)
since we do not want to have a crash if the user tries to close by clicking X (in the right corner)

I am going to post a patchfile against the trunk in two seconds.

Revision history for this message
ThorbjørnTux (martsummsw) wrote :
Revision history for this message
Jonathan Thomas (echidnaman) wrote :

Committed to trunk, thanks. Waiting on a core-dev to backport the fix with the next bugfix update.

Changed in jockey (Ubuntu):
importance: Undecided → Medium
milestone: none → ubuntu-10.04
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package jockey - 0.5.8-0ubuntu7

---------------
jockey (0.5.8-0ubuntu7) lucid; urgency=low

  [ Alberto Milone ]
  * Do not check that the module is loaded in enabled() in the nvidia
    handler, as this is what we should do in used(). Just make sure
    that the module and its alias are not blacklisted. This prevents
    Jockey from claiming that the installation of nvidia driver failed
    when when it didn't (LP: #552653).

  [ Jonathan Thomas ]
  * Don't crash when closed with the "x" button (LP: #552723, LP: #398912).
    Thanks ThorbjørnTux!

  [ Martin Pitt ]
  * i18n "Install Drivers" string. (LP: #542552)
 -- Alberto Milone <email address hidden> Mon, 12 Apr 2010 10:13:16 +0200

Changed in jockey (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.