diff options
| author | Jiri Novak <j.novak@netsystem.cz> | 2018-03-02 14:59:43 +0100 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2018-03-04 11:35:34 +0000 |
| commit | 5a05aa238ca69adfffcad21c755464bafa24249f (patch) | |
| tree | 656cc5447665ef719d8950c0412415855912212b /epan/dissectors/packet-rtsp.c | |
| parent | cf373df7963d9ed6afee8285e2e1633ff6f7d93f (diff) | |
RTSP: Fixed missing space in column info between multiple pipelined messages
When there is multiple RTSP messages in one packet, info column shows its headers right next to each other. It is ugly:
Reply: RTSP/1.0 200 OKReply: RTSP/1.0 200 OK
Patch adds ', ' between messages:
Reply: RTSP/1.0 200 OK, Reply: RTSP/1.0 200 OK
Ping-Bug: 14450
Change-Id: I151dbc72b669002ed02d91af43d683c5fc4fe4ba
Reviewed-on: https://code.wireshark.org/review/26222
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-rtsp.c')
| -rw-r--r-- | epan/dissectors/packet-rtsp.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/epan/dissectors/packet-rtsp.c b/epan/dissectors/packet-rtsp.c index 0d667c0dd6..295c91bc00 100644 --- a/epan/dissectors/packet-rtsp.c +++ b/epan/dissectors/packet-rtsp.c @@ -1482,6 +1482,13 @@ dissect_rtsp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data _U_ int len; while (tvb_reported_length_remaining(tvb, offset) != 0) { + /* + * Add separator between multiple messages in column info text + */ + if (offset > 0) { + col_set_str(pinfo->cinfo, COL_INFO, ", "); + col_set_fence(pinfo->cinfo, COL_INFO); + } len = (tvb_get_guint8(tvb, offset) == RTSP_FRAMEHDR) ? dissect_rtspinterleaved(tvb, offset, pinfo, tree) : dissect_rtspmessage(tvb, offset, pinfo, tree); |