diff options
author | Hadriel Kaplan <hadrielk@yahoo.com> | 2015-07-25 09:24:48 -0400 |
---|---|---|
committer | Hadriel Kaplan <hadrielk@yahoo.com> | 2015-07-25 17:05:01 +0000 |
commit | 190e313e9d949a837a0b3c3fe027647b35c2f8be (patch) | |
tree | 62c25c82c240d5aff4ef24b582c40c08d937e3d9 /rawshark.c | |
parent | 3fc4a831e035604b0af14ed8a5c9f6596a3448d0 (diff) |
DNS: move DNS name resolution pref to Name Resolution prefs
Move the boolean flag for using captured DNS packet info for name resolution
to the Name Resolution preferences settings, as it was rather surprising to
disable Name Resolution preferences and still have names being resolved. Also
disble them all if the '-n' command line switch is used, and re-enable it for
a 'd' character in the '-N' option.
Bug: 10337
Change-Id: Ie4d47bab0100db3360cc447cd3e446b2e39aa917
Reviewed-on: https://code.wireshark.org/review/9786
Petri-Dish: Hadriel Kaplan <hadrielk@yahoo.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Hadriel Kaplan <hadrielk@yahoo.com>
Diffstat (limited to 'rawshark.c')
-rw-r--r-- | rawshark.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/rawshark.c b/rawshark.c index 6981f6d3a5..c44bacef4b 100644 --- a/rawshark.c +++ b/rawshark.c @@ -201,7 +201,7 @@ print_usage(FILE *output) fprintf(output, " packet encapsulation or protocol\n"); fprintf(output, " -F <field> field to display\n"); fprintf(output, " -n disable all name resolution (def: all enabled)\n"); - fprintf(output, " -N <name resolve flags> enable specific name resolution(s): \"mntC\"\n"); + fprintf(output, " -N <name resolve flags> enable specific name resolution(s): \"mnNtCd\"\n"); fprintf(output, " -p use the system's packet header format\n"); fprintf(output, " (which may have 64-bit timestamps)\n"); fprintf(output, " -R <read filter> packet filter in Wireshark display filter syntax\n"); @@ -636,15 +636,12 @@ DIAG_ON(cast-qual) line_buffered = TRUE; break; case 'n': /* No name resolution */ - gbl_resolv_flags.mac_name = FALSE; - gbl_resolv_flags.network_name = FALSE; - gbl_resolv_flags.transport_name = FALSE; - gbl_resolv_flags.concurrent_dns = FALSE; + disable_name_resolution(); break; case 'N': /* Select what types of addresses/port #s to resolve */ badopt = string_to_name_resolve(optarg, &gbl_resolv_flags); if (badopt != '\0') { - cmdarg_err("-N specifies unknown resolving option '%c'; valid options are 'm', 'n', and 't'", + cmdarg_err("-N specifies unknown resolving option '%c'; valid options are 'C', 'd', m', 'n', 'N', and 't'", badopt); exit(1); } |