s3:libsmb: inline trust_pw_change_and_store_it() into trust_pw_find_change_and_store_it()
authorStefan Metzmacher <metze@samba.org>
Mon, 2 Sep 2013 18:19:28 +0000 (20:19 +0200)
committerAndrew Bartlett <abartlet@samba.org>
Wed, 16 Oct 2013 19:48:59 +0000 (08:48 +1300)
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
source3/include/proto.h
source3/libsmb/trusts_util.c

index 805e6e446e983519561a47a9c770ed562d2f6368..497828e52940f328ea21c43b8252340c67ac99cf 100644 (file)
@@ -972,11 +972,6 @@ void update_trustdom_cache( void );
 
 /* The following definitions come from libsmb/trusts_util.c  */
 
-NTSTATUS trust_pw_change_and_store_it(struct rpc_pipe_client *cli, TALLOC_CTX *mem_ctx, 
-                                     const char *domain,
-                                     const char *account_name,
-                                     unsigned char orig_trust_passwd_hash[16],
-                                     enum netr_SchannelType sec_channel_type);
 NTSTATUS trust_pw_find_change_and_store_it(struct rpc_pipe_client *cli, 
                                           TALLOC_CTX *mem_ctx, 
                                           const char *domain) ;
index 8a0e53dbe51bcc1d58d9c64ffcfceeea68f9c79c..428e0c17cd2cb20d350df310e5b1c3cd7aea7a76 100644 (file)
 
 /*********************************************************
  Change the domain password on the PDC.
- Store the password ourselves, but use the supplied password
Caller must have already setup the connection to the NETLOGON pipe
+ Do most of the legwork ourselfs.  Caller must have
+ already setup the connection to the NETLOGON pipe
 **********************************************************/
 
-NTSTATUS trust_pw_change_and_store_it(struct rpc_pipe_client *cli, TALLOC_CTX *mem_ctx, 
-                                     const char *domain,
-                                     const char *account_name,
-                                     unsigned char orig_trust_passwd_hash[16],
-                                     enum netr_SchannelType sec_channel_type)
+NTSTATUS trust_pw_find_change_and_store_it(struct rpc_pipe_client *cli,
+                                          TALLOC_CTX *mem_ctx,
+                                          const char *domain)
 {
+       unsigned char old_trust_passwd_hash[16];
        unsigned char new_trust_passwd_hash[16];
+       enum netr_SchannelType sec_channel_type = SEC_CHAN_NULL;
+       const char *account_name;
        char *new_trust_passwd;
        NTSTATUS nt_status;
 
+       if (!get_trust_pw_hash(domain, old_trust_passwd_hash, &account_name,
+                              &sec_channel_type)) {
+               DEBUG(0, ("could not fetch domain secrets for domain %s!\n", domain));
+               return NT_STATUS_UNSUCCESSFUL;
+       }
+
        switch (sec_channel_type) {
        case SEC_CHAN_WKSTA:
        case SEC_CHAN_DOMAIN:
@@ -64,7 +71,7 @@ NTSTATUS trust_pw_change_and_store_it(struct rpc_pipe_client *cli, TALLOC_CTX *m
 
        nt_status = rpccli_netlogon_set_trust_password(cli, mem_ctx,
                                                       account_name,
-                                                      orig_trust_passwd_hash,
+                                                      old_trust_passwd_hash,
                                                       new_trust_passwd,
                                                       new_trust_passwd_hash,
                                                       sec_channel_type);
@@ -108,30 +115,3 @@ NTSTATUS trust_pw_change_and_store_it(struct rpc_pipe_client *cli, TALLOC_CTX *m
 
        return nt_status;
 }
-
-/*********************************************************
- Change the domain password on the PDC.
- Do most of the legwork ourselfs.  Caller must have
- already setup the connection to the NETLOGON pipe
-**********************************************************/
-
-NTSTATUS trust_pw_find_change_and_store_it(struct rpc_pipe_client *cli, 
-                                          TALLOC_CTX *mem_ctx, 
-                                          const char *domain) 
-{
-       unsigned char old_trust_passwd_hash[16];
-       enum netr_SchannelType sec_channel_type = SEC_CHAN_NULL;
-       const char *account_name;
-
-       if (!get_trust_pw_hash(domain, old_trust_passwd_hash, &account_name,
-                              &sec_channel_type)) {
-               DEBUG(0, ("could not fetch domain secrets for domain %s!\n", domain));
-               return NT_STATUS_UNSUCCESSFUL;
-       }
-
-       return trust_pw_change_and_store_it(cli, mem_ctx, domain,
-                                           account_name,
-                                           old_trust_passwd_hash,
-                                           sec_channel_type);
-}
-