Current Lldp Dissector column information just shows the "System Description" information. In the Profinet applications, this information does not help too much. On the other hand Chassis Id and Port Id gives much more information. In the source code (packet-lldp.c), it is very easy to comment followig line: And instead of this line, adding following to dissect_lldp_chassis_id function: But this change will affect whole Lldp messages' column information. I want to ask if I have a right to change it in this manner or not before I begin to implementation. asked 27 Oct '15, 00:05  BirolCapa |
2 Answers:
Wireshark is an open source project licensed under the GPL. So, you have the all rights to change the source code based on what the GPL 'allows'. This translates to: yes, of course you can change that, as long as you adhere to the GPL. If you want to submit your changes back to the Wireshark code base, it will most certainly depend on other criteria if the submission will be accepted (code quality, useful enhancements, etc.), than just that single modification to the Info column. Regards answered 27 Oct '15, 01:27  Kurt Knochner ♦ |
As Kurt described, based on the GPL you have the right to make the changes and redistributed them, including (offering of) source code. What I won't see happening is this change being accepted into the main repository. This will not happen because the statement you quoted is part op What you seem to be missing is that answered 27 Oct '15, 05:01  Jaap ♦ Hello Jaap, thank you for your answer, To show Chassis id and Port id, I will comment following (at dissect_lldp_system_name) "col_append_fstr(pinfo->cinfo, COL_INFO, "System Name = %s ", strPtr);" "col_append_fstr(pinfo->cinfo, COL_INFO, "System Description = %s ", strPtr);" So column info will not show the System Name and description anymore, and I will add dissect_lldp_chassis_id to this: "col_append_fstr(pinfo->cinfo, COL_INFO, "Chassis ID = %s ", strPtr);" and I will add dissect_lldp_port_id to this: "col_append_fstr(pinfo->cinfo, COL_INFO, "Port ID = %s ", strPtr);" So whole LLDP will show Chassis Id and Port Id instead of System Name and Description. This change will be very useful for Profinet, on the other hand I am not sure if it is appropriate for other users. That's why I am asking if there is a community decision about column info. (27 Oct '15, 05:12) BirolCapa So there seems to be a difference in opinion about what is useful to represent in the INFO column when dissecting LLDP, either within profinet, or other contexts. In that case a dissector preference would allow the user to choose what his/her preference is, whereby the default preference would match the current behaviour. (27 Oct '15, 06:36) Jaap ♦ |
Hello Kurt, thank you for your answer.
I understand you. But I just don't understand if there is a rule or standard that community obeys about column information: For example, why did community choose system name instead of another field? Is there a rule or standard etc. Or it was a just decision of the developer of the lldp dissector?
As far as I know, there is no 'community' decison about these things. I'd say it's the decision of the developer.