Text   |  XML   |  ReML   |   Visible Warnings:

Redundant Condition  at osi-utils.c:126

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

print_system_id_buf

(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/osi-utils.c)expand/collapse
Show more  
 94  print_system_id_buf( const guint8 *ad, int length, gchar *buf, int buf_len)
 95  {
 96    gchar        *cur;
 97    int           tmp;
 98   
 99    if ( ( length <= 0 ) || ( length > MAX_SYSTEMID_LEN ) ) {
 100      g_snprintf(buf, buf_len, "<Invalid length of SYSTEM ID>");
 101      return;
 102    }
 103   
 104    cur = buf;
 105    if ( ( 6 == length ) || /* System-ID */
 106         ( 7 == length ) || /* LAN-ID */
 107         ( 8 == length )) { /* LSP-ID */
 108      cur += g_snprintf(cur, buf_len, "%02x%02x.%02x%02x.%02x%02x", ad[0], ad[1],
 109                      ad[2], ad[3], ad[4], ad[5] );
 110      if ( ( 7 == length ) ||
 111           ( 8 == length )) {
 112          cur += g_snprintf(cur, (gulong) (buf_len-(cur-buf)), ".%02x", ad[6] );
 113      }
 114      if ( 8 == length ) {
 115          cur += g_snprintf(cur, (gulong) (buf_len-(cur-buf)), "-%02x", ad[7] );
 116      }
 117    }
 118    else {
 119      tmp = 0;
 120      while ( tmp < length / 4 ) { /* 16 / 4 == 4 > four Octets left to print */
 121        cur += g_snprintf(cur, (gulong) (buf_len-(cur-buf)), "%02x", ad[tmp++] );
 122        cur += g_snprintf(cur, (gulong) (buf_len-(cur-buf)), "%02x", ad[tmp++] );
 123        cur += g_snprintf(cur, (gulong) (buf_len-(cur-buf)), "%02x", ad[tmp++] );
 124        cur += g_snprintf(cur, (gulong) (buf_len-(cur-buf)), "%02x.", ad[tmp++] );
 125      }
 126      if ( 1 == tmp ) {   /* Special case for Designated IS */
 127        cur--;
 128        g_snprintf(cur, (gulong) (buf_len-(cur-buf)), ".%02x", ad[tmp] );
 129      }
 130      else {
 131        for ( ; tmp < length; ) {  /* print the rest without dot */
 132          cur += g_snprintf(cur, (gulong) (buf_len-(cur-buf)), "%02x", ad[tmp++] );
 133        }
 134      }
 135    }
 136  }
Show more  




Change Warning 1226.29398 : Redundant Condition

Priority:
State:
Finding:
Owner:
Note: