fix handle memory leaks

Author: planetchili

IntelPresentMon/PresentMonService/Service.cpp

@@ -65,6 +65,13 @@ void Service::SignalClientSessionOpened()
 	SetEvent(clientSessionEvent_);
 }
 
+Service::~Service()
+{
+	if (clientSessionEvent_) {
+		CloseHandle(clientSessionEvent_);
+	}
+}
+
 ConcreteService::ConcreteService(const TCHAR* serviceName) : mServiceName(serviceName)
 {
 	mEventLogHandle = RegisterEventSource(NULL, mServiceName.c_str());

IntelPresentMon/PresentMonService/Service.h

@@ -23,7 +23,7 @@ class Service
 	virtual HANDLE GetResetPowerTelemetryHandle() = 0;
 	HANDLE GetClientSessionHandle();
 	void SignalClientSessionOpened();
-	virtual ~Service() = default;
+	virtual ~Service();
 private:
 	std::optional<int> errCode_;
 	HANDLE clientSessionEvent_;

PresentData/PresentMonTraceConsumer.cpp

@@ -156,6 +156,13 @@ PMTraceConsumer::PMTraceConsumer()
     hEventsReadyEvent = CreateEventW(nullptr, FALSE, FALSE, nullptr);
 }
 
+PMTraceConsumer::~PMTraceConsumer()
+{
+    if (hEventsReadyEvent && hEventsReadyEvent != INVALID_HANDLE_VALUE) {
+        CloseHandle(hEventsReadyEvent);
+    }
+}
+
 void PMTraceConsumer::HandleD3D9Event(EVENT_RECORD* pEventRecord)
 {
     auto const& hdr = pEventRecord->EventHeader;

PresentData/PresentMonTraceConsumer.hpp

@@ -443,6 +443,12 @@ struct PMTraceConsumer
     // Functions for decoding ETW and analysing process and present events.
 
     PMTraceConsumer();
+    ~PMTraceConsumer();
+
+    PMTraceConsumer(const PMTraceConsumer&) = delete;
+    PMTraceConsumer& operator=(const PMTraceConsumer&) = delete;
+    PMTraceConsumer(PMTraceConsumer&&) = delete;
+    PMTraceConsumer& operator=(PMTraceConsumer&&) = delete;
 
     void HandleDxgkBlt(EVENT_HEADER const& hdr, uint64_t hwnd, bool redirectedPresent);
     std::shared_ptr<PresentEvent> HandleDxgkFlip(EVENT_HEADER const& hdr);