Use HandleWrapper and initialize variables for FileOutputManager (#986)

src/AspNetCoreModuleV2/CommonLib/CommonLib.vcxproj

@@ -216,6 +216,7 @@
     <ClCompile Include="debugutil.cpp" />
     <ClCompile Include="fx_ver.cxx" />
     <ClCompile Include="GlobalVersionUtility.cpp" />
+    <ClCompile Include="HandleWrapper.cpp" />
     <ClCompile Include="hostfxr_utility.cpp" />
     <ClCompile Include="hostfxroptions.cpp" />
     <ClCompile Include="SRWExclusiveLock.cpp" />

src/AspNetCoreModuleV2/CommonLib/HandleWrapper.cpp

@@ -0,0 +1,7 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+#include "HandleWrapper.h"
+
+// Workaround for VC++ bug https://developercommunity.visualstudio.com/content/problem/33928/constexpr-failing-on-nullptr-v141-compiler-regress.html
+const HANDLE InvalidHandleTraits::DefaultHandle = INVALID_HANDLE_VALUE;

src/AspNetCoreModuleV2/CommonLib/HandleWrapper.h

@@ -12,9 +12,6 @@ struct InvalidHandleTraits
     static void Close(HANDLE handle) { CloseHandle(handle); }
 };
 
-// Workaround for VC++ bug https://developercommunity.visualstudio.com/content/problem/33928/constexpr-failing-on-nullptr-v141-compiler-regress.html
-const HANDLE InvalidHandleTraits::DefaultHandle = INVALID_HANDLE_VALUE;
-
 struct NullHandleTraits
 {
     using HandleType = HANDLE;

src/AspNetCoreModuleV2/InProcessRequestHandler/inprocessapplication.cpp

@@ -11,6 +11,7 @@
 #include "EventLog.h"
 #include "SRWExclusiveLock.h"
 #include "exceptions.h"
+#include "LoggingHelpers.h"
 
 IN_PROCESS_APPLICATION*  IN_PROCESS_APPLICATION::s_Application = NULL;
 

src/AspNetCoreModuleV2/InProcessRequestHandler/inprocessapplication.h

@@ -7,7 +7,7 @@
 #include "InProcessApplicationBase.h"
 #include "inprocesshandler.h"
 #include "requesthandler_config.h"
-
+#include "IOutputManager.h"
 
 typedef REQUEST_NOTIFICATION_STATUS(WINAPI * PFN_REQUEST_HANDLER) (IN_PROCESS_HANDLER* pInProcessHandler, void* pvRequestHandlerContext);
 typedef BOOL(WINAPI * PFN_SHUTDOWN_HANDLER) (void* pvShutdownHandlerContext);

src/AspNetCoreModuleV2/InProcessRequestHandler/inprocesshandler.cpp

@@ -4,6 +4,7 @@
 #include "inprocesshandler.h"
 #include "inprocessapplication.h"
 #include "aspnetcore_event.h"
+#include "IOutputManager.h"
 
 ALLOC_CACHE_HANDLER * IN_PROCESS_HANDLER::sm_pAlloc = NULL;
 

src/AspNetCoreModuleV2/RequestHandlerLib/FileOutputManager.cpp

@@ -2,12 +2,16 @@
 // Licensed under the MIT License. See License.txt in the project root for license information.
 
 #include "stdafx.h"
+#include "FileOutputManager.h"
 #include "sttimer.h"
 #include "utility.h"
 #include "exceptions.h"
 #include "debugutil.h"
 
-FileOutputManager::FileOutputManager()
+FileOutputManager::FileOutputManager() :
+    m_hLogFileHandle(INVALID_HANDLE_VALUE),
+    m_fdPreviousStdOut(0),
+    m_fdPreviousStdErr(0)
 {
 }
 
@@ -19,8 +23,6 @@ FileOutputManager::~FileOutputManager()
     if (m_hLogFileHandle != INVALID_HANDLE_VALUE)
     {
         m_Timer.CancelTimer();
-        CloseHandle(m_hLogFileHandle);
-        m_hLogFileHandle = INVALID_HANDLE_VALUE;
     }
 
     // delete empty log file
@@ -122,6 +124,7 @@ FileOutputManager::Start()
     RETURN_IF_FAILED(UTILITY::EnsureDirectoryPathExist(struPath.QueryStr()));
 
     GetSystemTime(&systemTime);
+
     RETURN_IF_FAILED(
         m_struLogFilePath.SafeSnwprintf(L"%s_%d%02d%02d%02d%02d%02d_%d.log",
         struPath.QueryStr(),

src/AspNetCoreModuleV2/RequestHandlerLib/FileOutputManager.h

@@ -4,6 +4,8 @@
 #pragma once
 
 #include "sttimer.h"
+#include "IOutputManager.h"
+#include "HandleWrapper.h"
 
 class FileOutputManager : public IOutputManager
 {
@@ -20,7 +22,7 @@ public:
     virtual void NotifyStartupComplete() override;
 
 private:
-    HANDLE m_hLogFileHandle;
+    HandleWrapper<InvalidHandleTraits> m_hLogFileHandle;
     STTIMER m_Timer;
     STRU m_wsStdOutLogFileName;
     STRU m_wsApplicationPath;

src/AspNetCoreModuleV2/RequestHandlerLib/LoggingHelpers.cpp

@@ -2,6 +2,11 @@
 // Licensed under the MIT License. See License.txt in the project root for license information.
 
 #include "stdafx.h"
+#include "LoggingHelpers.h"
+#include "IOutputManager.h"
+#include "FileOutputManager.h"
+#include "PipeOutputManager.h"
+#include "NullOutputManager.h"
 
 HRESULT
 LoggingHelpers::CreateLoggingProvider(

src/AspNetCoreModuleV2/RequestHandlerLib/LoggingHelpers.h

@@ -3,6 +3,8 @@
 
 #pragma once
 
+#include "IOutputManager.h"
+
 class LoggingHelpers
 {
 public:

src/AspNetCoreModuleV2/RequestHandlerLib/PipeOutputManager.cpp

@@ -2,6 +2,7 @@
 // Licensed under the MIT License. See License.txt in the project root for license information.
 
 #include "stdafx.h"
+#include "PipeOutputManager.h"
 #include "exceptions.h"
 #include "SRWExclusiveLock.h"
 

src/AspNetCoreModuleV2/RequestHandlerLib/PipeOutputManager.h

@@ -3,6 +3,8 @@
 
 #pragma once
 
+#include "IOutputManager.h"
+
 class PipeOutputManager : public IOutputManager
 {
     #define PIPE_OUTPUT_THREAD_TIMEOUT 2000

src/AspNetCoreModuleV2/RequestHandlerLib/requesthandler_config.cpp

@@ -4,6 +4,7 @@
 #include "stdafx.h"
 #include "requesthandler_config.h"
 #include "debugutil.h"
+#include "environmentvariablehash.h"
 
 REQUESTHANDLER_CONFIG::~REQUESTHANDLER_CONFIG()
 {

src/AspNetCoreModuleV2/RequestHandlerLib/requesthandler_config.h

@@ -42,6 +42,7 @@
 //#define HEX_TO_ASCII(c) ((CHAR)(((c) < 10) ? ((c) + '0') : ((c) + 'a' - 10)))
 
 #include "stdafx.h"
+#include "environmentvariablehash.h"
 
 enum APP_HOSTING_MODEL
 {

src/AspNetCoreModuleV2/RequestHandlerLib/stdafx.h

@@ -16,6 +16,7 @@
 
 #include "Shlwapi.h"
 #include <io.h>
+
 #include "hashtable.h"
 #include "stringu.h"
 #include "stringa.h"
@@ -23,10 +24,5 @@
 #include "dbgutil.h"
 #include "ahutil.h"
 #include "hashfn.h"
-#include "environmentvariablehash.h"
+
-#include "IOutputManager.h"
-#include "FileOutputManager.h"
-#include "PipeOutputManager.h"
-#include "NullOutputManager.h"
-#include "LoggingHelpers.h"
 

test/CommonLibTests/FileOutputManagerTests.cpp

@@ -3,6 +3,7 @@
 
 #include "stdafx.h"
 #include "gtest/internal/gtest-port.h"
+#include "FileOutputManager.h"
 
 class FileManagerWrapper
 {

test/CommonLibTests/PipeOutputManagerTests.cpp

@@ -3,6 +3,7 @@
 
 #include "stdafx.h"
 #include "gtest/internal/gtest-port.h"
+#include "PipeOutputManager.h"
 
 class FileManagerWrapper
 {

test/CommonLibTests/stdafx.h

@@ -58,5 +58,3 @@
 #include "gtest/gtest.h"
 #include "fakeclasses.h"
 
-\
-