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 <kre...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder

Reply via email to