PMD Results

The following document contains the results of PMD 4.2.5.

Files

net/sf/sshapi/impl/ganymed/GanymedHostKeyManager.java

ViolationLine
Found 'DD'-anomaly for variable 'type' (lines '98'-'101').98 - 101
Found 'DD'-anomaly for variable 'type' (lines '98'-'104').98 - 104
Found 'DD'-anomaly for variable 'keyBytes' (lines '99'-'102').99 - 102
Found 'DD'-anomaly for variable 'keyBytes' (lines '99'-'105').99 - 105
It is a good practice to call super() in a constructor154 - 158

net/sf/sshapi/impl/ganymed/GanymedSCPClient.java

ViolationLine
It is a good practice to call super() in a constructor23 - 25

net/sf/sshapi/impl/ganymed/GanymedSftpClient.java

ViolationLine
It is a good practice to call super() in a constructor59 - 61
Found 'DU'-anomaly for variable 'buf' (lines '72'-'89').72 - 89
Found 'DD'-anomaly for variable 'currentFile' (lines '92'-'94').92 - 94
Found 'DU'-anomaly for variable 'currentFile' (lines '94'-'138').94 - 138
Found 'DU'-anomaly for variable 'buf' (lines '111'-'138').111 - 138
Found 'DU'-anomaly for variable 'fileOffset' (lines '112'-'138').112 - 138
Use explicit scoping instead of the default package private level175 - 177
Use explicit scoping instead of the default package private level179 - 181
Use explicit scoping instead of the default package private level254 - 264
A method should have only one exit point, and that should be the last statement in the method256
A method should have only one exit point, and that should be the last statement in the method258
A method should have only one exit point, and that should be the last statement in the method260
Found 'DU'-anomaly for variable 't' (lines '316'-'333').316 - 333

net/sf/sshapi/impl/ganymed/GanymedSshClient.java

ViolationLine
Use explicit scoping instead of the default package private level82
A method should have only one exit point, and that should be the last statement in the method131
Found 'DU'-anomaly for variable 'authenticatorMap' (lines '141'-'214').141 - 214
Found 'DU'-anomaly for variable 'authenticatorMap' (lines '141'-'214').141 - 214
A method should have only one exit point, and that should be the last statement in the method158
Found 'DD'-anomaly for variable 'pw' (lines '170'-'178').170 - 178
Found 'DD'-anomaly for variable 'pw' (lines '170'-'178').170 - 178
A method should have only one exit point, and that should be the last statement in the method186
Found 'DU'-anomaly for variable 'kbi' (lines '196'-'214').196 - 214
Found 'DD'-anomaly for variable 'kbi' (lines '196'-'196').196
Found 'DU'-anomaly for variable 'kbi' (lines '196'-'214').196 - 214
Found 'DD'-anomaly for variable 'kbi' (lines '196'-'196').196
A method should have only one exit point, and that should be the last statement in the method204
Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 !249
Assigning an Object to null is a code smell. Consider refactoring.334
Found 'DU'-anomaly for variable 'hexFingerprint' (lines '422'-'451').422 - 451
Found 'DU'-anomaly for variable 'hexFingerprint' (lines '422'-'451').422 - 451
Found 'DU'-anomaly for variable 'hexFingerprint' (lines '422'-'451').422 - 451
Found 'DU'-anomaly for variable 'hexFingerprint' (lines '422'-'451').422 - 451
Found 'DU'-anomaly for variable 'hexFingerprint' (lines '422'-'451').422 - 451
A method should have only one exit point, and that should be the last statement in the method443
A method should have only one exit point, and that should be the last statement in the method445
Assigning an Object to null is a code smell. Consider refactoring.540

net/sf/sshapi/impl/ganymed/GanymedStreamChannel.java

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