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

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

Browse Source
This commit is contained in:
Chris.Watts90@outlook.com 2018-03-06 14:45:27 +00:00
parent 7ec0db9ebb
commit d443ecb24c
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;
}
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;
}