Address code analysis warnings

Signed-off-by: Simon Rozman <simon@rozman.si>
This commit is contained in:
Simon Rozman 2019-09-04 13:02:22 +02:00
parent fa1514e9b3
commit 03522c0047

View File

@ -1904,8 +1904,8 @@ inline _Success_(return != 0) BOOL LookupAccountSidA(_In_opt_z_ LPCSTR lpSystemN
DWORD dwNameLen = 0, dwRefDomainLen = 0; DWORD dwNameLen = 0, dwRefDomainLen = 0;
if (LookupAccountSidA(lpSystemName, lpSid, if (LookupAccountSidA(lpSystemName, lpSid,
NULL, sName ? &dwNameLen : NULL, NULL, &dwNameLen ,
NULL, sReferencedDomainName ? &dwRefDomainLen : NULL, NULL, &dwRefDomainLen,
peUse)) peUse))
{ {
// Name and domain is blank. // 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[]> bufName (new _Elem[dwNameLen ]);
std::unique_ptr<_Elem[]> bufRefDomain(new _Elem[dwRefDomainLen]); std::unique_ptr<_Elem[]> bufRefDomain(new _Elem[dwRefDomainLen]);
if (LookupAccountSidA(lpSystemName, lpSid, if (LookupAccountSidA(lpSystemName, lpSid,
bufName .get(), sName ? &dwNameLen : NULL, bufName .get(), &dwNameLen ,
bufRefDomain.get(), sReferencedDomainName ? &dwRefDomainLen : NULL, bufRefDomain.get(), &dwRefDomainLen,
peUse)) peUse))
{ {
if (sName ) sName ->assign(bufName .get(), dwNameLen - 1); 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; DWORD dwNameLen = 0, dwRefDomainLen = 0;
if (LookupAccountSidW(lpSystemName, lpSid, if (LookupAccountSidW(lpSystemName, lpSid,
NULL, sName ? &dwNameLen : NULL, NULL, &dwNameLen ,
NULL, sReferencedDomainName ? &dwRefDomainLen : NULL, NULL, &dwRefDomainLen,
peUse)) peUse))
{ {
// Name and domain is blank. // 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[]> bufName (new _Elem[dwNameLen ]);
std::unique_ptr<_Elem[]> bufRefDomain(new _Elem[dwRefDomainLen]); std::unique_ptr<_Elem[]> bufRefDomain(new _Elem[dwRefDomainLen]);
if (LookupAccountSidW(lpSystemName, lpSid, if (LookupAccountSidW(lpSystemName, lpSid,
bufName .get(), sName ? &dwNameLen : NULL, bufName .get(), &dwNameLen ,
bufRefDomain.get(), sReferencedDomainName ? &dwRefDomainLen : NULL, bufRefDomain.get(), &dwRefDomainLen,
peUse)) peUse))
{ {
if (sName ) sName ->assign(bufName .get(), dwNameLen - 1); if (sName ) sName ->assign(bufName .get(), dwNameLen - 1);