diff options
author | Bill Meier <wmeier@newsguy.com> | 2014-10-05 16:12:34 -0400 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2014-10-05 20:19:53 +0000 |
commit | ad6134bb4b686e7c70503ff169eb10f8fe463c97 (patch) | |
tree | ed5ab5bb2f8d8e2bc9ef2c4613adeab62a7df1d6 /plugins/docsis/packet-regreqmp.c | |
parent | 0b18d6cb77022494a875e6e0d9d2ad27fb6ac6d9 (diff) |
docsis: Add editor modelines; Adjust whitespace; Remove boilerplate comments.
Change-Id: I78f9815616f4fc3c14204339c5826f0ca7d0680e
Reviewed-on: https://code.wireshark.org/review/4485
Reviewed-by: Bill Meier <wmeier@newsguy.com>
Diffstat (limited to 'plugins/docsis/packet-regreqmp.c')
-rw-r--r-- | plugins/docsis/packet-regreqmp.c | 135 |
1 files changed, 64 insertions, 71 deletions
diff --git a/plugins/docsis/packet-regreqmp.c b/plugins/docsis/packet-regreqmp.c index da6df60328..849f8302ae 100644 --- a/plugins/docsis/packet-regreqmp.c +++ b/plugins/docsis/packet-regreqmp.c @@ -40,96 +40,89 @@ static int hf_docsis_regreqmp_fragment_sequence_number = -1; static dissector_handle_t docsis_tlv_handle; - - /* Initialize the subtree pointers */ static gint ett_docsis_regreqmp = -1; +/* Dissection */ static void dissect_regreqmp (tvbuff_t * tvb, packet_info * pinfo, proto_tree * tree) { + proto_item *it; + proto_tree *regreqmp_tree = NULL; + tvbuff_t *next_tvb; - proto_item *it; - proto_tree *regreqmp_tree = NULL; - tvbuff_t *next_tvb; - - col_set_str(pinfo->cinfo, COL_INFO, "REG-REQ-MP Message:"); - - if (tree) - { - it = proto_tree_add_protocol_format (tree, proto_docsis_regreqmp, tvb, 0, -1,"REG-REQ-MP Message"); - regreqmp_tree = proto_item_add_subtree (it, ett_docsis_regreqmp); - - proto_tree_add_item (regreqmp_tree, hf_docsis_regreqmp_sid, tvb, 0, 2, ENC_BIG_ENDIAN); - proto_tree_add_item (regreqmp_tree, hf_docsis_regreqmp_number_of_fragments, tvb, 2, 1, ENC_BIG_ENDIAN); - proto_tree_add_item (regreqmp_tree, hf_docsis_regreqmp_fragment_sequence_number, tvb, 3, 1, ENC_BIG_ENDIAN); - - } - /* Call Dissector for Appendix C TLV's */ - next_tvb = tvb_new_subset_remaining (tvb, 4); - call_dissector (docsis_tlv_handle, next_tvb, pinfo, regreqmp_tree); - } + col_set_str(pinfo->cinfo, COL_INFO, "REG-REQ-MP Message:"); + if (tree) + { + it = proto_tree_add_protocol_format (tree, proto_docsis_regreqmp, tvb, 0, -1,"REG-REQ-MP Message"); + regreqmp_tree = proto_item_add_subtree (it, ett_docsis_regreqmp); + proto_tree_add_item (regreqmp_tree, hf_docsis_regreqmp_sid, tvb, 0, 2, ENC_BIG_ENDIAN); + proto_tree_add_item (regreqmp_tree, hf_docsis_regreqmp_number_of_fragments, tvb, 2, 1, ENC_BIG_ENDIAN); + proto_tree_add_item (regreqmp_tree, hf_docsis_regreqmp_fragment_sequence_number, tvb, 3, 1, ENC_BIG_ENDIAN); + } + /* Call Dissector for Appendix C TLV's */ + next_tvb = tvb_new_subset_remaining (tvb, 4); + call_dissector (docsis_tlv_handle, next_tvb, pinfo, regreqmp_tree); +} /* Register the protocol with Wireshark */ - -/* this format is require because a script is used to build the C function - that calls all the protocol registration. -*/ - - void proto_register_docsis_regreqmp (void) { - /* Setup list of header fields See Section 1.6.1 for details*/ - static hf_register_info hf[] = { - {&hf_docsis_regreqmp_sid, - {"Sid", "docsis_regreqmp.sid", - FT_UINT16, BASE_DEC, NULL, 0x0, - "Reg-Req-Mp Sid", HFILL} - }, - {&hf_docsis_regreqmp_number_of_fragments, - {"Number of Fragments", "docsis_regreqmp.number_of_fragments", - FT_UINT8, BASE_DEC, NULL, 0x0, - "Reg-Req-Mp Number of Fragments", HFILL} - }, - {&hf_docsis_regreqmp_fragment_sequence_number, - {"Fragment Sequence Number", "docsis_regreqmp.fragment_sequence_number", - FT_UINT8, BASE_DEC, NULL, 0x0, - "Reg-Req-Mp Fragment Sequence Number", HFILL} - }, - }; - - /* Setup protocol subtree array */ - static gint *ett[] = { - &ett_docsis_regreqmp, - }; - - /* Register the protocol name and description */ - proto_docsis_regreqmp = - proto_register_protocol ("DOCSIS Registration Request Multipart", - "DOCSIS Reg-Req-Mp", "docsis_regreqmp"); - - /* Required function calls to register the header fields and subtrees used */ - proto_register_field_array (proto_docsis_regreqmp, hf, array_length (hf)); - proto_register_subtree_array (ett, array_length (ett)); - - register_dissector ("docsis_regreqmp", dissect_regreqmp, proto_docsis_regreqmp); + static hf_register_info hf[] = { + {&hf_docsis_regreqmp_sid, + {"Sid", "docsis_regreqmp.sid", + FT_UINT16, BASE_DEC, NULL, 0x0, + "Reg-Req-Mp Sid", HFILL} + }, + {&hf_docsis_regreqmp_number_of_fragments, + {"Number of Fragments", "docsis_regreqmp.number_of_fragments", + FT_UINT8, BASE_DEC, NULL, 0x0, + "Reg-Req-Mp Number of Fragments", HFILL} + }, + {&hf_docsis_regreqmp_fragment_sequence_number, + {"Fragment Sequence Number", "docsis_regreqmp.fragment_sequence_number", + FT_UINT8, BASE_DEC, NULL, 0x0, + "Reg-Req-Mp Fragment Sequence Number", HFILL} + }, + }; + + static gint *ett[] = { + &ett_docsis_regreqmp, + }; + + proto_docsis_regreqmp = + proto_register_protocol ("DOCSIS Registration Request Multipart", + "DOCSIS Reg-Req-Mp", "docsis_regreqmp"); + + proto_register_field_array (proto_docsis_regreqmp, hf, array_length (hf)); + proto_register_subtree_array (ett, array_length (ett)); + + register_dissector ("docsis_regreqmp", dissect_regreqmp, proto_docsis_regreqmp); } - -/* If this dissector uses sub-dissector registration add a registration routine. - This format is required because a script is used to find these routines and - create the code that calls these routines. -*/ void proto_reg_handoff_docsis_regreqmp (void) { - dissector_handle_t docsis_regreqmp_handle; + dissector_handle_t docsis_regreqmp_handle; - docsis_tlv_handle = find_dissector ("docsis_tlv"); - docsis_regreqmp_handle = find_dissector ("docsis_regreqmp"); - dissector_add_uint ("docsis_mgmt", 44, docsis_regreqmp_handle); + docsis_tlv_handle = find_dissector ("docsis_tlv"); + docsis_regreqmp_handle = find_dissector ("docsis_regreqmp"); + dissector_add_uint ("docsis_mgmt", 44, docsis_regreqmp_handle); } + +/* + * Editor modelines - http://www.wireshark.org/tools/modelines.html + * + * Local Variables: + * c-basic-offset: 2 + * tab-width: 8 + * indent-tabs-mode: nil + * End: + * + * ex: set shiftwidth=2 tabstop=8 expandtab: + * :indentSize=2:tabSize=8:noTabs=true: + */ |