diff options
| author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-07-13 21:20:00 +0000 |
|---|---|---|
| committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-07-13 21:20:00 +0000 |
| commit | ff63dcfa8ef615bd158744bdb56510b14b138093 (patch) | |
| tree | 8ba269c4236425a1bb0cfabfac50f3a439c39fe6 /epan/addr_resolv.c | |
| parent | ba376235daf625be3f37ed016b22c44595755a9c (diff) | |
Jeff Morriss indicated the following and I obliged:
Actually I suppose what was meant is more about whether tp->name[0] is the NULL character (that is, that the name is empty).
But since the result of the lookup is already there in 'found' the whole condition can probably be taken out.
svn path=/trunk/; revision=43701
Diffstat (limited to 'epan/addr_resolv.c')
| -rw-r--r-- | epan/addr_resolv.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/epan/addr_resolv.c b/epan/addr_resolv.c index 2d0ba04d6f..fa0c738728 100644 --- a/epan/addr_resolv.c +++ b/epan/addr_resolv.c @@ -3022,13 +3022,10 @@ add_ether_byip(const guint ip, const guint8 *eth) return; tp = host_lookup(ip, &found); - if (tp == NULL) - return; - - /* ok, we can add this entry in the ethers hashtable */ - - if (found) + if (found) { + /* ok, we can add this entry in the ethers hashtable */ add_eth_name(eth, tp->name); + } } /* add_ether_byip */ |