PMD Results

The following document contains the results of PMD 4.2.5.

Files

net/sf/sshapi/impl/j2ssh/J2SshClient.java

ViolationLine
Found 'DU'-anomaly for variable 'authenticatorMap' (lines '132'-'185').132 - 185
A method should have only one exit point, and that should be the last statement in the method182
Found 'DU'-anomaly for variable 'passwordAuthenticator' (lines '198'-'284').198 - 284
Found 'DU'-anomaly for variable 'passwordAuthenticator' (lines '198'-'284').198 - 284
Found 'DU'-anomaly for variable 'passwordAuthenticator' (lines '198'-'284').198 - 284
Found 'DU'-anomaly for variable 'passwordAuthenticator' (lines '198'-'284').198 - 284
A method should have only one exit point, and that should be the last statement in the method221
Found 'DU'-anomaly for variable 'publicKeyAuthenticator' (lines '224'-'284').224 - 284
Found 'DU'-anomaly for variable 'publicKeyAuthenticator' (lines '224'-'284').224 - 284
Found 'DU'-anomaly for variable 'publicKeyAuthenticator' (lines '224'-'284').224 - 284
Found 'DU'-anomaly for variable 'publicKeyAuthenticator' (lines '224'-'284').224 - 284
Found 'DD'-anomaly for variable 'pw' (lines '233'-'235').233 - 235
Found 'DD'-anomaly for variable 'pw' (lines '233'-'235').233 - 235
Found 'DD'-anomaly for variable 'pw' (lines '233'-'235').233 - 235
Found 'DD'-anomaly for variable 'pw' (lines '233'-'235').233 - 235
A method should have only one exit point, and that should be the last statement in the method256
Found 'DU'-anomaly for variable 'keyboardInteractiveAuthenticator' (lines '259'-'284').259 - 284
Found 'DU'-anomaly for variable 'keyboardInteractiveAuthenticator' (lines '259'-'284').259 - 284
Found 'DU'-anomaly for variable 'keyboardInteractiveAuthenticator' (lines '259'-'284').259 - 284
Found 'DU'-anomaly for variable 'keyboardInteractiveAuthenticator' (lines '259'-'284').259 - 284
Found 'DD'-anomaly for variable 'prompt' (lines '266'-'269').266 - 269
Found 'DD'-anomaly for variable 'prompt' (lines '266'-'269').266 - 269
Found 'DD'-anomaly for variable 'prompt' (lines '266'-'269').266 - 269
Found 'DD'-anomaly for variable 'prompt' (lines '266'-'269').266 - 269
Found 'DD'-anomaly for variable 'echo' (lines '267'-'270').267 - 270
Found 'DD'-anomaly for variable 'echo' (lines '267'-'270').267 - 270
Found 'DD'-anomaly for variable 'echo' (lines '267'-'270').267 - 270
Found 'DD'-anomaly for variable 'echo' (lines '267'-'270').267 - 270
Found 'DD'-anomaly for variable 'prompt' (lines '269'-'269').269
Found 'DD'-anomaly for variable 'prompt' (lines '269'-'269').269
Found 'DD'-anomaly for variable 'prompt' (lines '269'-'269').269
Found 'DD'-anomaly for variable 'prompt' (lines '269'-'269').269
Found 'DD'-anomaly for variable 'echo' (lines '270'-'270').270
Found 'DD'-anomaly for variable 'echo' (lines '270'-'270').270
Found 'DD'-anomaly for variable 'echo' (lines '270'-'270').270
Found 'DD'-anomaly for variable 'echo' (lines '270'-'270').270
Found 'DU'-anomaly for variable 'fwd' (lines '359'-'381').359 - 381
Found 'DU'-anomaly for variable 'fwd' (lines '359'-'381').359 - 381
Found 'DU'-anomaly for variable 'fwd' (lines '359'-'381').359 - 381
Found 'DU'-anomaly for variable 'fwd' (lines '386'-'412').386 - 412
Found 'DU'-anomaly for variable 'fwd' (lines '386'-'412').386 - 412
Found 'DU'-anomaly for variable 'fwd' (lines '386'-'412').386 - 412
Found 'DU'-anomaly for variable 'status' (lines '459'-'486').459 - 486
Found 'DU'-anomaly for variable 'status' (lines '459'-'486').459 - 486
Found 'DU'-anomaly for variable 'status' (lines '459'-'486').459 - 486
Found 'DU'-anomaly for variable 'status' (lines '459'-'486').459 - 486
Found 'DU'-anomaly for variable 'status' (lines '459'-'486').459 - 486
A method should have only one exit point, and that should be the last statement in the method476
A method should have only one exit point, and that should be the last statement in the method483
Avoid assignments in operands577
Assigning an Object to null is a code smell. Consider refactoring.591
It is a good practice to call super() in a constructor619 - 621
Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 !633
A method should have only one exit point, and that should be the last statement in the method635

net/sf/sshapi/impl/j2ssh/J2SshSftpClient.java

ViolationLine
It is a good practice to call super() in a constructor47 - 49
Use explicit scoping instead of the default package private level75 - 77
Use explicit scoping instead of the default package private level146 - 148
Use explicit scoping instead of the default package private level150 - 152
Use explicit scoping instead of the default package private level154 - 170
A method should have only one exit point, and that should be the last statement in the method156
A method should have only one exit point, and that should be the last statement in the method158
A method should have only one exit point, and that should be the last statement in the method160
A method should have only one exit point, and that should be the last statement in the method162
A method should have only one exit point, and that should be the last statement in the method164
A method should have only one exit point, and that should be the last statement in the method166

net/sf/sshapi/impl/j2ssh/J2SshStreamChannel.java

ViolationLine
It is a good practice to call super() in a constructor42 - 45