From 3eef3017a08a53052f240b20d1435e417be699e5 Mon Sep 17 00:00:00 2001 From: "Chris.Watts90@outlook.com" Date: Tue, 6 Mar 2018 14:45:27 +0000 Subject: [PATCH] Fix memory leak issue, where we would create a duplicate cardmonitor for an "unaware" card reader state. --- CardReaderService/CardReaderService/Service1.cs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/CardReaderService/CardReaderService/Service1.cs b/CardReaderService/CardReaderService/Service1.cs index 2254187..9cd3413 100644 --- a/CardReaderService/CardReaderService/Service1.cs +++ b/CardReaderService/CardReaderService/Service1.cs @@ -37,9 +37,11 @@ namespace CardReaderService { return false; } - if (_cardMonitor.GetCurrentState(0) == SCRState.Unknown || - _cardMonitor.GetCurrentState(0) == SCRState.Unavailable || - _cardMonitor.GetCurrentState(0) == SCRState.Unaware) + Console.WriteLine(_cardMonitor.GetCurrentState(0)); + if (_cardMonitor.GetCurrentState(0) == SCRState.Unknown + || _cardMonitor.GetCurrentState(0) == SCRState.Unavailable + //|| _cardMonitor.GetCurrentState(0) == SCRState.Unaware //if we say this is an invalid state, we cause a memory leak where we create a duplicate card monitor, subscribe and overwrite. + ) { return false; }