[reactos] 01/01: [NETAPI32] Add RPC binding code to remaining functions that require explicit binding
https://git.reactos.org/?p=reactos.git;a=commitdiff;h=18a6d7c36fee20639f7a2e...
commit 18a6d7c36fee20639f7a2ea3a5ac28a39da656e9 Author: Eric Kohl eric.kohl@reactos.org AuthorDate: Sun Dec 8 12:46:35 2019 +0100 Commit: Eric Kohl eric.kohl@reactos.org CommitDate: Sun Dec 8 12:48:28 2019 +0100
[NETAPI32] Add RPC binding code to remaining functions that require explicit binding --- dll/win32/netapi32/wksta_new.c | 94 ++++++++++++++++++++++++++++++------------ 1 file changed, 67 insertions(+), 27 deletions(-)
diff --git a/dll/win32/netapi32/wksta_new.c b/dll/win32/netapi32/wksta_new.c index f3516e9f5e5..a8eb1da2c84 100644 --- a/dll/win32/netapi32/wksta_new.c +++ b/dll/win32/netapi32/wksta_new.c @@ -211,8 +211,8 @@ NetAddAlternateComputerName( _In_opt_ LPCWSTR DomainAccountPassword, _In_ ULONG Reserved) { - PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword; - handle_t BindingHandle; + PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword = NULL; + handle_t BindingHandle = NULL; NET_API_STATUS status;
TRACE("NetAddAlternateComputerName(%s %s %s %s 0x%lx)\n", @@ -220,8 +220,14 @@ NetAddAlternateComputerName( debugstr_w(DomainAccountPassword), Reserved);
/* FIXME */ - BindingHandle = NULL; - EncryptedPassword = NULL; + + status = NetpBind(Server, + &BindingHandle); + if (status != NERR_Success) + { + ERR("NetpBind() failed (status 0x%lx)\n", status); + return status; + }
RpcTryExcept { @@ -238,6 +244,8 @@ NetAddAlternateComputerName( } RpcEndExcept;
+ NetpUnbind(BindingHandle); + return status; }
@@ -340,15 +348,15 @@ NetGetJoinInformation( NET_API_STATUS WINAPI NetGetJoinableOUs( - _In_ LPCWSTR lpServer, + _In_opt_ LPCWSTR lpServer, _In_ LPCWSTR lpDomain, - _In_ LPCWSTR lpAccount, - _In_ LPCWSTR lpPassword, + _In_opt_ LPCWSTR lpAccount, + _In_opt_ LPCWSTR lpPassword, _Out_ DWORD *OUCount, _Out_ LPWSTR **OUs) { - PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword; - handle_t BindingHandle; + PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword = NULL; + handle_t BindingHandle = NULL; NET_API_STATUS status;
TRACE("NetGetJoinableOUs(%s %s %s %s %p %p)\n", @@ -356,8 +364,14 @@ NetGetJoinableOUs( debugstr_w(lpPassword), OUCount, OUs);
/* FIXME */ - BindingHandle = NULL; - EncryptedPassword = NULL; + + status = NetpBind(lpServer, + &BindingHandle); + if (status != NERR_Success) + { + ERR("NetpBind() failed (status 0x%lx)\n", status); + return status; + }
RpcTryExcept { @@ -375,6 +389,8 @@ NetGetJoinableOUs( } RpcEndExcept;
+ NetpUnbind(BindingHandle); + return status; }
@@ -440,8 +456,8 @@ NetRemoveAlternateComputerName( _In_opt_ LPCWSTR DomainAccountPassword, _In_ ULONG Reserved) { - PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword; - handle_t BindingHandle; + PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword = NULL; + handle_t BindingHandle = NULL; NET_API_STATUS status;
TRACE("NetRemoveAlternateComputerName(%s %s %s %s 0x%lx)\n", @@ -449,8 +465,14 @@ NetRemoveAlternateComputerName( debugstr_w(DomainAccountPassword), Reserved);
/* FIXME */ - BindingHandle = NULL; - EncryptedPassword = NULL; + + status = NetpBind(Server, + &BindingHandle); + if (status != NERR_Success) + { + ERR("NetpBind() failed (status 0x%lx)\n", status); + return status; + }
RpcTryExcept { @@ -467,6 +489,8 @@ NetRemoveAlternateComputerName( } RpcEndExcept;
+ NetpUnbind(BindingHandle); + return status; }
@@ -474,14 +498,14 @@ NetRemoveAlternateComputerName( NET_API_STATUS WINAPI NetRenameMachineInDomain( - _In_ LPCWSTR lpServer, - _In_ LPCWSTR lpNewMachineName, - _In_ LPCWSTR lpAccount, - _In_ LPCWSTR lpPassword, + _In_opt_ LPCWSTR lpServer, + _In_opt_ LPCWSTR lpNewMachineName, + _In_opt_ LPCWSTR lpAccount, + _In_opt_ LPCWSTR lpPassword, _In_ DWORD fRenameOptions) { - PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword; - handle_t BindingHandle; + PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword = NULL; + handle_t BindingHandle = NULL; NET_API_STATUS status;
TRACE("NetRenameMachineInDomain(%s %s %s %s 0x%lx)\n", @@ -489,8 +513,14 @@ NetRenameMachineInDomain( debugstr_w(lpPassword), fRenameOptions);
/* FIXME */ - BindingHandle = NULL; - EncryptedPassword = NULL; + + status = NetpBind(lpServer, + &BindingHandle); + if (status != NERR_Success) + { + ERR("NetpBind() failed (status 0x%lx)\n", status); + return status; + }
RpcTryExcept { @@ -507,6 +537,8 @@ NetRenameMachineInDomain( } RpcEndExcept;
+ NetpUnbind(BindingHandle); + return status; }
@@ -520,8 +552,8 @@ NetSetPrimaryComputerName( _In_opt_ LPCWSTR DomainAccountPassword, _In_ ULONG Reserved) { - PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword; - handle_t BindingHandle; + PJOINPR_ENCRYPTED_USER_PASSWORD EncryptedPassword = NULL; + handle_t BindingHandle = NULL; NET_API_STATUS status;
TRACE("NetSetPrimaryComputerName(%s %s %s %s %lu)\n", @@ -529,8 +561,14 @@ NetSetPrimaryComputerName( debugstr_w(DomainAccountPassword), Reserved);
/* FIXME */ - BindingHandle = NULL; - EncryptedPassword = NULL; + + status = NetpBind(Server, + &BindingHandle); + if (status != NERR_Success) + { + ERR("NetpBind() failed (status 0x%lx)\n", status); + return status; + }
RpcTryExcept { @@ -547,6 +585,8 @@ NetSetPrimaryComputerName( } RpcEndExcept;
+ NetpUnbind(BindingHandle); + return status; }
-
Eric Kohl