Ethereal-dev: Re: [Ethereal-dev] patch for packet-ncp2222.inc to fix GLIB-CRITICAL error.

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next
From: Guy Harris <guy@xxxxxxxxxx>
Date: Tue, 25 Feb 2003 10:44:59 -0800
On Tue, Feb 25, 2003 at 08:38:36AM -0700, Greg Morris wrote:
> Potential error if hash lookup for NDS EID is null.
>  
> GLib-CRITICAL **: file ghash.c: line 138 (g_hash_table_lookup):
> assertion `hash_table != NULL' failed.
>  
> Added check prior to hash lookup for flags.visited.

Unfortunately, one result of that check is that one item will be added
to the protocol tree only on the first pass through the capture file and
on any subsequent pass that forces a redissection, as there's a
"proto_tree_add_string_format()" call inside

	if (!pinfo->fd->flags.visited) {

		...

	}

You need to come up with a scheme to get the EID value even *after*
you've destroyed the hash table, e.g. by attaching it to the frame as
per-packet data.