Harald Welte has submitted this change and it was merged. (
https://gerrit.osmocom.org/10312 )
Change subject: sniff: send incomplete TPDU when reset is asserted
......................................................................
sniff: send incomplete TPDU when reset is asserted
Change-Id: I8d7e4d604cded535e40d27c2be872268e0f24c20
---
M firmware/libcommon/source/sniffer.c
1 file changed, 17 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
Harald Welte: Looks good to me, approved
diff --git a/firmware/libcommon/source/sniffer.c
b/firmware/libcommon/source/sniffer.c
index 982653c..3cd0b89 100644
--- a/firmware/libcommon/source/sniffer.c
+++ b/firmware/libcommon/source/sniffer.c
@@ -1050,6 +1050,23 @@
/* Handle flags */
if (change_flags) { /* WARNING this is not synced with the data buffer
handling */
if (change_flags & SNIFF_CHANGE_FLAG_RESET_ASSERT) {
+ switch (iso_state) {
+ case ISO7816_S_IN_ATR:
+ led_blink(LED_RED, BLINK_2F_O); /* indicate
error to user */
+ usb_send_atr(SNIFF_DATA_FLAG_ERROR_INCOMPLETE);
/* send incomplete ATR to host software using USB */
+ break;
+ case ISO7816_S_IN_TPDU:
+ led_blink(LED_RED, BLINK_2F_O); /* indicate
error to user */
+
usb_send_tpdu(SNIFF_DATA_FLAG_ERROR_INCOMPLETE); /* send incomplete PPS to host
software using USB */
+ break;
+ case ISO7816_S_IN_PPS_REQ:
+ case ISO7816_S_IN_PPS_RSP:
+ led_blink(LED_RED, BLINK_2F_O); /* indicate
error to user */
+ usb_send_pps(SNIFF_DATA_FLAG_ERROR_INCOMPLETE);
/* send incomplete TPDU to host software using USB */
+ break;
+ default:
+ break;
+ }
if (ISO7816_S_RESET != iso_state) {
change_state(ISO7816_S_RESET);
printf("reset asserted\n\r");
--
To view, visit https://gerrit.osmocom.org/10312
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: simtrace2
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8d7e4d604cded535e40d27c2be872268e0f24c20
Gerrit-Change-Number: 10312
Gerrit-PatchSet: 1
Gerrit-Owner: Kévin Redon <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
