diff options
| author | Guy Harris <guy@alum.mit.edu> | 2012-01-16 08:10:49 +0000 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2012-01-16 08:10:49 +0000 |
| commit | 7e4c02ee18afbd89c758e555c0135e1898e243ca (patch) | |
| tree | 5fd72452681b94ecffd9953b74e6ac9fa80d596f /epan/dissectors/packet-rtmpt.c | |
| parent | cc0c99a01f6938d9d6d07c4682810a49097b77d1 (diff) | |
The frame number is an unsigned integer, so there's no guarantee that it
won't equal (guint32)-1.
Fortunately, frame numbers are 1-based, so there *is* a guarantee that
it won't equal 0.
svn path=/trunk/; revision=40544
Diffstat (limited to 'epan/dissectors/packet-rtmpt.c')
| -rw-r--r-- | epan/dissectors/packet-rtmpt.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/packet-rtmpt.c b/epan/dissectors/packet-rtmpt.c index f574fd5480..d056c5804c 100644 --- a/epan/dissectors/packet-rtmpt.c +++ b/epan/dissectors/packet-rtmpt.c @@ -1092,7 +1092,7 @@ dissect_rtmpt(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, rtmpt_conv_t proto_tree *rtmpt_tree = NULL; proto_tree *rtmptroot_tree = NULL; proto_item *ti = NULL; - static gint iPreviousFrameNumber = -1; + static guint iPreviousFrameNumber = 0; gint offset = 0; gchar *sDesc = NULL; @@ -1103,12 +1103,12 @@ dissect_rtmpt(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, rtmpt_conv_t col_set_str(pinfo->cinfo, COL_PROTOCOL, "RTMP"); - RTMPT_DEBUG("Dissect: frame=%d prev=%d visited=%d len=%d col=%d tree=%p\n", + RTMPT_DEBUG("Dissect: frame=%u prev=%u visited=%d len=%d col=%d tree=%p\n", pinfo->fd->num, iPreviousFrameNumber, pinfo->fd->flags.visited, tvb_length_remaining(tvb, offset), check_col(pinfo->cinfo, COL_INFO), tree); /* This is a trick to know whether this is the first PDU in this packet or not */ - if (iPreviousFrameNumber != (gint) PINFO_FD_NUM(pinfo)) + if (iPreviousFrameNumber != PINFO_FD_NUM(pinfo)) col_clear(pinfo->cinfo, COL_INFO); else col_append_str(pinfo->cinfo, COL_INFO, " | "); |