2 VS Code Analyzer warnings were bogus. Avoid one and add a comment about the second...
authormartinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 16 Mar 2011 22:51:45 +0000 (22:51 +0000)
committermartinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 16 Mar 2011 22:51:45 +0000 (22:51 +0000)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@36201 f5534014-38df-0310-8fa8-9805f1628bb7

epan/dissectors/packet-umts_fp.c

index 42785362e1f112e03ed7d616cf54c1c50e4d9c1a..0df4b39b88401ed18c026d4f7937dc4fb56a47dc 100644 (file)
@@ -1398,7 +1398,7 @@ static void dissect_rach_channel_info(tvbuff_t *tvb, packet_info *pinfo, proto_t
                                     break;
                                 default:
                                     /* Not defined */
-                                    proto_tree_add_item(new_ie_flags_tree, hf_fp_rach_new_ie_flag_unused[n],
+                                    proto_tree_add_item(new_ie_flags_tree, hf_fp_rach_new_ie_flag_unused[6],
                                                         tvb, offset, 1, FALSE);
                                     break;
                             }
@@ -1430,6 +1430,7 @@ static void dissect_rach_channel_info(tvbuff_t *tvb, packet_info *pinfo, proto_t
 
                         default:
                             /* No defined meanings */
+                            /* Visual Studio Code Analyzer wrongly thinks n can be 7 here. It can't */
                             proto_tree_add_item(new_ie_flags_tree, hf_fp_rach_new_ie_flag_unused[n],
                                                 tvb, offset, 1, FALSE);
                             break;