AuroraSupport/AuroraRuntime
2
1
Fork 0
Code Issues 27 Pull Requests Releases Wiki Activity

[*] Harden auprocesses / async outputof

Browse Source
This commit is contained in:
Reece Wilson 2023-12-29 20:08:26 +00:00
parent 7df1d1dd27
commit b3557ed938
2 changed files with 66 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

60
Source/Processes/AuOutputOf.cpp
View File

@ -600,7 +600,10 @@ namespace Aurora::Processes
if (!pResult->pTransactionB->StartRead(0, AuUnsafeRaiiToShared(&pResult->readErrView)))
{
pResult->result.bFailed = true;
pResult->pTransactionA->SetCallback({});
if (pResult->pTransactionA)
{
pResult->pTransactionA->SetCallback({});
}
return pResult;
}
@ -612,40 +615,75 @@ namespace Aurora::Processes
if (!pResult->pLoopQueue->SourceAdd(pResult->pTransactionSourceA))
{
pResult->result.bFailed = true;
pResult->pTransactionA->SetCallback({});
pResult->pTransactionB->SetCallback({});
if (pResult->pTransactionA)
{
pResult->pTransactionA->SetCallback({});
}
if (pResult->pTransactionB)
{
pResult->pTransactionB->SetCallback({});
}
pResult->CleanUp();
return pResult;
}
if (!pResult->pLoopQueue->SourceAdd(pResult->pTransactionSourceB))
{
pResult->result.bFailed = true;
pResult->pTransactionA->SetCallback({});
pResult->pTransactionB->SetCallback({});
if (pResult->pTransactionA)
{
pResult->pTransactionA->SetCallback({});
}
if (pResult->pTransactionB)
{
pResult->pTransactionB->SetCallback({});
}
pResult->CleanUp();
return pResult;
}
if (!pResult->pLoopQueue->SourceAdd(pResult->pProcessDeadLoopSource))
{
pResult->result.bFailed = true;
pResult->pTransactionA->SetCallback({});
pResult->pTransactionB->SetCallback({});
if (pResult->pTransactionA)
{
pResult->pTransactionA->SetCallback({});
}
if (pResult->pTransactionB)
{
pResult->pTransactionB->SetCallback({});
}
pResult->CleanUp();
return pResult;
}
if (!pResult->pLoopQueue->AddCallback(pResult->pProcessDeadLoopSource, pProcessDeadCallback))
{
pResult->result.bFailed = true;
pResult->pTransactionA->SetCallback({});
pResult->pTransactionB->SetCallback({});
if (pResult->pTransactionA)
{
pResult->pTransactionA->SetCallback({});
}
if (pResult->pTransactionB)
{
pResult->pTransactionB->SetCallback({});
}
pResult->CleanUp();
return pResult;
}
if (!pResult->pLoopQueue->Commit())
{
pResult->result.bFailed = true;
pResult->pTransactionA->SetCallback({});
pResult->pTransactionB->SetCallback({});
if (pResult->pTransactionA)
{
pResult->pTransactionA->SetCallback({});
}
if (pResult->pTransactionB)
{
pResult->pTransactionB->SetCallback({});
}
pResult->CleanUp();
return pResult;
}

17
Source/Processes/AuProcessElevation.NT.cpp
View File

@ -697,11 +697,13 @@ namespace Aurora::Processes
if (this->desc.username)
{
wideusername = AuLocale::ConvertFromUTF8(this->desc.username.value().c_str());
AuMemset(this->desc.username.value().data(), 0xFF, this->desc.username.value().size());
}
if (this->desc.password)
{
widepassword = AuLocale::ConvertFromUTF8(this->desc.password.value().c_str());
AuMemset(this->desc.password.value().data(), 0xFF, this->desc.password.value().size());
}
if (this->desc.server)
@ -759,6 +761,7 @@ namespace Aurora::Processes
if (wideusername.empty())
{
AuMemset(widepassword.data(), 0xFF, widepassword.size());
SysPushErrorPermissionError("Missing Username");
return false;
}
@ -791,11 +794,16 @@ namespace Aurora::Processes
if (status == EPrompt::eCancel ||
wideusername.empty())
{
AuMemset(wideusername.data(), 0xFF, wideusername.size() * sizeof(wchar_t));
AuMemset(widepassword.data(), 0xFF, widepassword.size() * sizeof(wchar_t));
return false;
}
if (status == EPrompt::eError)
{
AuMemset(wideusername.data(), 0xFF, wideusername.size() * sizeof(wchar_t));
AuMemset(widepassword.data(), 0xFF, widepassword.size() * sizeof(wchar_t));
SysPushErrorGeneric("Generic logon prompt failure");
return false;
}
@ -848,6 +856,8 @@ namespace Aurora::Processes
{}, {}, {}, {}) != ERROR_SUCCESS)
{
SysPushErrorPermissionError("Unable to elevate");
AuMemset(wideusername.data(), 0xFF, wideusername.size() * sizeof(wchar_t));
AuMemset(widepassword.data(), 0xFF, widepassword.size() * sizeof(wchar_t));
return false;
}
@ -874,6 +884,8 @@ namespace Aurora::Processes
LOGON32_PROVIDER_DEFAULT,
&hBestToken))
{
AuMemset(wideusername.data(), 0xFF, wideusername.size() * sizeof(wchar_t));
AuMemset(widepassword.data(), 0xFF, widepassword.size() * sizeof(wchar_t));
return false;
}
@ -882,6 +894,8 @@ namespace Aurora::Processes
GetTokenIntegrityLevel()))
{
SysPushErrorPermissionError("Unable to elevate");
AuMemset(wideusername.data(), 0xFF, wideusername.size() * sizeof(wchar_t));
AuMemset(widepassword.data(), 0xFF, widepassword.size() * sizeof(wchar_t));
return false;
}
}
@ -955,6 +969,9 @@ namespace Aurora::Processes
pRevertToSelf();
}
AuMemset(wideusername.data(), 0xFF, wideusername.size() * sizeof(wchar_t));
AuMemset(widepassword.data(), 0xFF, widepassword.size() * sizeof(wchar_t));
AuWin32CloseHandle(hBestToken);
if (!bResult)