Showing 19 of 19 total issues
Either the condition cmd==200
is redundant, otherwise there is negative array index -51. Open
if (tncnts[index][m] > MAXTNCNT) {
- Read upRead up
- Exclude checks
Either the condition (cmd)==200
is redundant, otherwise there is negative array index -51. Open
if (tncnts[index][m] > MAXTNCNT) {
- Read upRead up
- Exclude checks
XXX found Open
/* XXX maybe we should have some retry handling if not the whole first
- Exclude checks
XXX found Open
debug(F101,"tn_ini switching from XXX to Telnet","",ttnproto);
- Exclude checks
BUG found Open
int tn_sb_bug = 1; /* Telnet BUG - SB w/o WILL or DO */
- Exclude checks
BUG found Open
int tn_auth_krb5_des_bug = 1; /* Telnet BUG - AUTH KRB5 DES */
- Exclude checks
Too many #ifdef configurations - cppcheck only checks 12 of 389 configurations. Use --force to check all configurations. Open
#endif /* TNCODE */
- Read upRead up
- Exclude checks
Local variable i shadows outer variable Open
int i;
- Read upRead up
- Exclude checks
The scope of the variable alias
can be reduced. Open
static int alias = -1; /* which alias are we using ? */
- Read upRead up
- Exclude checks
The scope of the variable ch
can be reduced. Open
int ch = 0, count = 0;
- Read upRead up
- Exclude checks
The scope of the variable y
can be reduced. Open
int c, x, y, n, m, flag;
- Read upRead up
- Exclude checks
Consecutive return, break, continue, goto or throw statements are unnecessary. Open
break;
- Read upRead up
- Exclude checks
Local variable sb_out shadows outer variable Open
CHAR sb_out[24]; /* multiple threads */
- Read upRead up
- Exclude checks
Variable ttn
is assigned a value that is never used. Open
ttn = (char *)sb_out; /* Point back to beginning */
- Read upRead up
- Exclude checks
Local variable local shadows outer variable Open
extern int ckxech, local;
- Read upRead up
- Exclude checks