Commit Graph

66 Commits

Author SHA1 Message Date
Richard van der Hoff 25f64f9d27 fix a signedness-in-comparison warning
svn path=/trunk/; revision=22569
2007-08-21 20:32:22 +00:00
Jaap Keuter 7b593acb18 From Neil Piercy:
This patch set provides a an API for out of band signalling protocols to
register flows as SRTP/SRTCP using extended versions of the existing
rt(c)p_add_address functions. At present the encrypted portions of the payloads
are simply skipped, and the auth tags etc added as fields.

svn path=/trunk/; revision=22562
2007-08-21 07:23:34 +00:00
Stephen Fisher 2745f06d96 From Lars Ruoff:
find attached several trivial diffs for viewing RTP SSRC values in Hex
rather than Dec at various places in the UI.
Also includes change from BASE_DEC to BASE_HEX_DEC for corresponding RTP
and RTCP dissector header fields.


svn path=/trunk/; revision=22017
2007-05-31 18:41:25 +00:00
Martin Mathieson 8e98967075 When reporting roundtrip delay, show delay since timestamp of LSR frame.
svn path=/trunk/; revision=21985
2007-05-29 14:37:34 +00:00
Martin Mathieson 198f8f8185 Fix problem from previous commit (confused myself with FT_UINT_STRING needing length of 1 when adding to tree...), and show display-name in info column.
svn path=/trunk/; revision=21790
2007-05-15 15:26:13 +00:00
Martin Mathieson dc9745267b TBCP fixes:
- more careful checking of packet length
- skip padding at end of SDES items block
- formatting

svn path=/trunk/; revision=21789
2007-05-15 14:49:26 +00:00
Sebastien Tandel be81a9cabb fix gcc warning
svn path=/trunk/; revision=21416
2007-04-13 20:38:19 +00:00
Ulf Lamping 411249419d fix some more warnings (type casts)
svn path=/trunk/; revision=21141
2007-03-23 00:51:21 +00:00
Martin Mathieson 09f3f70a7a Try to squash a warning seen by win32 buildbot
svn path=/trunk/; revision=21074
2007-03-20 17:20:36 +00:00
Martin Mathieson 262de6677a Append ms calculation to DLSR field.
Show number of bytes derived from length field.

svn path=/trunk/; revision=20982
2007-03-06 13:37:39 +00:00
Anders Broman 7a615b9729 Add basic support for RFC4585.
svn path=/trunk/; revision=20916
2007-02-24 17:31:55 +00:00
Martin Mathieson b9f69234ad Check in trivial changes before generating a big diff...
svn path=/trunk/; revision=20304
2007-01-04 13:25:25 +00:00
Martin Mathieson 7e2db474c8 Fix for bug 1279 (Negative values for RTCP round trip delay
cannot be stored in guint32).
- Makes the threshold preference value an absolute value.
- There is now a separate expert info item for -ve roundtrips

N.B. There is still a problem with filtering -ve values on this
FT_INT32 field, i.e. rtcp.roundtrip-delay < 0 never matches with
frames that it should (even if rtcp.roundtrip-delay == -3 can
be used to match specific frames...).

svn path=/trunk/; revision=20264
2007-01-02 11:46:30 +00:00
Martin Mathieson 7a84ccbe0f TBCP changes:
- Fix 'ignored' flag that wasn't wide enough to take result
- Use FP_UINT_STRING in a couple of places

svn path=/trunk/; revision=20183
2006-12-21 17:05:42 +00:00
Martin Mathieson 9cc8175f5f Use value_string for info column. Also use fences to handle compound packets
svn path=/trunk/; revision=19872
2006-11-09 16:46:58 +00:00
Martin Mathieson 8dbd2e1f1d Don't try to report -ve roundtrip delays
svn path=/trunk/; revision=19778
2006-11-02 11:51:09 +00:00
Martin Mathieson e7323183d0 Try to declare and cast away warnings
svn path=/trunk/; revision=19709
2006-10-26 16:52:55 +00:00
Martin Mathieson 0812a4cb6f Report roundtrip delay as expert item
svn path=/trunk/; revision=19627
2006-10-20 14:05:18 +00:00
Martin Mathieson bd6931e2d3 Use FT_UINT_STRING for TBCP DENY reason-phrase
svn path=/trunk/; revision=18968
2006-08-21 11:41:32 +00:00
Martin Mathieson e2767088d8 Minor TBCP fixes
svn path=/trunk/; revision=18860
2006-08-09 17:23:10 +00:00
Martin Mathieson 9e1f690fc6 Update to TBCP ("PoC1" application). This is based on the "Approved Version 1.0 09 Jun 2006"
- updated to the current (approved) spec.  I'm not sure how backwards-compatible this is with older drafts...
- prettified the existing code, including more details in the info column

