endless loop after "hf search" ...

thunder0666 commented 5 years ago

proxmark not responding after a hf search command the leds "A and C" ligth up.

for reproduce promark3 -c "hf search"

pm3 -c "hf search"
[=] Session log /home/vincent/.proxmark3/log_20190912.txt
[+] execute command from commandline: hf search

[=] Using UART port /dev/ttyACM0
[=] Communicating with PM3 over USB-CDC
pm3 --> hf search
[=] Checking for known tags...

[/] Searching for iClass / PicoPass tag...[=] You can cancel this operation by pressing the pm3 button

[-] No known/supported 13.56 MHz tags found

and the proxmark goes to endless loop ( it's comming hot :) ) the only way to restart it's to disconnect the device. ( no tag is normal i try with nothing )

HW and SOFT Versions

pm3 -c "hw version"
[=] Session log /home/vincent/.proxmark3/log_20190912.txt
[+] execute command from commandline: hw version

[=] Using UART port /dev/ttyACM0
[=] Communicating with PM3 over USB-CDC
pm3 --> hw version

 [ Proxmark3 RFID instrument ]

 [ CLIENT ]
  client: RRG/Iceman
  compiled with GCC 8.3.0 OS:Linux ARCH:x86_64

 [ PROXMARK RDV4 ]
  external flash:                  present
  smartcard reader:                present

 [ PROXMARK RDV4 Extras ]
  FPC USART for BT add-on support: absent

 [ ARM ]
  bootrom: RRG/Iceman/master/aca164da 2019-09-12 16:17:53
       os: RRG/Iceman/master/aca164da 2019-09-12 16:17:57
  compiled with GCC 7.3.1 20180622 (release) [ARM/embedded-7-branch revision 261907]

 [ FPGA ]
  LF image built for 2s30vq100 on 2019-07-31 at 15:57:16
  HF image built for 2s30vq100 on 2018-09-03 at 21:40:23

 [ Hardware ]
  --= uC: AT91SAM7S512 Rev A
  --= Embedded Processor: ARM7TDMI
  --= Nonvolatile Program Memory Size: 512K bytes, Used: 270323 bytes (52%) Free: 253965 bytes (48%)
  --= Second Nonvolatile Program Memory Size: None
  --= Internal SRAM Size: 64K bytes
  --= Architecture Identifier: AT91SAM7Sxx Series
  --= Nonvolatile Program Memory Type: Embedded Flash Memory

doegox commented 5 years ago

Thanks! @iceman1001 it comes from 3587aeff2aaeb8d326ac2baa35824eecb02a7936 cf this strange part in the commit, from ongoing changes and committed too soon?

@@ -1215,7 +1212,11 @@ static void PacketReceived(PacketCommandNG *packet) {
             break;
         }
         case CMD_HF_ICLASS_READER: {
-            ReaderIClass(packet->oldarg[0]);
+            struct p {
+                uint8_t flags;
+            } PACKED;
+            struct p *payload = (struct p *)packet->data.asBytes;
+            ReaderIClass(payload->flags);
             break;
         }

doegox commented 5 years ago

CMD_HF_ICLASS_READER is still used with SendCommandMIX, I'll revert that snippet for now.

iceman1001 commented 5 years ago

*#&/"¤%... that is part of my upcoming iclass change from mix -> NG...

RfidResearchGroup / proxmark3

endless loop after "hf search" ... #397