diff options
author | Anthony Coddington <anthony.coddington@endace.com> | 2017-06-09 17:16:27 +1200 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-06-13 21:07:57 +0000 |
commit | f3267f17a3fc3c8ab138940653f2e968c5d27378 (patch) | |
tree | 6862a1eb6e793e3548b8c12d5f77b5b0d02e2297 /epan/addr_resolv.c | |
parent | ec4ba8b275572c50c1c8f7097c4a47c683291d51 (diff) |
Fix automatic name resolution not saved to NRB
Clear DUMMY_ADDRESS_ENTRY when add_ipv4_name/add_ipv6_name. This flag is checked in ipv4_hash_table_resolved_to_list().
TODO: clean up these flags as they are confusing and DUMMY appears somewhat redundant.
Change-Id: I81d40cc778cbe5c36314631d3fa0997cee409368
Bug: 13798
Reviewed-on: https://code.wireshark.org/review/22109
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/addr_resolv.c')
-rw-r--r-- | epan/addr_resolv.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/epan/addr_resolv.c b/epan/addr_resolv.c index d55e857520..7cf134c6c2 100644 --- a/epan/addr_resolv.c +++ b/epan/addr_resolv.c @@ -2711,6 +2711,8 @@ add_ipv4_name(const guint addr, const gchar *name) new_resolved_objects = TRUE; } tp->flags |= TRIED_RESOLVE_ADDRESS|NAME_RESOLVED; + /* Clear DUMMY_ADDRESS_ENTRY */ + tp->flags &= ~DUMMY_ADDRESS_ENTRY; } /* add_ipv4_name */ /* -------------------------- */ @@ -2741,6 +2743,8 @@ add_ipv6_name(const struct e_in6_addr *addrp, const gchar *name) new_resolved_objects = TRUE; } tp->flags |= TRIED_RESOLVE_ADDRESS|NAME_RESOLVED; + /* Clear DUMMY_ADDRESS_ENTRY */ + tp->flags &= ~DUMMY_ADDRESS_ENTRY; } /* add_ipv6_name */ static void |