Bug 227117

Summary:	Further attempts to authenticate fail if one closes the dialog after a wrong password
Product:	[Unmaintained] kdelibs	Reporter:	Jonathan Thomas <echidnaman>
Component:	kdecore	Assignee:	Dario Freddi <drf>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Jonathan Thomas 2010-02-16 02:59:28 UTC

Version:           0.95.1 (using KDE 4.4.0)
OS:                Linux
Installed from:    Ubuntu Packages

Techincally a polkit-kde-1 bug, but this was as close as I could find.
This is using polkit-kde-1 0.95.1 and libpolkit-qt-1 0.95.1

To reproduce:
- Try to commit an auth-needing action, such as trying to change CD drive permissions with K3b trunk
- Purposefully enter a wrong password and hit OK
- Once the Polkit dialog complains, hit cancel.
- Make a few more changes and press "Apply" again.
- The action will instafail, with polkit complaining about a cryptic "error 6" Exact text: "Unable to authenticate/execute the action: 6, "

You can even make the second set of changes within a different System Settings module, such as Date and Time, and it still won't work. Restarting System Settings is the only way to fix this. Until then, no KAuth actions will work.

Comment 1 Dario Freddi 2010-04-14 13:40:38 UTC

SVN commit 1114744 by dafre:

BUG: 227117
CCMAIL: igor@shg.ru

 - Make polkit-1 backend more safe by waiting for authentication asynchronously and through an event loop
 - Fix a bug in the DBus backend where an early error compromised action results and the list of ongoing actions.
   This fixes bug 227117.
 - Fixed some copyrights around, also

Igor, this should also fix your problem. Can you please try and let me know?

P.S.: Backporting in a minute



 M  +2 -1      BackendsManager.cpp  
 M  +1 -0      BackendsManager.h  
 M  +28 -22    backends/dbus/DBusHelperProxy.cpp  
 M  +35 -2     backends/polkit-1/Polkit1Backend.cpp  
 M  +22 -4     backends/polkit-1/Polkit1Backend.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1114744

Comment 2 Dario Freddi 2010-04-14 13:42:46 UTC

SVN commit 1114745 by dafre:

CCBUG: 227117

Backporting r1114744



 M  +2 -1      BackendsManager.cpp  
 M  +1 -0      BackendsManager.h  
 M  +28 -22    backends/dbus/DBusHelperProxy.cpp  
 M  +35 -2     backends/polkit-1/Polkit1Backend.cpp  
 M  +22 -4     backends/polkit-1/Polkit1Backend.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1114745

Comment 3 Dario Freddi 2010-04-14 13:44:22 UTC

Changing the component to the correct one - this was a bug in KAuth, so no updates to polkit-kde-1 or polkit-qt-1 are needed.