diff options
author | Guy Harris <guy@alum.mit.edu> | 2016-11-11 14:37:06 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2016-11-11 22:37:54 +0000 |
commit | 134f5ad96c55806c3410555a52a4b5407f472e48 (patch) | |
tree | ff4c7aa6c65e8ea3065c77251600763843f498a3 /epan/dissectors/packet-gdb.c | |
parent | 0decc3fb57416ef003afd9cbe2810aba98a74f89 (diff) |
Dissect the checksum field as a string, as that's what it is.
Bug: 13125
Change-Id: Ib5d02e8841eb7e4d04e6a1df44a5eba5b925ce6c
Reviewed-on: https://code.wireshark.org/review/18749
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'epan/dissectors/packet-gdb.c')
-rw-r--r-- | epan/dissectors/packet-gdb.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/epan/dissectors/packet-gdb.c b/epan/dissectors/packet-gdb.c index b6d3014ab9..a90107387a 100644 --- a/epan/dissectors/packet-gdb.c +++ b/epan/dissectors/packet-gdb.c @@ -98,9 +98,8 @@ dissect_gdb_token(void *tvbparse_data, const void *wanted_data, tvbparse_elem_t tok->tvb, tok->offset, tok->len, ENC_ASCII|ENC_NA); break; case GDB_TOK_CHKSUM: - /* the spec is not really explicit but it seems that the - checksum is big endian */ - proto_tree_add_checksum(tree, tok->tvb, tok->offset, hf_gdb_chksum, -1, NULL, NULL, 0, ENC_BIG_ENDIAN, PROTO_CHECKSUM_NO_FLAGS); + proto_tree_add_item(tree, hf_gdb_chksum, + tok->tvb, tok->offset, tok->len, ENC_ASCII|ENC_NA); break; default: break; @@ -208,8 +207,8 @@ proto_register_gdb(void) { "Terminating character", "gdb.end", FT_STRING, BASE_NONE, NULL, 0, NULL, HFILL } }, { &hf_gdb_chksum, - { "Checksum", "gdb.chksum", FT_UINT8, BASE_HEX, - NULL, 0, NULL, HFILL } } + { "Checksum", "gdb.chksum", FT_STRING, BASE_NONE, + NULL, 0, NULL, HFILL } } }; static gint *ett[] = { |