Removed some unneeded assignments.
authorstig <stig@f5534014-38df-0310-8fa8-9805f1628bb7>
Mon, 18 Apr 2011 15:29:25 +0000 (15:29 +0000)
committerstig <stig@f5534014-38df-0310-8fa8-9805f1628bb7>
Mon, 18 Apr 2011 15:29:25 +0000 (15:29 +0000)
Found by clang.

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

15 files changed:
epan/dissectors/packet-btamp.c
epan/dissectors/packet-btobex.c
epan/dissectors/packet-cisco-wids.c
epan/dissectors/packet-dplay.c
epan/dissectors/packet-dtls.c
epan/dissectors/packet-isup.c
epan/dissectors/packet-memcache.c
epan/dissectors/packet-mgcp.c
epan/dissectors/packet-nfs.c
epan/dissectors/packet-rpc.c
epan/dissectors/packet-scsi.c
epan/dissectors/packet-smb-common.c
epan/dissectors/packet-ssh.c
epan/dissectors/packet-ucp.c
epan/dissectors/packet-windows-common.c

index f997632c6cded01f7fa5a01993b9d69ff6ba46c8..06cc59e56ae4580007fca3afc2de30aa56e341cf 100644 (file)
@@ -208,7 +208,7 @@ dissect_discoverresponse(tvbuff_t *tvb, int offset, packet_info *pinfo, proto_tr
             "Controller list");
     btamp_controller_list_tree=proto_item_add_subtree(ti_controller_list, ett_btamp_controller_list);
 
-    while ((length = tvb_length_remaining(tvb, offset)) >= 3) {
+    while (tvb_length_remaining(tvb, offset) >= 3) {
         offset = dissect_controller_entry(tvb, offset, pinfo, btamp_controller_list_tree, idx);
         ++idx;
     }
@@ -231,7 +231,7 @@ dissect_changenotify(tvbuff_t *tvb, int offset, packet_info *pinfo, proto_tree *
             "Controller list");
     btamp_controller_list_tree=proto_item_add_subtree(ti_controller_list, ett_btamp_controller_list);
 
-    while ((length = tvb_length_remaining(tvb, offset)) >= 3) {
+    while (tvb_length_remaining(tvb, offset) >= 3) {
         offset = dissect_controller_entry(tvb, offset, pinfo, btamp_controller_list_tree, idx);
         ++idx;
     }
index 07ad3c231df355385591ed0eb3bc6926c5313369..e30a265a898fc930c445f07f424b8d1ebed7edf9 100644 (file)
@@ -271,7 +271,7 @@ display_unicode_string(tvbuff_t *tvb, proto_tree *tree, int offset, char **data)
        * Get the length of the string.
        */
        len = 0;
-       while ((character = tvb_get_ntohs(tvb, offset + len)) != '\0')
+       while (tvb_get_ntohs(tvb, offset + len) != '\0')
                len += 2;
 
        len += 2;   /* count the '\0' too */
index a1d07150ddd0ba8719c6d0da036e355a6320c5ed..c04222fe1c9d5dd66dbb04807d7bb64209e63a34 100644 (file)
@@ -77,7 +77,6 @@ dissect_cwids(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
        proto_tree *ti, *cwids_tree;
        volatile int offset = 0;
        guint16 capturelen;
-       guint remain;
        void *pd_save;
 
        col_set_str(pinfo->cinfo, COL_PROTOCOL, "CWIDS");
@@ -86,7 +85,7 @@ dissect_cwids(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 
        cwids_tree = NULL;
 
-       while((remain = tvb_length_remaining(tvb, offset)) > 0) {
+       while(tvb_length_remaining(tvb, offset) > 0) {
                ti = proto_tree_add_item(tree, proto_cwids, tvb, offset, 28, FALSE);
                cwids_tree = proto_item_add_subtree(ti, ett_cwids);
 
index cc0d6f030c29c41728aec06e83013b703ababcbb..08f402c3b3125f388f20a5e88a374e3def586210 100644 (file)
@@ -420,7 +420,7 @@ static gint display_unicode_string(proto_tree *tree, gint hf_index, tvbuff_t *tv
      * if we don't find the '\0'?  I think it's a feature.
      */
     len = 0;
-    while ((character = tvb_get_letohs(tvb, offset + len)) != '\0')
+    while (tvb_get_letohs(tvb, offset + len) != '\0')
         len += 2;
     len += 2;   /* count the '\0' too */
 
index eea7341b95966999800058d667be6c47f1be2d69..2ee81ca2a0f593d03e926dbbb0375948fa5f9f9a 100644 (file)
@@ -465,7 +465,6 @@ decrypt_dtls_record(tvbuff_t *tvb, packet_info *pinfo, guint32 offset,
                     gboolean save_plaintext)
 {
   gint ret;
-  gint direction;
   SslDecoder* decoder;
   ret = 0;
 
@@ -479,7 +478,7 @@ decrypt_dtls_record(tvbuff_t *tvb, packet_info *pinfo, guint32 offset,
   }
 
   /* retrive decoder for this packet direction */
-  if ((direction = ssl_packet_from_server(ssl, dtls_associations, pinfo)) != 0) {
+  if (ssl_packet_from_server(ssl, dtls_associations, pinfo) != 0) {
     ssl_debug_printf("decrypt_dtls_record: using server decoder\n");
     decoder = ssl->server;
   }
index e9b1147441a234c9221075caf9fd339755d01772..ad08f5c586e27ad142615ea6bf29502966d6fb23 100644 (file)
@@ -2621,9 +2621,8 @@ dissect_isup_circuit_state_ind_parameter(tvbuff_t *parameter_tvb, proto_tree *pa
   guint8 circuit_state;
   gint offset=0;
   gint i=0;
-  gint length;
 
-  while((length = tvb_reported_length_remaining(parameter_tvb, offset)) > 0){
+  while(tvb_reported_length_remaining(parameter_tvb, offset) > 0){
     circuit_state_item = proto_tree_add_text(parameter_tree, parameter_tvb,
                                              offset, -1,
                                              "Circuit# CIC+%u state", i);
index 57a638b5b0c6bd21edd277b050c7270b30a911eb..83b09440e145248e222790b57d8909cb960a120a 100644 (file)
@@ -921,7 +921,6 @@ content_data_dissector (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, int
                        int content_length, guint8 opcode)
 {
   gint          datalen;
-  int           len;
   gboolean      short_pkt = FALSE;
 
   /*
@@ -929,7 +928,7 @@ content_data_dissector (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, int
    * the buffer. It is not necessary that we have all the
    * content_length bytes available to read.
    */
-  if ((len = tvb_reported_length_remaining (tvb, offset) != 0)) {
+  if (tvb_reported_length_remaining (tvb, offset) != 0) {
     /* bytes actually remaining in this tvbuff. */
     datalen = tvb_length_remaining (tvb, offset);
     if (content_length >= 0) {
@@ -1000,14 +999,14 @@ static int
 incr_dissector (tvbuff_t *tvb, proto_tree *tree, int offset)
 {
   gint           next_offset;
-  int            linelen, len;
+  int            linelen;
   const guchar  *line, *lineend;
 
   const guchar  *next_token;
   int            tokenlen;
 
   /* expecting to read 'bytes' number of bytes from the buffer. */
-  if ((len = tvb_reported_length_remaining (tvb, offset)) != 0) {
+  if (tvb_reported_length_remaining (tvb, offset) != 0) {
     /* Find the end of the line. */
     linelen = tvb_find_line_end (tvb, offset,
                                  tvb_ensure_length_remaining (tvb, offset), &next_offset,
@@ -1057,11 +1056,10 @@ stat_dissector (tvbuff_t *tvb, proto_tree *tree, int offset)
   gint          next_offset;
   const guchar *next_token;
   const guchar *line, *lineend;
-  int           reported_datalen = -1;
   guint32       slabclass;
   guchar        response_chars[21];
 
-  while ((reported_datalen = tvb_reported_length_remaining (tvb, offset)) != 0) {
+  while (tvb_reported_length_remaining (tvb, offset) != 0) {
     /* Find the end of the line. */
     linelen = tvb_find_line_end (tvb, offset,
                                  tvb_ensure_length_remaining (tvb, offset), &next_offset,
@@ -1167,7 +1165,7 @@ static int
 get_response_dissector (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, int offset)
 {
   gint           next_offset;
-  int            linelen, len;
+  int            linelen;
   const guchar  *line, *lineend;
   const guchar  *next_token;
   int            tokenlen;
@@ -1178,7 +1176,7 @@ get_response_dissector (tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, int
   gchar          response_chars[21]; /* cover uint64 (20 + 1) bytes*/
 
   /* expecting to read 'bytes' number of bytes from the buffer. */
-  while ((len = tvb_reported_length_remaining (tvb, offset)) != 0) {
+  while (tvb_reported_length_remaining (tvb, offset) != 0) {
     /* Find the end of the line. */
     linelen = tvb_find_line_end (tvb, offset,
                                  tvb_ensure_length_remaining (tvb, offset), &next_offset,
index 34e5b44fcd9b7029a3a61597b65e3c69ba3dcbda..c243873463d51a3c6347ecc36d1642783a8165f3 100644 (file)
@@ -1338,7 +1338,7 @@ static gint tvb_parse_param(tvbuff_t* tvb, gint offset, gint len, int** hf)
 
                /* Add a recognised parameter type if we have one */
                if (*hf != NULL && len > (tvb_current_offset - offset) &&
-                   (tempchar = tvb_get_guint8(tvb,tvb_current_offset)) == ':')
+                   tvb_get_guint8(tvb,tvb_current_offset) == ':')
                {
                        tvb_current_offset++;
                        tvb_current_offset = tvb_skip_wsp(tvb,tvb_current_offset, (len - tvb_current_offset + offset));
index d58d4fb1fee2141d825cb17ce862dda7fed100e8..4a605d07ff45254e6b8dd930c1ab3de5d15013fa 100644 (file)
@@ -4923,7 +4923,7 @@ dissect_access_reply(tvbuff_t *tvb, int offset, packet_info *pinfo, proto_tree*
        acc_req = civ->private_data;
        /* Should never happen because ONC-RPC requires the call in order to dissect the reply. */
        if (acc_req==NULL) {
-               return offset+=4;
+               return offset+4;
        }
        if(nfsv4) {
                acc_supp = tvb_get_ntohl(tvb, offset+0);
@@ -4967,7 +4967,7 @@ dissect_access_reply(tvbuff_t *tvb, int offset, packet_info *pinfo, proto_tree*
                                offset, 4, (mask_denied > 0 ? TRUE : FALSE ));
        PROTO_ITEM_SET_GENERATED(ditem);
 
-       return offset+=4;
+       return offset+4;
 }
 
 /* RFC 1813, Page 40..43 */
index a674997862503d602db8a0f9831cf40b2077727c..40c272e4b179d1703370aa73ea088b24cf481c9e 100644 (file)
@@ -1646,15 +1646,12 @@ dissect_rpc_continuation(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
 static void 
 make_fake_rpc_prog_if_needed (rpc_prog_info_key *prpc_prog_key, guint prog_ver)
 {
-
-       rpc_prog_info_value *rpc_prog = NULL;
-
        /* sanity check: no one uses versions > 10 */
        if(prog_ver>10){
                return;
        }
 
-       if( (rpc_prog = g_hash_table_lookup(rpc_progs, prpc_prog_key)) == NULL) {
+       if(g_hash_table_lookup(rpc_progs, prpc_prog_key) == NULL) {
                /* ok this is not a known rpc program so we
                 * will have to fake it.
                 */
index c13e49d262a2d1e43ca7a74f0fe08c1a9b1171f9..026fb035beb2007e1e131c1e16deae5ca559e8b9 100644 (file)
@@ -3645,7 +3645,6 @@ dissect_scsi_pagecode (tvbuff_t *tvb, packet_info *pinfo _U_,
                        scsi_task_data_t *cdata)
 {
     guint8 pcode;
-    const gchar *valstr;
     int hf_pagecode;
 
     /* unless we have cdata there is not much point in continuing */
@@ -3653,8 +3652,8 @@ dissect_scsi_pagecode (tvbuff_t *tvb, packet_info *pinfo _U_,
         return;
 
     pcode = tvb_get_guint8 (tvb, offset);
-    if ((valstr = match_strval (pcode & SCSI_MS_PCODE_BITS,
-                                scsi_spc_modepage_val)) == NULL) {
+    if (match_strval (pcode & SCSI_MS_PCODE_BITS,
+                                scsi_spc_modepage_val) == NULL) {
         /*
          * This isn't a generic mode page that applies to all SCSI
          * device types; try to interpret it based on what we deduced,
index dba733a81abdedbbb025ca99f42112ef63fa9d0c..0f76486d6c54a6bc5adb60dc83dec83211bdbe4b 100644 (file)
@@ -94,7 +94,7 @@ int display_unicode_string(tvbuff_t *tvb, proto_tree *tree, int offset, int hf_i
         * if we don't find the '\0'?  I think it's a feature.
         */
        len = 0;
-       while ((character = tvb_get_letohs(tvb, offset + len)) != '\0')
+       while (tvb_get_letohs(tvb, offset + len) != '\0')
                len += 2;
        len += 2;       /* count the '\0' too */
 
index 93d7701a323363189968782bd91c62d5fafa30a9..d58e6f31375fdf0bb1f1a4b673ce260ac2343457 100644 (file)
@@ -363,7 +363,7 @@ dissect_ssh(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree)
                return;
        }
 
-       while((remain_length = tvb_reported_length_remaining(tvb,offset))> 0 ) {
+       while(tvb_reported_length_remaining(tvb,offset)> 0 ) {
                need_desegmentation = FALSE;
                last_offset = offset;
                this_number = this_data->counter+number;
index 1e876fed15bf71bc9e4fff85d767b99fa6551c40..ece530bad4c4f27fdf3be945ebdc22ce08a29d0b 100644 (file)
@@ -1631,12 +1631,12 @@ add_5xO(proto_tree *tree, tvbuff_t *tvb)
     UcpHandleByte(hf_ucp_parm_DCs);
     UcpHandleByte(hf_ucp_parm_MCLs);
     UcpHandleByte(hf_ucp_parm_RPI);
-    if ((intval = tvb_get_guint8(tvb, offset++)) != '/') {
+    if (tvb_get_guint8(tvb, offset++) != '/') {
         proto_tree_add_string(tree, hf_ucp_parm_CPg, tvb, offset - 1,1,
                               "(reserved for Code Page)");
         offset++;
     }
-    if ((intval = tvb_get_guint8(tvb, offset++)) != '/') {
+    if (tvb_get_guint8(tvb, offset++) != '/') {
         proto_tree_add_string(tree, hf_ucp_parm_RPLy, tvb, offset - 1,1,
                               "(reserved for Reply type)");
         offset++;
index 4b738cac1ec9ff5073df2385326f45c2a40b1e17..0a4b48ab79323c1ffe0fad4d051c686bc9d56f3a 100644 (file)
@@ -1515,7 +1515,7 @@ dissect_nt_sid(tvbuff_t *tvb, int offset, proto_tree *parent_tree,
        if(hf_sid==-1){
        /* if no tree, just return the offset of the end_of_SID+1 */
        if (!parent_tree)
-               return(offset+=(6+(num_auth*4)));
+               return(offset+(6+(num_auth*4)));
 
                hf_sid=hf_nt_sid;
        }