Chris/FlexitimeTracker
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix memory leak issue, where we would create a duplicate cardmonitor for an "unaware" card reader state.

Browse Source
This commit is contained in:
Chris.Watts90@outlook.com 2018-03-06 14:45:27 +00:00
parent 7ec0db9ebb
commit 3eef3017a0
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
CardReaderService/CardReaderService/Service1.cs
View File

@ -37,9 +37,11 @@ namespace CardReaderService
{ {
return false; return false;
} }
if (_cardMonitor.GetCurrentState(0) == SCRState.Unknown || Console.WriteLine(_cardMonitor.GetCurrentState(0));
_cardMonitor.GetCurrentState(0) == SCRState.Unavailable || if (_cardMonitor.GetCurrentState(0) == SCRState.Unknown
_cardMonitor.GetCurrentState(0) == SCRState.Unaware) || _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; return false;
} }