From 7cbc790832cfb067857b523ef2ad440966ef824a Mon Sep 17 00:00:00 2001 From: Guy Harris Date: Wed, 9 Apr 2014 17:48:32 -0700 Subject: Avoid bug 9962 - don't use variables that haven't been set. See the comment for details. Change-Id: I0b1800dd884736e00201f09166c0d2bd6795b6c3 Reviewed-on: https://code.wireshark.org/review/1047 Reviewed-by: Guy Harris --- epan/dissectors/packet-lbmc.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'epan') diff --git a/epan/dissectors/packet-lbmc.c b/epan/dissectors/packet-lbmc.c index 36b5318d5e..88645377c3 100644 --- a/epan/dissectors/packet-lbmc.c +++ b/epan/dissectors/packet-lbmc.c @@ -11018,7 +11018,11 @@ int lbmc_dissect_lbmc_packet(tvbuff_t * tvb, int offset, packet_info * pinfo, pr COPY_ADDRESS_SHALLOW(&tcp_addr, &(pinfo->dst)); tcp_port = (guint16)pinfo->destport; } - if ((pinfo->fd->flags.visited == 0) && (tcp_sid_info.set) && !lbm_channel_is_known(channel)) + /* XXX - do we need to check lbm_channel_is_unknown_stream_tcp(channel)? + We must *NOT* call lbttcp_transport_sid_add() unless + lbm_channel_is_unknown_transport_lbttcp(channel) is true as, if + it's not true, we will *NOT* have set tcp_addr or tcp_port above! */ + if ((pinfo->fd->flags.visited == 0) && (tcp_sid_info.set) && lbm_channel_is_unknown_transport_lbttcp(channel)) { lbttcp_transport_sid_add(&tcp_addr, tcp_port, pinfo->fd->num, tcp_sid_info.session_id); } -- cgit v1.2.3