Graham Bloice wrote:
On 26/04/2010 10:35, Graham Bloice wrote:
On 26/04/2010 10:10, ankur madan wrote:
I have latest version code from the svn and latest development
wireshark version.1.3.4 installed.i am still not able to resolve this
issue.any help or comments are welcome
thanks
ankur
On Mon, Apr 26, 2010 at 12:51 PM, ankur madan <ankurmadan9@xxxxxxxxx
<mailto:ankurmadan9@xxxxxxxxx>> wrote:
I am compiling on windows XP n 32 bit.
On Mon, Apr 26, 2010 at 3:37 AM, Stephen Fisher
<steve@xxxxxxxxxxxxxxxxxx <mailto:steve@xxxxxxxxxxxxxxxxxx>> wrote:
On Fri, Apr 23, 2010 at 02:42:35PM +0530, ankur madan wrote:
> packet-ieee802154.c(982) : warning C4244: '=' : conversion from
> 'guint64' to 'gu int32', possible loss of data
I see that you're compiling on Windows, but which version and
is it
32-bit or 64-bit?
Personally I'm just carrying fixes to this in my local source tree as
I haven't had time to push a patch back into the repo. I don't know
why this compiles OK on the buildbot and not with VC8.
IMHO the current code is incorrect as it incorrectly casts a 32 bit
value to a 64 bit one and then calls proto_tree_add_uint64 with it
when the value really is a 32 bit one (lines 981-985):
if (packet->key_id_mode == KEY_ID_MODE_KEY_EXPLICIT_4) {
packet->key_source.addr32 = tvb_get_ntohl(tvb, offset);
proto_tree_add_uint64(field_tree,
hf_ieee802154_aux_sec_key_source, tvb, offset, 4,
packet->key_source.addr32);
proto_item_set_len(ti, 1 + 4);
offset += sizeof (guint32);
and the other errors simply need a cast to gchar to allow them to
compile (lines 2006 - 2013).
I've attached a patch that works for me.
Oops. Compile before posting, the last second change to
proto_tree_add_uint64 should have been to proto_tree_add_uint.
Corrected patch supplied.
Also note that if you are using VC8 you may run into other compilation
or run-time issues. I strongly recommend you grab a copy of the VS 2008
Express edition or even the 2010 version if you want to live at the
bleeding edge.
The problem with this patch is that hf_ieee802154_aux_sec_key_source is
an FT_UINT64. Trying to use proto_tree_add_uint() on it will result in
a dissector assertion (see epan/proto.c: proto_tree_add_uint()).