From e1979529b682a3b9b0bcefe3a54058c5ae314c5b Mon Sep 17 00:00:00 2001 From: Pascal Quantin Date: Sat, 24 Oct 2015 11:32:16 +0200 Subject: SDP: protect against out of bounds access Change-Id: I4b24441cb26757b639e8113cab18d64c7f07112f Ping-Bug: 9887 Reviewed-on: https://code.wireshark.org/review/11241 Reviewed-by: Pascal Quantin (cherry picked from commit 2ddd92b6f8f587325b9e14598658626f3a007c5c) Reviewed-on: https://code.wireshark.org/review/11261 --- epan/dissectors/packet-sdp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/epan/dissectors/packet-sdp.c b/epan/dissectors/packet-sdp.c index 8062045c0d..96c2bd4ad8 100644 --- a/epan/dissectors/packet-sdp.c +++ b/epan/dissectors/packet-sdp.c @@ -2617,7 +2617,7 @@ dissect_sdp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) /* Free the remainded hash tables not used */ /* XXX - more placeholder functionality */ if (transport_info == &local_transport_info) { - for (n = transport_info->media_count; n < SDP_MAX_RTP_CHANNELS; n++) + for (n = MAX(transport_info->media_count, 0); n < SDP_MAX_RTP_CHANNELS; n++) { if (!transport_info->media[n].set_rtp) { -- cgit v1.2.1