diff options
author | Anders Broman <anders.broman@ericsson.com> | 2006-04-19 19:45:56 +0000 |
---|---|---|
committer | Anders Broman <anders.broman@ericsson.com> | 2006-04-19 19:45:56 +0000 |
commit | 746598b9e9206960b2e5e7249c53a8c9592dab1a (patch) | |
tree | d2786c66ea356bfac85f86c45b7b7415d67f0770 /epan/dissectors | |
parent | 02821ce8e5a6e0e21b34b440c8364f7e4cdc7bc4 (diff) |
From Martin mathieson:
While looking at bug 882 (which seems to have been fixed since 0.10.14) I noticed the RTCP heuristic dissector wasn't picking up the frame.
This patch fixes the parenthesis to test the packet-type properly.
svn path=/trunk/; revision=17914
Diffstat (limited to 'epan/dissectors')
-rw-r--r-- | epan/dissectors/packet-rtcp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-rtcp.c b/epan/dissectors/packet-rtcp.c index 79e830f9d8..d4cd32563c 100644 --- a/epan/dissectors/packet-rtcp.c +++ b/epan/dissectors/packet-rtcp.c @@ -523,7 +523,7 @@ dissect_rtcp_heur( tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree ) - allow BYE because this happens anyway - allow APP because TBCP ("PoC1") packets aren't compound... */ if (!((packet_type == RTCP_SR) || (packet_type == RTCP_RR) || - packet_type == RTCP_BYE) || (packet_type == RTCP_APP)) + (packet_type == RTCP_BYE) || (packet_type == RTCP_APP))) { return FALSE; } @@ -720,7 +720,7 @@ dissect_rtcp_app( tvbuff_t *tvb,packet_info *pinfo, int offset, proto_tree *tree * a SSRC field and SDES items, CNAME and MAY carry SDES item NAME to identify the * PoC Client that has been granted permission to send a Talk Burst. * The SDES item NAME SHALL be included if it is known by the PoC Server. - * Therefore the length of the packet will vary depending on numbwe of SDES items + * Therefore the length of the packet will vary depending on number of SDES items * and the size of the SDES items. */ if ( packet_len == 0 ) |