On Sun, Feb 15, 2004 at 10:14:25AM +0200, Yaniv Kaul wrote:
> + if(ResolveWin32UUID(if_id, UUID_NAME, MAX_PATH))
> + iface_item = proto_tree_add_string_format (ctx_tree,
> hf_dcerpc_cn_bind_if_id, tvb,
> + offset, 16, uuid_str, "Interface [%s] UUID:
> %s", UUID_NAME, uuid_str);
> + else
> iface_item = proto_tree_add_string_format (ctx_tree,
> hf_dcerpc_cn_bind_if_id, tvb,
Another point with this patch: IMO the orginal UUID should still be shown,
to make the association between the hex values and the dissected data clear.
The name could be added either as a subtree or prefix/postfix. In that
case, ResolveWin32UUID could return "unresolved" on non-win platforms.
Also, the function itself should be put into epan/resolv.[hc]
Ciao
J�rg
--
Joerg Mayer <[EMAIL PROTECTED]>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
_______________________________________________
Ethereal-dev mailing list
[EMAIL PROTECTED]
http://www.ethereal.com/mailman/listinfo/ethereal-dev
