Redundant Condition at packet-range.c:226 |
No properties have been set. edit properties |
Jump to warning location ↓ | warning details... |
| |
packet_range_process_packet (/home/sate/Testcases/c/cve/wireshark-1.2.0/packet-range.c)![]() | ||||||
![]() | ||||||
196 | range_process_e packet_range_process_packet(packet_range_t *range, frame_data *fdata) { | |||||
197 | ||||||
198 | switch(range->process) { | |||||
199 | case(range_process_all): | |||||
200 | break; | |||||
201 | case(range_process_selected): | |||||
202 | if (range->selected_done) { | |||||
203 | return range_processing_finished; | |||||
204 | } | |||||
205 | if (fdata->num != cfile.current_frame->num) { | |||||
206 | return range_process_next; | |||||
207 | } | |||||
208 | range->selected_done = TRUE; | |||||
209 | break; | |||||
210 | case(range_process_marked): | |||||
211 | if (fdata->flags.marked == FALSE) { | |||||
212 | return range_process_next; | |||||
213 | } | |||||
214 | break; | |||||
215 | case(range_process_marked_range): | |||||
216 | if (range->marked_range_left == 0) { | |||||
217 | return range_processing_finished; | |||||
218 | } | |||||
219 | if (fdata->flags.marked == TRUE) { | |||||
220 | range->marked_range_active = TRUE; | |||||
221 | } | |||||
222 | if (range->marked_range_active == FALSE ) { | |||||
223 | return range_process_next; | |||||
224 | } | |||||
225 | if (!range->process_filtered || | |||||
226 | (range->process_filtered && fdata->flags.passed_dfilter == TRUE))
| |||||
227 | { | |||||
228 | range->marked_range_left--; | |||||
229 | } | |||||
230 | break; | |||||
231 | case(range_process_user_range): | |||||
232 | if (value_is_in_range(range->user_range, fdata->num) == FALSE) { | |||||
233 | return range_process_next; | |||||
234 | } | |||||
235 | break; | |||||
236 | default: | |||||
237 | g_assert_not_reached(); | |||||
238 | } | |||||
239 | ||||||
240 | /* this packet has to pass the display filter but didn't? -> try next */ | |||||
241 | if (range->process_filtered && fdata->flags.passed_dfilter == FALSE) { | |||||
242 | return range_process_next; | |||||
243 | } | |||||
244 | ||||||
245 | /* We fell through the conditions above, so we accept this packet */ | |||||
246 | return range_process_this; | |||||
247 | } | |||||
![]() |