diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2016-02-16 17:12:26 +0100 |
---|---|---|
committer | Martin Kaiser <wireshark@kaiser.cx> | 2016-02-16 22:24:49 +0000 |
commit | d4ee4a9c942456be8a06a544b6989b51f1d2c61d (patch) | |
tree | da55f00d303c763aa0efdcb35c861bd20f1073ec /epan/dissectors/packet-iso14443.c | |
parent | 989739c4fcb229f51389997e0a5e2947b97c68d2 (diff) |
iso14443: fix the NAK/ACK bit of an R-block
if the bit is set, it's an R(NAK) block
Change-Id: I0e44bd72d1c2a69a582792d08bf450e6ef2d163b
Reviewed-on: https://code.wireshark.org/review/13976
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Diffstat (limited to 'epan/dissectors/packet-iso14443.c')
-rw-r--r-- | epan/dissectors/packet-iso14443.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/epan/dissectors/packet-iso14443.c b/epan/dissectors/packet-iso14443.c index 25fc454468..af72ca5710 100644 --- a/epan/dissectors/packet-iso14443.c +++ b/epan/dissectors/packet-iso14443.c @@ -148,7 +148,7 @@ static const true_false_string tfs_compliant_not_compliant = { "Compliant", "Not static const true_false_string tfs_incomplete_complete = { "Incomplete", "Complete" }; static const true_false_string tfs_iso_propr = { "As defined in ISO14443-3", "Proprietary" }; static const true_false_string tfs_not_required_required = { "Not required", "Required" }; -static const true_false_string tfs_ack_nak = { "ACK", "NAK" }; +static const true_false_string tfs_nak_ack = { "NAK", "ACK" }; #define CT_BYTE 0x88 @@ -246,7 +246,7 @@ static int hf_iso14443_block_type = -1; static int hf_iso14443_i_blk_chaining = -1; static int hf_iso14443_cid_following = -1; static int hf_iso14443_nad_following = -1; -static int hf_iso14443_ack_nak = -1; +static int hf_iso14443_nak = -1; static int hf_iso14443_blk_num = -1; static int hf_iso14443_s_blk_cmd = -1; static int hf_iso14443_pwr_lvl_ind = -1; @@ -839,8 +839,8 @@ dissect_iso14443_cmd_type_block(tvbuff_t *tvb, packet_info *pinfo, case R_BLOCK_TYPE: col_append_sep_fstr(pinfo->cinfo, COL_INFO, NULL, "%s", (pcb & 0x10) ? - tfs_ack_nak.true_string : tfs_ack_nak.false_string); - proto_tree_add_item(pcb_tree, hf_iso14443_ack_nak, + tfs_nak_ack.true_string : tfs_nak_ack.false_string); + proto_tree_add_item(pcb_tree, hf_iso14443_nak, tvb, offset, 1, ENC_BIG_ENDIAN); proto_tree_add_item(pcb_tree, hf_iso14443_cid_following, tvb, offset, 1, ENC_BIG_ENDIAN); @@ -1445,9 +1445,9 @@ proto_register_iso14443(void) { "NAD following", "iso14443.nad_following", FT_BOOLEAN, 8, TFS(&tfs_true_false), 0x04, NULL, HFILL } }, - { &hf_iso14443_ack_nak, - { "ACK/NAK", "iso14443.ack_nak", FT_BOOLEAN, 8, - TFS(&tfs_ack_nak), 0x10, NULL, HFILL } + { &hf_iso14443_nak, + { "NAK/ACK", "iso14443.nak", FT_BOOLEAN, 8, + TFS(&tfs_nak_ack), 0x10, NULL, HFILL } }, { &hf_iso14443_blk_num, { "Block number", "iso14443.block_number", |