huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix the uninitialized g_hWinHttpModule global variable to avoid empty error messages (#225)

This commit is contained in:
George Chakhidze 2017-11-06 22:16:08 +04:00 committed by Justin Kotalik
parent 483e0e9491
commit c315b27ad9
2 changed files with 12 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/AspNetCore/Src/dllmain.cpp
View File

@ -191,6 +191,7 @@ HRESULT
g_pModuleId = pModuleInfo->GetId();
g_pszModuleName = pModuleInfo->GetName();
g_pHttpServer = pHttpServer;
g_hWinHttpModule = GetModuleHandle(TEXT("winhttp.dll"));
//
// WinHTTP does not create enough threads, ask it to create more.

29
src/AspNetCore/Src/forwardinghandler.cxx
View File

@ -1451,9 +1451,8 @@ Failure:
//
pResponse->SetStatus(502, "Bad Gateway", 3, hr);
if (hr > HRESULT_FROM_WIN32(WINHTTP_ERROR_BASE) &&
hr <= HRESULT_FROM_WIN32(WINHTTP_ERROR_LAST))
{
if (!(hr > HRESULT_FROM_WIN32(WINHTTP_ERROR_BASE) &&
hr <= HRESULT_FROM_WIN32(WINHTTP_ERROR_LAST)) ||
#pragma prefast (suppress : __WARNING_FUNCTION_NEEDS_REVIEW, "Function and parameters reviewed.")
FormatMessage(
FORMAT_MESSAGE_IGNORE_INSERTS | FORMAT_MESSAGE_FROM_HMODULE,
@ -1462,9 +1461,7 @@ Failure:
0,
strDescription.QueryStr(),
strDescription.QuerySizeCCH(),
NULL);
}
else
NULL) == 0)
{
LoadString(g_hModule,
IDS_SERVER_ERROR,
@ -1774,9 +1771,8 @@ REQUEST_NOTIFICATION_STATUS
pResponse->SetStatus(502, "Bad Gateway", 3, hr);
if (hr > HRESULT_FROM_WIN32(WINHTTP_ERROR_BASE) &&
hr <= HRESULT_FROM_WIN32(WINHTTP_ERROR_LAST))
{
if (!(hr > HRESULT_FROM_WIN32(WINHTTP_ERROR_BASE) &&
hr <= HRESULT_FROM_WIN32(WINHTTP_ERROR_LAST)) ||
#pragma prefast (suppress : __WARNING_FUNCTION_NEEDS_REVIEW, "Function and parameters reviewed.")
FormatMessage(
FORMAT_MESSAGE_IGNORE_INSERTS | FORMAT_MESSAGE_FROM_HMODULE,
@ -1785,15 +1781,14 @@ REQUEST_NOTIFICATION_STATUS
0,
strDescription.QueryStr(),
strDescription.QuerySizeCCH(),
NULL);
}
else
NULL) == 0)
{
LoadString(g_hModule,
IDS_SERVER_ERROR,
strDescription.QueryStr(),
strDescription.QuerySizeCCH());
}
(VOID)strDescription.SyncWithBuffer();
if (strDescription.QueryCCH() != 0)
{
@ -2379,9 +2374,8 @@ Failure:
pResponse->SetStatus(502, "Bad Gateway", 3, hr);
if (hr > HRESULT_FROM_WIN32(WINHTTP_ERROR_BASE) &&
hr <= HRESULT_FROM_WIN32(WINHTTP_ERROR_LAST))
{
if (!(hr > HRESULT_FROM_WIN32(WINHTTP_ERROR_BASE) &&
hr <= HRESULT_FROM_WIN32(WINHTTP_ERROR_LAST)) ||
#pragma prefast (suppress : __WARNING_FUNCTION_NEEDS_REVIEW, "Function and parameters reviewed.")
FormatMessage(
FORMAT_MESSAGE_IGNORE_INSERTS | FORMAT_MESSAGE_FROM_HMODULE,
@ -2390,15 +2384,14 @@ Failure:
0,
strDescription.QueryStr(),
strDescription.QuerySizeCCH(),
NULL);
}
else
NULL) == 0)
{
LoadString(g_hModule,
IDS_SERVER_ERROR,
strDescription.QueryStr(),
strDescription.QuerySizeCCH());
}
strDescription.SyncWithBuffer();
if (strDescription.QueryCCH() != 0)
{