diff options
| author | Bill Meier <wmeier@newsguy.com> | 2011-05-20 15:34:29 +0000 |
|---|---|---|
| committer | Bill Meier <wmeier@newsguy.com> | 2011-05-20 15:34:29 +0000 |
| commit | 4910bc142fcf8072e26179a4fe77c257ecc20225 (patch) | |
| tree | d8abd6fb9c92478992de3a29f6472ea1fd37d64d /epan/dissectors/packet-bt-utp.c | |
| parent | befd2a6f744da5fbdf9625374b3299cd01b6f71d (diff) | |
Add to comment about using a heuristic; Fix a typo.
svn path=/trunk/; revision=37330
Diffstat (limited to 'epan/dissectors/packet-bt-utp.c')
| -rw-r--r-- | epan/dissectors/packet-bt-utp.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/epan/dissectors/packet-bt-utp.c b/epan/dissectors/packet-bt-utp.c index 2c003269ec..4bfcc28ef8 100644 --- a/epan/dissectors/packet-bt-utp.c +++ b/epan/dissectors/packet-bt-utp.c @@ -177,8 +177,10 @@ utp_is_v1(tvbuff_t *tvb) { * So: Also do some length checking: * The length of "V1" frames should be 20, 26, 30, 34, 36, 38, ... * fixed hdr len: 20 - * extension9s) len: 6, 10, 14, 16, 18, 20, ... + * extension(s) len: 6, 10, 14, 16, 18, 20, ... * XXX: this is a hack and should be replaced !! + * In fact this heuristic probably fails for some SF_DATA packets since they + * presumably can contain a variable number of data bytes after the header. */ len = tvb_reported_length(tvb); if (len < V1_FIXED_HDR_SIZE) { |