Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

All authorizations fail with 0.9.2 #16

Closed
rworkman opened this issue Dec 6, 2014 · 4 comments
Closed

All authorizations fail with 0.9.2 #16

rworkman opened this issue Dec 6, 2014 · 4 comments

Comments

@rworkman
Copy link
Contributor

rworkman commented Dec 6, 2014

Basically, with CK2-0.9.2 (or from current git master), all authorizations that I checked are failing. I first noticed with suspend/hibernate/reboot/shutdown support from Xfce, but it turns out that others (e.g. nm-applet) are also lacking.

upower --dump was showing "no" for "can-suspend" and other related items, yet ck-list-sessions was showing the session both is-local=true and active=true. I spent a lot of time trying to figure it out and eventually decided to do a git bisect. The result was commit 417e387 and in fact, reverting it makes things work just as I expect (as they did with 0.4.5).

EricKoegel added a commit that referenced this issue Dec 6, 2014
This reverts commit 417e387.

Causes a regression with polkit authorizations. See:
#16
#12
@EricKoegel
Copy link
Member

Hey,
Thanks for narrowing down the issue. I was waiting for a day off to look into it but you beat me to it. I've reverted the commit you pointed out.

@mancha1
Copy link

mancha1 commented Dec 6, 2014

Nice detective work.

So, as a result of commit 417e387, the database doesn't get updated when changing to a new active session if the old session is not type CK_TYPE_SESSION - ergo the problem reported.

Unfortunately, issue 28598 is not very detailed and the OpenSolaris bug tracker is long gone.

But, emitting ACTIVE_SESSION_CHANGED_FULL with old_session not type CK_TYPE_SESSION isn't enough to crash CK so I wonder if the Sun Ray problem was due to their particular GDM configuration and/or run-session.d helper setup.

--mancha

@rworkman
Copy link
Contributor Author

Unless you're leaving this open with the intent of trying to figure out the reason for the original bug that the revert patch was intended to address, this one can be closed...

@EricKoegel
Copy link
Member

Yeah, as mancha pointed out there's not much to go on. Closing bug report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants