Useless Assignment at gtp_stat.c:108 |
No properties have been set. edit properties |
Jump to warning location ↓ | warning details... |
| |
gtpstat_packet (/home/sate/Testcases/c/cve/wireshark-1.2.0/gtk/gtp_stat.c)![]() | ||||||
![]() | ||||||
86 | gtpstat_packet(void *pgtp, packet_info *pinfo, epan_dissect_t *edt _U_, const void *psi) | |||||
87 | { | |||||
88 | const gtp_msg_hash_t *gtp=psi; | |||||
89 | gtpstat_t *fs=(gtpstat_t *)pgtp; | |||||
90 | int index=0; | |||||
91 | ||||||
92 | /* we are only interested in reply packets */ | |||||
93 | if(gtp->is_request){ | |||||
94 | return 0; | |||||
95 | } | |||||
96 | /* if we have not seen the request, just ignore it */ | |||||
97 | if(!gtp->req_frame){ | |||||
98 | return 0; | |||||
99 | } | |||||
100 | ||||||
101 | /* Only use the commands we know how to handle, this is not a comprehensive list */ | |||||
102 | /* Redoing the indexing is bit reduntant,*/ | |||||
103 | /* but using type as such would yield a long gtp_srt_table.*/ | |||||
104 | /* Only a fragtion of the are matchable req/resp pairs,*/ | |||||
105 | /* it's just doesn't feel feasible*/ | |||||
106 | ||||||
107 | switch(gtp->msgtype){ | |||||
108 | case GTP_MSG_ECHO_REQ: index=0;
| |||||
109 | break; | |||||
110 | case GTP_MSG_CREATE_PDP_REQ: index=1; | |||||
111 | break; | |||||
112 | case GTP_MSG_UPDATE_PDP_REQ: index=2; | |||||
113 | break; | |||||
114 | case GTP_MSG_DELETE_PDP_REQ: index=3; | |||||
115 | break; | |||||
116 | default: | |||||
117 | return 0; | |||||
118 | } | |||||
119 | ||||||
120 | add_srt_table_data(&fs->gtp_srt_table, index, >p->req_time, pinfo); | |||||
121 | ||||||
122 | return 1; | |||||
123 | } | |||||
![]() |