Wireshark mailing list archives
Re: WSDG - errors in message reassembly documentation
From: Dario Lombardo <lomato () gmail com>
Date: Thu, 25 Jun 2020 16:11:10 +0200
Hi Brett You could 1) file a bug on bugzilla 2) push the change yourself on gerrit, since the wsdg is versioned as the rest of the code. (or both of course). On Thu, Jun 25, 2020 at 4:04 PM Brett Hallen <sw () dh net au> wrote:
Hello! I've got my first dissector up and running using the excellent documentation and help from the community. I'm now going to (try to) get re-assembly working on messages that our protocol fragments. However the documentation seems to have some small errors that could confuse the easily confused like me: https://www.wireshark.org/docs/wsdg_html_chunked/ChDissectReassemble.html The code fragment examples use a variable called msg_reassembly_table, but then later on it is initialised as reassembly_table (same as the datatype name), and then registered as msg_reassemble_table: static reassembly_table reassembly_table; static void proto_register_msg(void){ reassembly_table_register(&msg_reassemble_table, &addresses_ports_reassembly_table_functions);} Obviously (I think) it should be use the variable name msg_reassembly_table instead of assembly_table and msg_assemble_table? static reassembly_table *msg_*reassembly_table; static void proto_register_msg(void){ reassembly_table_register(&msg_reassembly_table, &addresses_ports_reassembly_table_functions);} Brett. ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org ?subject=unsubscribe
-- Naima is online.
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- WSDG - errors in message reassembly documentation Brett Hallen (Jun 25)
- Re: WSDG - errors in message reassembly documentation Dario Lombardo (Jun 25)