Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 51119: /trunk/ /trunk/epan/: addr_re

From: Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>
Date: Wed, 07 Aug 2013 12:46:33 -0400
On 08/05/13 11:17, Anders Broman wrote:


-----Original Message-----
From: wireshark-dev-bounces@xxxxxxxxxxxxx [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Jeff Morriss
Sent: den 5 augusti 2013 17:14
To: wireshark-dev@xxxxxxxxxxxxx
Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 51119: /trunk/ /trunk/epan/: addr_resolv.c addr_resolv.h /trunk/ui/gtk/: addr_resolution_dlg.c

On 08/05/13 09:18, Jeff Morriss wrote:
On 08/02/13 14:22, Jeff Morriss wrote:
On 08/02/13 11:05, etxrab@xxxxxxxxxxxxx wrote:
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=511
19

User: etxrab
Date: 2013/08/02 08:05 AM

Log:
   - Use a hashtable for eth lookup.
   Currently broken get_ether_addr().
   - Show hastable content in address dlg.(temp solution).

This (or maybe another rev) is causing plenty of errors like:

(lt-wireshark:6958): GLib-CRITICAL **: g_hash_table_lookup:
assertion `hash_table != NULL' failed

Michael fixed this in r51133.

Oops, I spoke too soon.  There are still some in there.

Perhaps the table(s) should be crated at initialize_ethers()?

I did that (and some more) in r51191.