Merge lp://staging/~3v1n0/unity/safer-pam-handle into lp://staging/unity

Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Andrea Azzarone
Approved revision: no longer in the source branch.
Merged at revision: 4172
Proposed branch: lp://staging/~3v1n0/unity/safer-pam-handle
Merge into: lp://staging/unity
Diff against target: 22 lines (+3/-2)
1 file modified
lockscreen/UserAuthenticatorPam.cpp (+3/-2)
To merge this branch: bzr merge lp://staging/~3v1n0/unity/safer-pam-handle
Reviewer Review Type Date Requested Status
Andrea Azzarone (community) Approve
Review via email: mp+302518@code.staging.launchpad.net

Commit message

UserAuthenticatorPam: ensure pam_handle_ is null initialized and don't proceed if not set

To post a comment you must log in.
Revision history for this message
Andrea Azzarone (azzar1) wrote :

I'm not completely sure this is going to fix the problem described in bug #1611668. But this should not harm so +1 for me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.