[
https://issues.apache.org/jira/browse/CLOUDSTACK-8551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14583214#comment-14583214
]
ASF subversion and git services commented on CLOUDSTACK-8551:
-------------------------------------------------------------
Commit d46b658ec0a997d8e55cfc818dca2ad869447d46 in cloudstack's branch
refs/heads/master from [~rajanik]
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=d46b658 ]
Fixed CLOUDSTACK-8551 Findbugs warning in LdapCreateAccountCmd.java
byte[].toString() would give reference to the array (ex: [B@6c521576 )
but not the original string. used new String() to get the text.
> 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)
