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.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@40543 f5534014-38df-0310-8fa8-9805f1628bb7

diff --git a/epan/dissectors/packet-mq.c b/epan/dissectors/packet-mq.c
index 4b1a5642bd900c44f9e65ed57ac63527fb09b6e5..420edf81b8d8efeed97db34bbe9216cf252111c7 100644 (file)
--- a/epan/dissectors/packet-mq.c
+++ b/epan/dissectors/packet-mq.c
@@ -1246,13 +1246,13 @@ dissect_mq_pdu(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
     gboolean bEBCDIC = FALSE;
     gint iDistributionListSize = 0;
     struct mq_msg_properties tMsgProps;
-    static gint iPreviousFrameNumber = -1;
+    static guint iPreviousFrameNumber = 0;
 
     col_set_str(pinfo->cinfo, COL_PROTOCOL, "MQ");
     if (check_col(pinfo->cinfo, COL_INFO))
     {
         /* This is a trick to know whether this is the first PDU in this packet or not */
-        if (iPreviousFrameNumber != (gint) pinfo->fd->num)
+        if (iPreviousFrameNumber != pinfo->fd->num)
             col_clear(pinfo->cinfo, COL_INFO);
         else
             col_append_str(pinfo->cinfo, COL_INFO, " | ");