From 03522c004709f2e54eda4b10f0ef8321b682f28c Mon Sep 17 00:00:00 2001 From: Simon Rozman Date: Wed, 4 Sep 2019 13:02:22 +0200 Subject: [PATCH] Address code analysis warnings Signed-off-by: Simon Rozman --- include/WinStd/Win.h | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/include/WinStd/Win.h b/include/WinStd/Win.h index ddf06ada..abd615d3 100644 --- a/include/WinStd/Win.h +++ b/include/WinStd/Win.h @@ -1904,8 +1904,8 @@ inline _Success_(return != 0) BOOL LookupAccountSidA(_In_opt_z_ LPCSTR lpSystemN DWORD dwNameLen = 0, dwRefDomainLen = 0; if (LookupAccountSidA(lpSystemName, lpSid, - NULL, sName ? &dwNameLen : NULL, - NULL, sReferencedDomainName ? &dwRefDomainLen : NULL, + NULL, &dwNameLen , + NULL, &dwRefDomainLen, peUse)) { // Name and domain is blank. @@ -1917,8 +1917,8 @@ inline _Success_(return != 0) BOOL LookupAccountSidA(_In_opt_z_ LPCSTR lpSystemN std::unique_ptr<_Elem[]> bufName (new _Elem[dwNameLen ]); std::unique_ptr<_Elem[]> bufRefDomain(new _Elem[dwRefDomainLen]); if (LookupAccountSidA(lpSystemName, lpSid, - bufName .get(), sName ? &dwNameLen : NULL, - bufRefDomain.get(), sReferencedDomainName ? &dwRefDomainLen : NULL, + bufName .get(), &dwNameLen , + bufRefDomain.get(), &dwRefDomainLen, peUse)) { if (sName ) sName ->assign(bufName .get(), dwNameLen - 1); @@ -1939,8 +1939,8 @@ inline _Success_(return != 0) BOOL LookupAccountSidW(_In_opt_z_ LPCWSTR lpSystem DWORD dwNameLen = 0, dwRefDomainLen = 0; if (LookupAccountSidW(lpSystemName, lpSid, - NULL, sName ? &dwNameLen : NULL, - NULL, sReferencedDomainName ? &dwRefDomainLen : NULL, + NULL, &dwNameLen , + NULL, &dwRefDomainLen, peUse)) { // Name and domain is blank. @@ -1952,8 +1952,8 @@ inline _Success_(return != 0) BOOL LookupAccountSidW(_In_opt_z_ LPCWSTR lpSystem std::unique_ptr<_Elem[]> bufName (new _Elem[dwNameLen ]); std::unique_ptr<_Elem[]> bufRefDomain(new _Elem[dwRefDomainLen]); if (LookupAccountSidW(lpSystemName, lpSid, - bufName .get(), sName ? &dwNameLen : NULL, - bufRefDomain.get(), sReferencedDomainName ? &dwRefDomainLen : NULL, + bufName .get(), &dwNameLen , + bufRefDomain.get(), &dwRefDomainLen, peUse)) { if (sName ) sName ->assign(bufName .get(), dwNameLen - 1);