[skinny]: Add struct/union guards to multi-part message handlers

Improve multi-part message handling
- Calculate (sub) structure/union length
- Add a guard when iterating over multi-part message
- For fixed length messages, advance the cursor when iteration falls short of max length

Change-Id: Id2af3aa1a878328f105e173cabe2b68dd0343507
Reviewed-on: https://code.wireshark.org/review/20057
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>

1 files changed, 7 insertions, 4 deletions

diff --git a/epan/dissectors/packet-skinny.h.in b/epan/dissectors/packet-skinny.h.in
index e7c4528b0b..0ef076ab05 100644
--- a/epan/dissectors/packet-skinny.h.in
+++ b/epan/dissectors/packet-skinny.h.in
@@ -54,16 +54,19 @@ typedef struct _skinny_info_t
 {
   guint32       messId;
   guint32       maxProtocolVersion;
-  guint32       lineId;
+  gint32        lineId;
   guint32       callId;
   guint32       passThruId;
   const gchar * messageName;
   guint32       callState;
+  gboolean      hasCallInfo;
   gchar       * callingParty;
   gchar       * calledParty;
-  gboolean      hasCallInfo;
-  guint32       openreceiveStatus;
-  guint32       startmediatransmisionStatus;
+  gint32        mediaReceptionStatus;
+  gint32        mediaTransmissionStatus;
+  gint32        multimediaReceptionStatus;
+  gint32        multimediaTransmissionStatus;
+  gint32        multicastReceptionStatus;
 } skinny_info_t;
 
 /*