secur32: Slightly simplify the GetUserNameExW implementation.
Dmitry Timoshkov
dmitry at baikal.ru
Mon Jan 13 04:07:31 CST 2014
---
dlls/secur32/secur32.c | 35 ++++++++++++++---------------------
1 file changed, 14 insertions(+), 21 deletions(-)
diff --git a/dlls/secur32/secur32.c b/dlls/secur32/secur32.c
index acb09cf..867fa2c 100644
--- a/dlls/secur32/secur32.c
+++ b/dlls/secur32/secur32.c
@@ -1096,16 +1096,16 @@ BOOLEAN WINAPI GetUserNameExA(
BOOLEAN WINAPI GetUserNameExW(
EXTENDED_NAME_FORMAT NameFormat, LPWSTR lpNameBuffer, PULONG nSize)
{
- BOOLEAN status;
- WCHAR samname[UNLEN + 1 + MAX_COMPUTERNAME_LENGTH + 1];
- LPWSTR out;
- DWORD len;
TRACE("(%d %p %p)\n", NameFormat, lpNameBuffer, nSize);
switch (NameFormat)
{
case NameSamCompatible:
{
+ WCHAR samname[UNLEN + 1 + MAX_COMPUTERNAME_LENGTH + 1];
+ LPWSTR out;
+ DWORD len;
+
/* This assumes the current user is always a local account */
len = MAX_COMPUTERNAME_LENGTH + 1;
if (GetComputerNameW(samname, &len))
@@ -1115,25 +1115,20 @@ BOOLEAN WINAPI GetUserNameExW(
len = UNLEN + 1;
if (GetUserNameW(out, &len))
{
- status = (lstrlenW(samname) < *nSize);
- if (status)
+ if (lstrlenW(samname) < *nSize)
{
lstrcpyW(lpNameBuffer, samname);
*nSize = lstrlenW(samname);
+ return TRUE;
}
- else
- {
- SetLastError(ERROR_MORE_DATA);
- *nSize = lstrlenW(samname) + 1;
- }
+
+ SetLastError(ERROR_MORE_DATA);
+ *nSize = lstrlenW(samname) + 1;
}
- else
- status = FALSE;
}
- else
- status = FALSE;
+ return FALSE;
}
- break;
+
case NameUnknown:
case NameFullyQualifiedDN:
case NameDisplay:
@@ -1144,14 +1139,12 @@ BOOLEAN WINAPI GetUserNameExW(
case NameServicePrincipal:
case NameDnsDomain:
SetLastError(ERROR_NONE_MAPPED);
- status = FALSE;
- break;
+ return FALSE;
+
default:
SetLastError(ERROR_INVALID_PARAMETER);
- status = FALSE;
+ return FALSE;
}
-
- return status;
}
BOOLEAN WINAPI TranslateNameA(
--
1.8.5.2
More information about the wine-patches
mailing list