Minor improvements.

Author: adityamandaleeka

src/Servers/IIS/AspNetCoreModuleV2/CommonLib/debugutil.cpp

@@ -114,7 +114,7 @@ std::wstring
 GetModuleName()
 {
     WCHAR path[MAX_PATH];
-    LOG_LAST_ERROR_IF(!GetModuleFileName(g_hModule, path, MAX_PATH));
+    LOG_LAST_ERROR_IF(!GetModuleFileName(g_hModule, path, _countof(path)));
     return path;
 }
 

src/Servers/IIS/AspNetCoreModuleV2/DefaultRules.ruleset

@@ -80,6 +80,7 @@
     <Rule Id="C26475" Action="Error" />
     <Rule Id="C26476" Action="None" /> <!-- Expression/symbol 'name' uses a naked union 'union' with multiple type pointers: Use variant instead -->
     <Rule Id="C26477" Action="Error" />
+    <Rule Id="C26478" Action="Error" />
     <Rule Id="C26481" Action="None" /> <!-- Don't use pointer arithmetic. Use span instead -->
     <Rule Id="C26482" Action="None" /> <!-- Only index into arrays using constant expressions. -->
     <Rule Id="C26483" Action="Error" />

src/Servers/IIS/AspNetCoreModuleV2/IISLib/stringa.cpp

@@ -470,7 +470,7 @@ Return Value:
 
     HRESULT hr = SafeVsnprintf(pszFormatString, argsList);
 
-#pragma warning(suppress: 26477)
+#pragma warning(suppress: 26477) // va_end uses 0
     va_end( argsList );
     return hr;
 }

src/Servers/IIS/AspNetCoreModuleV2/OutOfProcessRequestHandler/winhttphelper.cpp

@@ -21,9 +21,7 @@ WINHTTP_HELPER::sm_pfnWinHttpWebSocketQueryCloseStatus;
 
 //static
 HRESULT
-WINHTTP_HELPER::StaticInitialize(
-    VOID
-)
+WINHTTP_HELPER::StaticInitialize()
 {
     //
     // Initialize the function pointers for WinHttp Websocket API's.