diff options
author | Darius Davis <darius@vmware.com> | 2018-12-01 17:59:00 +1000 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2018-12-01 22:28:44 +0000 |
commit | b44172b4ee1dcdc6b1d7b41f944cec17b117776f (patch) | |
tree | 0d7885f8571d4173bd9ed2edbba376463b6708c9 | |
parent | 71268f8bd1458048e2d75f5a3b8bc94feb3b7d0c (diff) |
TFTP: Minor tweaks to tftp.destination_file field.
Two trivial cleanups of the definition of the tftp.destination_file field:
There is probably no need to shout DESTINATION in capital letters, and change
"source" to "destination" in the field's blurb.
Testing Done: Built on macOS 10.12.6. Examined the capture attached to
bug 10305 (tftpConversationError.pcapng, which includes a TFTP WRQ), and saw
that the capitalization of the "Destination File" field is as expected in
the packet dissection, and that the status bar now describes the field as
the "TFTP destination file name".
Change-Id: I9f5bded321c16d4e200bf1caf80ad5733ecc8287
Reviewed-on: https://code.wireshark.org/review/30857
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
-rw-r--r-- | epan/dissectors/packet-tftp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-tftp.c b/epan/dissectors/packet-tftp.c index 0bcda017ed..852d33653e 100644 --- a/epan/dissectors/packet-tftp.c +++ b/epan/dissectors/packet-tftp.c @@ -798,9 +798,9 @@ proto_register_tftp(void) "TFTP source file name", HFILL }}, { &hf_tftp_destination_file, - { "DESTINATION File", "tftp.destination_file", + { "Destination File", "tftp.destination_file", FT_STRINGZ, BASE_NONE, NULL, 0x0, - "TFTP source file name", HFILL }}, + "TFTP destination file name", HFILL }}, { &hf_tftp_transfer_type, { "Type", "tftp.type", |