Fabian Deutsch has submitted this change and it was merged.

Change subject: password: Always persist passwd and shadow
......................................................................


password: Always persist passwd and shadow

Befor eit could happen that one of the files ended up to be unpersisted,
by catching the exception and putting the persistence in the finaly branch,
we ensure that the files (and the set passwords) are now always persisted.

In the worst case it could have happened that a user can not login anymore,
because the password got unpersisted.

Change-Id: I30d579672c49bd0d07d0b8f73308b3d6238da5d3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1246117
Signed-off-by: Fabian Deutsch <[email protected]>
---
M src/ovirtnode/password.py
1 file changed, 9 insertions(+), 5 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/44138
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I30d579672c49bd0d07d0b8f73308b3d6238da5d3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <[email protected]>
Gerrit-Reviewer: Fabian Deutsch <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: [email protected]
_______________________________________________
node-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/node-patches

Reply via email to