Skip to content

fix hw status (and USB comm in general)#434

Merged
pwpiwi merged 1 commit intoProxmark:masterfrom
pwpiwi:fix_comm
Oct 24, 2017
Merged

fix hw status (and USB comm in general)#434
pwpiwi merged 1 commit intoProxmark:masterfrom
pwpiwi:fix_comm

Conversation

@pwpiwi
Copy link
Contributor

@pwpiwi pwpiwi commented Oct 20, 2017

  • don't ignore ReadLine() errors (Windows)
  • lock CmdBuffer with Mutex
  • refactor WaitForResponseTimeoutW

* don't ignore ReadLine() errors (Windows)
* lock CmdBuffer with Mutex
* refactor WaitForResponseTimeoutW
iceman1001 added a commit to iceman1001/proxmark3 that referenced this pull request Oct 20, 2017
iceman1001 added a commit to iceman1001/proxmark3 that referenced this pull request Oct 20, 2017
@iceman1001
Copy link
Member

Great idea with more mutex when setting these global variables from different threads.
I went with the idea of _atomic for some parts.

I belive this one can be merged.

@pwpiwi pwpiwi merged commit 23af932 into Proxmark:master Oct 24, 2017
@pwpiwi pwpiwi deleted the fix_comm branch October 27, 2017 07:53
@micolous micolous mentioned this pull request Nov 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants