aboutsummaryrefslogtreecommitdiffstats
path: root/plugins
diff options
context:
space:
mode:
authorAlexis La Goutte <alexis.lagoutte@gmail.com>2012-06-04 22:25:10 +0000
committerAlexis La Goutte <alexis.lagoutte@gmail.com>2012-06-04 22:25:10 +0000
commit7174762b51375c0c95a7b3dd8108904f9a90afca (patch)
tree76f7c57ee4d369b7e8fb4220e0427d08b776254a /plugins
parentcb2000b9f311aef69fe1f701c7fbae92011b3663 (diff)
Fix some Dead Store (Dead assignement/Dead increment) Warning found by Clang
svn path=/trunk/; revision=43088
Diffstat (limited to 'plugins')
-rw-r--r--plugins/unistim/packet-unistim.c60
1 files changed, 59 insertions, 1 deletions
diff --git a/plugins/unistim/packet-unistim.c b/plugins/unistim/packet-unistim.c
index 7991dbee4f..c19031ba27 100644
--- a/plugins/unistim/packet-unistim.c
+++ b/plugins/unistim/packet-unistim.c
@@ -506,7 +506,10 @@ dissect_unistim_message(proto_tree *unistim_tree,packet_info *pinfo,tvbuff_t *tv
offset+=(msg_len-2);
}
-
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
return offset;
}
@@ -594,6 +597,11 @@ dissect_basic_phone(proto_tree *msg_tree,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
/*DONE*/
@@ -666,6 +674,11 @@ dissect_basic_switch(proto_tree *msg_tree,
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
@@ -739,6 +752,11 @@ dissect_broadcast_switch(proto_tree *msg_tree,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
/*DONE Haven't seen any phone broadcasts, wouldn't expect to*/
@@ -1382,6 +1400,11 @@ dissect_display_switch(proto_tree *msg_tree,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
/*DONE*/
@@ -1488,6 +1511,11 @@ dissect_display_phone(proto_tree *msg_tree,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
@@ -1601,6 +1629,11 @@ dissect_key_indicator_switch(proto_tree *msg_tree,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
@@ -1690,6 +1723,11 @@ dissect_key_indicator_phone(proto_tree *msg_tree,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
@@ -1829,6 +1867,11 @@ dissect_network_switch(proto_tree *msg_tree,
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
@@ -2007,6 +2050,11 @@ dissect_network_phone(proto_tree *msg_tree,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
/*DONE*/
@@ -2436,6 +2484,11 @@ dissect_audio_switch(proto_tree *msg_tree,packet_info *pinfo,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}
@@ -2655,6 +2708,11 @@ dissect_audio_phone(proto_tree *msg_tree,
offset+=msg_len;
}
+ if(msg_len){
+ /* TODO: add Expert info to indicate there is unknown data !
+ For the moment, this code only remove Clang Warnings about not used msg_len... */
+ }
+
return offset;
}