lp:~aacid/unity8/removeUnusedHasAttributes
- Get this branch:
- bzr branch lp:~aacid/unity8/removeUnusedHasAttributes
Branch merges
- Ying-Chun Liu (community): Approve
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
-
Diff: 28 lines (+5/-8)1 file modifiedqml/Dash/CardTool.qml (+5/-8)
Branch information
Recent revisions
- 1196. By Mirco Müller
-
Made notification qml-test pass again by using Component.
onCompleted instead of onOpacityChanged for the time being.
Approved by: Michał Sawicz, PS Jenkins bot - 1195. By Albert Astals Cid
-
PreviewExpandable: "widgets" is a model, not an array
Approved by: Andrea Cimitan, Pawel Stolowski, Michał Sawicz - 1194. By Gerry Boland
-
Cancel open PAM interactions on shutdown - fixes hang on logout on desktop Fixes: 1353041
Approved by: Michał Sawicz, Michael Terry, Christopher Townsend, PS Jenkins bot - 1192. By Michael Terry
-
Allow logging into a desktop or tablet session again, by properly dismissing old PAM conversations. (LP: #1350878)
In a desktop or tablet, we were accidentally starting two PAM conversations in sequence on startup. Which is a small bug; it shouldn't normally be a problem, since each new PAM conversation should kill the old one.
But the way we were killing the old one was subject to a thread race condition. See, a PAM conversation thread won't exit until all its prompts are answered. And what we do when we kill a PAM conversation is to answer all prompts with empty strings.
But it's possible that when we want to kill a PAM conversation that it hasn't actually gotten to the point of prompting us yet. And when those prompts do come through, we were treating them as prompts for the new PAM conversation.
So I've changed the PAM conversation logic to include a pam_handle and compare the handle with the current handle when being prompted. If it's an old handle, we just dismiss the prompt with an empty string response.
Oh, and I fixed the bug that caused two prompts on startup in the first place. (But we still need the above logic anyway, for when you switch users quickly.) Fixes: 1350878
Approved by: Michał Sawicz, Stephen M. Webb, PS Jenkins bot
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:unity8