Text   |  XML   |  ReML   |   Visible Warnings:

Redundant Condition  at netscreen.c:169

No properties have been set. | edit properties
Jump to warning location ↓ warning details...
Show Events | Options

netscreen_check_file_type

(/home/sate/Testcases/c/cve/wireshark-1.2.0/wiretap/netscreen.c)expand/collapse
Show more  
 157  static gboolean netscreen_check_file_type(wtap *wth, int *err)
 158  {
 159          char    buf[NETSCREEN_LINE_LENGTH];
 160          guint   reclen, line;
 161   
 162          buf[NETSCREEN_LINE_LENGTH-1] = '\0';
 163   
 164          for (line = 0; line < NETSCREEN_HEADER_LINES_TO_CHECK; line++) {
 165                  if (file_gets(buf, NETSCREEN_LINE_LENGTH, wth->fh) != NULL) {
 166   
 167                          reclen = (guint) strlen(buf);
 168                          if (reclen < strlen(NETSCREEN_HDR_MAGIC_STR1) ||
 169                                  reclen < strlen(NETSCREEN_HDR_MAGIC_STR2)) {
 170                                  continue;
 171                          }
 172   
 173                          if (strstr(buf, NETSCREEN_HDR_MAGIC_STR1) ||
 174                              strstr(buf, NETSCREEN_HDR_MAGIC_STR2)) {
 175                                  return TRUE;
 176                          }
 177                  } else {
 178                          /* EOF or error. */
 179                          if (file_eof(wth->fh))
 180                                  *err = 0;
 181                          else 
 182                                  *err = file_error(wth->fh);
 183                          return FALSE;
 184                  }
 185          }
 186          *err = 0;
 187          return FALSE;
 188  }
Show more  




Change Warning 1022.29686 : Redundant Condition

Priority:
State:
Finding:
Owner:
Note: