Fix some Dead Store (Dead assignement/Dead increment) Warning found by Clang
authoralagoutte <alagoutte@f5534014-38df-0310-8fa8-9805f1628bb7>
Fri, 13 Jan 2012 16:37:21 +0000 (16:37 +0000)
committeralagoutte <alagoutte@f5534014-38df-0310-8fa8-9805f1628bb7>
Fri, 13 Jan 2012 16:37:21 +0000 (16:37 +0000)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@40475 f5534014-38df-0310-8fa8-9805f1628bb7

plugins/ethercat/packet-ams.c

index 7a0f50261856bc9b6b09321678c4d7ed7ca1521d..228c4e6d124bfd4942c59aa05248a3eaceca4090 100644 (file)
@@ -375,7 +375,7 @@ static void NetIdFormater(tvbuff_t *tvb, guint offset, char *szText, gint nMax)
 
 
 /*ams*/
 
 
 /*ams*/
-static void dissect_ams(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
+static gint dissect_ams(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 {
    proto_item *ti, *anItem;
    proto_tree *ams_tree = NULL, *ams_adstree, *ams_statetree;
 {
    proto_item *ti, *anItem;
    proto_tree *ams_tree = NULL, *ams_adstree, *ams_statetree;
@@ -394,14 +394,14 @@ static void dissect_ams(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
    if( pinfo->ethertype != 0x88a4 )
    {
       if( TcpAdsParserHDR_Len > ams_length )
    if( pinfo->ethertype != 0x88a4 )
    {
       if( TcpAdsParserHDR_Len > ams_length )
-         return;
+         return offset;
       ams_length -= TcpAdsParserHDR_Len;
 
       offset = TcpAdsParserHDR_Len;
    }
 
    if( ams_length < AmsHead_Len )
       ams_length -= TcpAdsParserHDR_Len;
 
       offset = TcpAdsParserHDR_Len;
    }
 
    if( ams_length < AmsHead_Len )
-      return;
+      return offset;
 
   if (tree)
   {
 
   if (tree)
   {
@@ -829,7 +829,7 @@ static void dissect_ams(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
     if( tree && ams_length-offset > 0 )
         proto_tree_add_item(ams_tree, hf_ams_data, tvb, offset, ams_length-offset, ENC_NA);
   }
     if( tree && ams_length-offset > 0 )
         proto_tree_add_item(ams_tree, hf_ams_data, tvb, offset, ams_length-offset, ENC_NA);
   }
-
+return offset;
 }
 
 
 }
 
 
@@ -1197,7 +1197,7 @@ void proto_register_ams(void)
    proto_register_field_array(proto_ams, hf, array_length(hf));
    proto_register_subtree_array(ett, array_length(ett));
 
    proto_register_field_array(proto_ams, hf, array_length(hf));
    proto_register_subtree_array(ett, array_length(ett));
 
-   register_dissector("ams", dissect_ams, proto_ams);
+   new_register_dissector("ams", dissect_ams, proto_ams);
 }
 
 /* The registration hand-off routing */
 }
 
 /* The registration hand-off routing */