dissectors: use '/' when stacking protocols in COL_PROTOCOL.
A commoly used convention when adding more then 1 protocols in COL_PROTOCOL (using col_set_fence) is to separate them using the '/' character. Some dissectors use ' ', others use '|'. Make them all use '/'. Change-Id: Ibcddd7500f637d96313b264122d48ac6bff1e96c Reviewed-on: https://code.wireshark.org/review/31804 Petri-Dish: Anders Broman <a.broman58@gmail.com> Tested-by: Petri Dish Buildbot Reviewed-by: Anders Broman <a.broman58@gmail.com>
This commit is contained in:
parent
30f8175303
commit
8d599e1952
|
@ -743,7 +743,7 @@ dissect_imf(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data _U_)
|
|||
}
|
||||
|
||||
/* Want to preserve existing protocol name and show that it is carrying IMF */
|
||||
col_append_str(pinfo->cinfo, COL_PROTOCOL, "|");
|
||||
col_append_str(pinfo->cinfo, COL_PROTOCOL, "/");
|
||||
col_set_fence(pinfo->cinfo, COL_PROTOCOL);
|
||||
col_set_str(pinfo->cinfo, COL_PROTOCOL, PSNAME);
|
||||
|
||||
|
|
|
@ -6973,7 +6973,7 @@ dissect_mbim_bulk(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *dat
|
|||
datagram_tvb = tvb_new_subset_length(tvb, datagram_index, datagram_length);
|
||||
if (dissector) {
|
||||
if (total) {
|
||||
col_set_str(pinfo->cinfo, COL_PROTOCOL, "|");
|
||||
col_set_str(pinfo->cinfo, COL_PROTOCOL, "/");
|
||||
col_set_fence(pinfo->cinfo, COL_PROTOCOL);
|
||||
col_set_str(pinfo->cinfo, COL_INFO, " | ");
|
||||
col_set_fence(pinfo->cinfo, COL_INFO);
|
||||
|
|
|
@ -72,7 +72,7 @@ dissect_tte(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _U_)
|
|||
}
|
||||
|
||||
/* Make entries in Protocol column and Info column on summary display */
|
||||
col_set_str(pinfo->cinfo, COL_PROTOCOL, "TTE ");
|
||||
col_set_str(pinfo->cinfo, COL_PROTOCOL, "TTE/");
|
||||
|
||||
if (tvb_get_ntohs(tvb, TTE_MAC_LENGTH * 2) <= IEEE_802_3_MAX_LEN)
|
||||
{
|
||||
|
|
Loading…
Reference in New Issue