miroatme / proxmark3

Automatically exported from code.google.com/p/proxmark3
GNU General Public License v2.0
0 stars 0 forks source link

execute data sample will crash the client #46

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. run the proxmark3.exe
2. call data samples
3. then the client will crash

What version of the product are you using? On what operating system?

i am using the r729 on windows xp sp3

Please provide any additional information below.

I think it is the problem by the WaitForResponse(CMD_ACK, NULL) call from the 
cmdmain.c files. I have change the code to get a more safety way to resolve it. 

Original issue reported on code.google.com by gpe...@gmail.com on 3 Jun 2013 at 9:54

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks for the detailed report!
I implemented the patch (but moved it around so the usbcommand is only 
allocated if the pointer is null) and committed as r730. Could you confirm that 
it works and I'll close this ? 

Original comment by martin.holst on 3 Jun 2013 at 11:06

GoogleCodeExporter commented 9 years ago
I will test it and give u the result later.

�� 2013-6-3������7:07��proxmark3@googlecode.com ���

Best wishes,

Peica Chen ������
other mail: 
peica@qq.com

Original comment by gpe...@gmail.com on 3 Jun 2013 at 1:06

GoogleCodeExporter commented 9 years ago
OK. it is work now.

�� 2013-6-3������7:07��proxmark3@googlecode.com ���

Best wishes,

Peica Chen ������
other mail: 
peica@qq.com

Original comment by gpe...@gmail.com on 3 Jun 2013 at 1:09

GoogleCodeExporter commented 9 years ago
Thanks!

Original comment by martin.holst on 3 Jun 2013 at 1:42