[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8551?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rajani Karuturi updated CLOUDSTACK-8551:
----------------------------------------
    Description: 
LdapCreateAccountCmd.java:146, DMI_INVOKING_TOSTRING_ON_ARRAY, Priority: High

Invocation of toString on org.bouncycastle.util.encoders.Base64.encode(byte[]) 
in org.apache.cloudstack.api.command.LdapCreateAccountCmd.generatePassword()

The code invokes toString on an array, which will generate a fairly useless 
result such as [C@16f0472. Consider using Arrays.toString to convert the array 
into a readable String that gives the contents of the array. See Programming 
Puzzlers, chapter 3, puzzle 12. 


LdapImportUsersCmd.java:231, DM_DEFAULT_ENCODING, Priority: High

Found reliance on default encoding in 
org.apache.cloudstack.api.command.LdapImportUsersCmd.generatePassword(): new 
String(byte[])

Found a call to a method which will perform a byte to String (or String to 
byte) conversion, and will assume that the default platform encoding is 
suitable. This will cause the application behaviour to vary between platforms. 
Use an alternative API and specify a charset name or Charset object explicitly. 

  was:
LdapCreateAccountCmd.java:146, DMI_INVOKING_TOSTRING_ON_ARRAY, Priority: High

Invocation of toString on org.bouncycastle.util.encoders.Base64.encode(byte[]) 
in org.apache.cloudstack.api.command.LdapCreateAccountCmd.generatePassword()

The code invokes toString on an array, which will generate a fairly useless 
result such as [C@16f0472. Consider using Arrays.toString to convert the array 
into a readable String that gives the contents of the array. See Programming 
Puzzlers, chapter 3, puzzle 12. 


> Findbugs warning in LdapCreateAccountCmd.java and LdapImportUsersCmd.java
> -------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-8551
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8551
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>    Affects Versions: 4.3.0, 4.4.0, 4.5.1
>            Reporter: Rajani Karuturi
>            Assignee: Rajani Karuturi
>              Labels: ldap
>             Fix For: 4.6.0
>
>
> LdapCreateAccountCmd.java:146, DMI_INVOKING_TOSTRING_ON_ARRAY, Priority: High
> Invocation of toString on 
> org.bouncycastle.util.encoders.Base64.encode(byte[]) in 
> org.apache.cloudstack.api.command.LdapCreateAccountCmd.generatePassword()
> The code invokes toString on an array, which will generate a fairly useless 
> result such as [C@16f0472. Consider using Arrays.toString to convert the 
> array into a readable String that gives the contents of the array. See 
> Programming Puzzlers, chapter 3, puzzle 12. 
> LdapImportUsersCmd.java:231, DM_DEFAULT_ENCODING, Priority: High
> Found reliance on default encoding in 
> org.apache.cloudstack.api.command.LdapImportUsersCmd.generatePassword(): new 
> String(byte[])
> Found a call to a method which will perform a byte to String (or String to 
> byte) conversion, and will assume that the default platform encoding is 
> suitable. This will cause the application behaviour to vary between 
> platforms. Use an alternative API and specify a charset name or Charset 
> object explicitly. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to