summaryrefslogtreecommitdiff
path: root/source3/passdb
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2017-05-22 12:31:01 +0200
committerStefan Metzmacher <metze@samba.org>2017-06-27 16:57:45 +0200
commit5bc2764fe517748c03a57b61f2f7ef889c92825d (patch)
tree9a70f9d53a2ee89955a023758c2fdc03b3023e60 /source3/passdb
parent5b95cb74e7b2838d228f9773c0e20982b81d1e7d (diff)
downloadsamba-5bc2764fe517748c03a57b61f2f7ef889c92825d.tar.gz
s3:secrets: rewrite secrets_delete_machine_password_ex() using helper variables
BUG: https://bugzilla.samba.org/show_bug.cgi?id=12782 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Andreas Schneider <asn@samba.org>
Diffstat (limited to 'source3/passdb')
-rw-r--r--source3/passdb/machine_account_secrets.c26
1 files changed, 22 insertions, 4 deletions
diff --git a/source3/passdb/machine_account_secrets.c b/source3/passdb/machine_account_secrets.c
index 2457ac73ab6..56a944246b8 100644
--- a/source3/passdb/machine_account_secrets.c
+++ b/source3/passdb/machine_account_secrets.c
@@ -377,16 +377,34 @@ bool secrets_fetch_trust_account_password(const char *domain, uint8_t ret_pwd[16
bool secrets_delete_machine_password_ex(const char *domain)
{
- if (!secrets_delete(machine_prev_password_keystr(domain))) {
+ const char *tmpkey = NULL;
+ bool ok;
+
+ tmpkey = machine_prev_password_keystr(domain);
+ ok = secrets_delete(tmpkey);
+ if (!ok) {
+ return false;
+ }
+
+ tmpkey = machine_password_keystr(domain);
+ ok = secrets_delete_entry(tmpkey);
+ if (!ok) {
return false;
}
- if (!secrets_delete_entry(machine_password_keystr(domain))) {
+
+ tmpkey = machine_sec_channel_type_keystr(domain);
+ ok = secrets_delete_entry(tmpkey);
+ if (!ok) {
return false;
}
- if (!secrets_delete_entry(machine_sec_channel_type_keystr(domain))) {
+
+ tmpkey = machine_last_change_time_keystr(domain);
+ ok = secrets_delete_entry(tmpkey);
+ if (!ok) {
return false;
}
- return secrets_delete_entry(machine_last_change_time_keystr(domain));
+
+ return true;
}
/************************************************************************