Guy Harris wrote:
> What's the code in the subdissector that adds the top-level entry for
> the protocol?
Sorry - I just realized you asked for the subdissector code, and I sent
the dissector code for the top-level protocol. At least you can check that
I'm calling dissector_try_port properly...
Here's the subdissector code:
static int
dissect_alp_commonr(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
{
guint seqno = tvb_get_ntoh24(tvb, 0);
if (check_col(pinfo->cinfo,COL_INFO)){
col_append_fstr(pinfo->cinfo, COL_INFO, " CmdSeq=0x%x", seqno);
}
if (tree) { // we are being asked for details
proto_item *ti = NULL;
proto_tree *alp_commonr_tree = NULL;
ti = proto_tree_add_item(tree, proto_alp_commonr, tvb, 0,
sizeof(struct n_basic_cmd), FALSE);
proto_item_append_text(ti, ", CmdSeq: 0x%x", seqno);
alp_commonr_tree = proto_item_add_subtree(ti, ett_alp_commonr);
proto_tree_add_item(alp_commonr_tree, hf_alp_commonr_seq_num, tvb,
OFFANDSIZE(n_basic_cmd, cmd_seq),
FALSE);
proto_tree_add_item(alp_commonr_tree, hf_alp_commonr_x, tvb,
OFFANDSIZE(n_basic_cmd, x),
FALSE);
proto_tree_add_item(alp_commonr_tree, hf_alp_commonr_y, tvb,
OFFANDSIZE(n_basic_cmd, y),
FALSE);
proto_tree_add_item(alp_commonr_tree, hf_alp_commonr_width, tvb,
OFFANDSIZE(n_basic_cmd, width),
FALSE);
proto_tree_add_item(alp_commonr_tree, hf_alp_commonr_height, tvb,
OFFANDSIZE(n_basic_cmd, height),
FALSE);
}
return(sizeof(struct n_basic_cmd));
}
I get the COL_INFO update, and I can trace that the code is going into
the branch where it's making the proto_tree_add_item() calls (and verified
that proto_alp_commonr is getting registered/initialized), but none of
that appears...
-Bob
_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@wireshark.org
http://www.wireshark.org/mailman/listinfo/wireshark-dev