Also included is a fix to the way the offset at the end of an RTCP BYE packet is calculated (taking into account the NULL.  This avoids the 'length wrong' expert item) 

svn path=/trunk/; revision=18820
2006-08-02 16:00:07 +00:00
Luis Ontanon ed3b76044e as per RFC 3611 - 4.1.1 run length is 14 bits not 15.
Fixes Bug 1022


svn path=/trunk/; revision=18805
2006-07-28 16:40:46 +00:00
Jörg Mayer eb808f2cdb rtcp.profile-specific-extension was defined twice
svn path=/trunk/; revision=18400
2006-06-08 14:42:53 +00:00
Anders Broman 65aadc0d79 From Martin Mathieson:
- shows profile-specific extension data at the end of SR/RR reports (if 
packet length has not yet been reached after parsing normal data) and 
advances offset (further packets were not recognised+dissected as this 
data wasn't being skipped).
- checks that the length of the RTCP data in the whole frame matches the 
combined length from the length fields (the last check in RFC 3550, "A.2 
RTCP Header Validity Checks") with a generated field and expert info 
when wrong.
- reports the length field in all of the message types consistently (the 
length was confusingly shown multiplied by 4 only in APP packets...)

svn path=/trunk/; revision=18357
2006-06-06 18:26:57 +00:00
Anders Broman 9491711ab4 From Martin Mathieson:
- shows profile-specific extension data at the end of SR/RR reports (if 
packet length has not yet been reached after parsing normal data) and 
advances offset (further packets were not recognised+dissected as this 
data wasn't being skipped).

svn path=/trunk/; revision=18245
2006-05-29 18:16:23 +00:00
Ronnie Sahlberg feab79e328 change a whole bunch of ethereal into wireshark
svn path=/trunk/; revision=18196
2006-05-21 04:49:01 +00:00
Anders Broman 746598b9e9 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
2006-04-19 19:45:56 +00:00
Anders Broman f92ed70a3f From Martin Mathieson:
-  Also show vpi/vci in info column 

svn path=/trunk/; revision=17859
2006-04-13 22:30:20 +00:00
Jaap Keuter cdc0de3de5 Better read NTP LSW from the right spot
svn path=/trunk/; revision=17836
2006-04-06 20:47:22 +00:00
Jaap Keuter 0834b09ca2 From: Martin Mathieson
- fixes bug 863 (RTCP sender report's NTP time display incorrect - use using wrong bytes)
- makes NTP timestamp MSW and LSW proper filterable fields, displaying them in dec and hex
- when doing roundtrip calculations, rationalise fields added, add link to LSR frame whenever it matches

svn path=/trunk/; revision=17834
2006-04-06 13:30:45 +00:00
Anders Broman b05b62d64a From Andrei Emeltchenko:
OMA again has changed the format of the 2 RTCP packets:

RTCP Grant and RTCP Taken

In RTCP Grant instead of spare 2 bytes there are: magic number and item_len.

In RTCP Taken there is an extra field "Number of participants" like in Grant.
http://www.openmobilealliance.org/release_program/docs/CopyrightClick.asp?pck=PoC&file=V1_0-20051104-C/OMA-TS-PoC-UserPlane-V1_0-20051104-C.pdf

svn path=/trunk/; revision=17125
2006-01-30 05:35:39 +00:00
Anders Broman d74d5995e2 From Martin Mathieson
I keep finding finding traces that show new problems with this code.  This patch fixes 2 problems:
- I've seen RTCP frames containing a SR and RR with identical source info and the lsr matching the current MSW/LSW timestamp of the SR.  Don't want to do calculation without real roundtrip info
- calculating the gap between the 2 frames was still wrong (sigh)

svn path=/trunk/; revision=16934
2006-01-03 16:24:56 +00:00
Luis Ontanon ada9131e3b Fixes Bug 531 (Incorrect Interpretation of Bit vector and RLE RTCP XR Blocks).
svn path=/trunk/; revision=16921
2005-12-31 02:19:24 +00:00
Anders Broman 5b106306fd From Martin Mathieson:
there are only 1000 ms in one second

svn path=/trunk/; revision=16882
2005-12-22 16:26:26 +00:00
Guy Harris 9fea197bf9 Why duplicate the code of "tvb_get_string()" when you could just use
"tvb_get_string()"?

Why even bother with "tvb_get_string()" when you can just use
"proto_tree_add_item()" with a string item?

Make sure that the prefix in a PRIV item isn't bigger than the item
itself.  That fixes bug 603.

svn path=/trunk/; revision=16716
2005-12-07 03:03:37 +00:00
Ronnie Sahlberg a9a437d64a From Martin M various updates and fixes
Hi, Some tiddly changes: pppoe- don't create an empty discovery tags tree when the payload length is 0 chap- make chap.value work as a filterable field rtcp- append the packet type to the protocol tree name 


svn path=/trunk/; revision=16712
2005-12-07 00:35:16 +00:00
Ronnie Sahlberg e93abb6669 rtcp fix for sdes_prefix_string
svn path=/trunk/; revision=16656
2005-12-02 21:12:22 +00:00
Anders Broman f86d2d458c From Andrei Emeltchenko
I have added new RTCP POC fields as OMA standard says. With some additions. 

svn path=/trunk/; revision=16452
2005-11-09 21:50:39 +00:00
Guy Harris 6c521f90f5 The return value of tvb_get_ptr() is a const ptr; assign it to a const
ptr.

The answer to the question
"??????????????????????????????????????????????????????????????????" is
"No - the return value of tvb_get_ptr() is a reference, not an allocated
copy, and it cannot be freed and doesn't need to be freed."

svn path=/trunk/; revision=16426
2005-11-07 21:10:52 +00:00
Anders Broman 708d793e63 Display MSW and LSW as NTP timestamp as well.
svn path=/trunk/; revision=16421
2005-11-07 19:40:55 +00:00
Ronnie Sahlberg 8b32f13476 get rid of strcpy in packet-ntp.c
at the same time  change ntp_fmt_ts to return a pointer to ian ep-allocated buffer.
remove the redundant buffer parameter in the signature and change all callers.


svn path=/trunk/; revision=15939
2005-09-21 20:05:13 +00:00
Ronnie Sahlberg 6caf6d5d1c add missing value_string termination
svn path=/trunk/; revision=15584
2005-08-28 03:01:37 +00:00
Ulf Lamping 6f43fbb2f0 EVERYTHING IN THE BUILDBOT IS GOING TO BE RED!!! Sorry!
I've done more than a day to change the timestamp resolution from microseconds to nanoseconds. As I really don't want to loose those changes, I'm going to check in the changes I've done so far. Hopefully someone else will give me a helping hand with the things left ...

What's done: I've changed the timestamp resolution from usec to nsec in almost any place in the sources. I've changed parts of the implementation in nstime.s/.h and a lot of places elsewhere.

As I don't understand the editcap source (well, I'm maybe just too tired right now), hopefully someone else might be able to fix this soon.

Doing all those changes, we get native nanosecond timestamp resolution in Ethereal. After fixing all the remaining issues, I'll take a look how to display this in a convenient way...

As I've also changed the wiretap timestamp resolution from usec to nsec we might want to change the wiretap version number...

svn path=/trunk/; revision=15520
2005-08-24 21:31:56 +00:00
Gerald Combs ef81f7d060 Properly initialize our hf_ arrays. Make hf_rtcp_app_poc1_conn_session_type
an FT_UINT8 instead of an FT_BOOLEAN.  This keeps the Solaris builder from
crashing while running "-G values".

svn path=/trunk/; revision=15519
2005-08-24 20:34:15 +00:00
Anders Broman bdec656b1a From Timo Metsala:
Updates the  PoC floor control parsing to comply to the latest version of the 
specification. 

svn path=/trunk/; revision=15511
2005-08-22 16:26:29 +00:00
Ronnie Sahlberg d7cf8bfc97 remove g_malloc()s from rtcp
svn path=/trunk/; revision=15382
2005-08-17 08:54:25 +00:00
Ronnie Sahlberg 234f09c663 more gmemchunk -> se_alloc() improvements
svn path=/trunk/; revision=15312
2005-08-12 10:50:55 +00:00
Ronnie Sahlberg ec3ab9ec3e rename ep_tvb_get_string() to tvb_get_ephemeral_string() asnd update the documentation in README.developer
svn path=/trunk/; revision=15270
2005-08-10 13:41:13 +00:00
Jörg Mayer 5602f5bb83 Make some funtions static. More char -> const char warning fixes
svn path=/trunk/; revision=15146
2005-07-30 01:01:02 +00:00
Ronnie Sahlberg 9ef6568c56 some more dissectors converted to ep_tvb_get_string
again   at least one obvious memleak was found in the old code



svn path=/trunk/; revision=15031
2005-07-24 03:01:41 +00:00