PMD Results

The following document contains the results of PMD 4.2.5.

Files

net/sf/sshapi/impl/jsch/JschHostKeyManager.java

ViolationLine
It is a good practice to call super() in a constructor157 - 159

net/sf/sshapi/impl/jsch/JschIdentityManager.java

ViolationLine
Found 'DU'-anomaly for variable 'tempFile' (lines '103'-'131').103 - 131
Found 'DD'-anomaly for variable 'type' (lines '135'-'137').135 - 137
Found 'DD'-anomaly for variable 'type' (lines '135'-'139').135 - 139
Found 'DU'-anomaly for variable 'type' (lines '135'-'148').135 - 148
Use explicit scoping instead of the default package private level150 - 162
Avoid assignments in operands155
Found 'DU'-anomaly for variable 'b' (lines '155'-'162').155 - 162
A method should have only one exit point, and that should be the last statement in the method206
A method should have only one exit point, and that should be the last statement in the method208
Avoid empty catch blocks210 - 211
Found 'DU'-anomaly for variable 'jschKey' (lines '269'-'306').269 - 306
Found 'DU'-anomaly for variable 'jschKey' (lines '269'-'306').269 - 306
Found 'DU'-anomaly for variable 'jschKey' (lines '269'-'306').269 - 306
Found 'DU'-anomaly for variable 'jschKey' (lines '269'-'306').269 - 306
Found 'DU'-anomaly for variable 'jschKey' (lines '269'-'306').269 - 306
Found 'DU'-anomaly for variable 'jschKey' (lines '269'-'306').269 - 306

net/sf/sshapi/impl/jsch/JschSCPClient.java

ViolationLine
It is a good practice to call super() in a constructor24 - 26
Found 'DU'-anomaly for variable 'out' (lines '47'-'82').47 - 82
Found 'DD'-anomaly for variable 'buf' (lines '108'-'123').108 - 123
Found 'DU'-anomaly for variable 'in' (lines '142'-'232').142 - 232
Found 'DD'-anomaly for variable 'buf' (lines '144'-'147').144 - 147
Found 'DU'-anomaly for variable 'fos' (lines '159'-'232').159 - 232
Found 'DD'-anomaly for variable 'filesize' (lines '164'-'172').164 - 172
Found 'DD'-anomaly for variable 'foo' (lines '188'-'191').188 - 191
Found 'DD'-anomaly for variable 'foo' (lines '190'-'191').190 - 191
Found 'DU'-anomaly for variable 'run' (lines '220'-'232').220 - 232
Found 'DU'-anomaly for variable 'i' (lines '236'-'244').236 - 244
Found 'DD'-anomaly for variable 'i' (lines '236'-'236').236

net/sf/sshapi/impl/jsch/JschSftpClient.java

ViolationLine
It is a good practice to call super() in a constructor50 - 52
Found 'DU'-anomaly for variable 'paths' (lines '61'-'74').61 - 74
Use explicit scoping instead of the default package private level76 - 78
Use explicit scoping instead of the default package private level131 - 139
A method should have only one exit point, and that should be the last statement in the method133
A method should have only one exit point, and that should be the last statement in the method135

net/sf/sshapi/impl/jsch/JschSshClient.java

ViolationLine
Found 'DU'-anomaly for variable 'paw' (lines '152'-'184').152 - 184
Found 'DU'-anomaly for variable 'ki' (lines '154'-'184').154 - 184
Assigning an Object to null is a code smell. Consider refactoring.292
A method should have only one exit point, and that should be the last statement in the method307
A method should have only one exit point, and that should be the last statement in the method394
Use explicit scoping instead of the default package private level401 - 419
A method should have only one exit point, and that should be the last statement in the method415
Use explicit scoping instead of the default package private level422
Found 'DU'-anomaly for variable 'hash' (lines '437'-'474').437 - 474
Found 'DU'-anomaly for variable 'hash' (lines '437'-'474').437 - 474
Found 'DU'-anomaly for variable 'hash' (lines '437'-'474').437 - 474
Found 'DU'-anomaly for variable 'hash' (lines '437'-'474').437 - 474
Found 'DU'-anomaly for variable 'hash' (lines '437'-'474').437 - 474
A method should have only one exit point, and that should be the last statement in the method463
A method should have only one exit point, and that should be the last statement in the method465
A method should have only one exit point, and that should be the last statement in the method467
Use explicit scoping instead of the default package private level501 - 527
A method should have only one exit point, and that should be the last statement in the method508
A method should have only one exit point, and that should be the last statement in the method510
A method should have only one exit point, and that should be the last statement in the method533
A method should have only one exit point, and that should be the last statement in the method535
A method should have only one exit point, and that should be the last statement in the method537
A method should have only one exit point, and that should be the last statement in the method539
Found 'DD'-anomaly for variable 'sshapiLevel' (lines '547'-'550').547 - 550
Found 'DD'-anomaly for variable 'sshapiLevel' (lines '547'-'553').547 - 553
Found 'DD'-anomaly for variable 'sshapiLevel' (lines '547'-'556').547 - 556
Found 'DD'-anomaly for variable 'sshapiLevel' (lines '547'-'559').547 - 559
It is a good practice to call super() in a constructor573 - 575
Assigning an Object to null is a code smell. Consider refactoring.647

net/sf/sshapi/impl/jsch/JschStreamChannel.java

ViolationLine
It is a good practice to call super() in a constructor45 - 48