You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Christian Hammers (JIRA)" <ji...@apache.org> on 2012/09/01 18:02:08 UTC
[jira] [Updated] (CODEC-151) Remove unnecessary attempt to fill up
the salt variable in UnixCrypt
[ https://issues.apache.org/jira/browse/CODEC-151?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christian Hammers updated CODEC-151:
------------------------------------
Attachment: commons-codec-chammers-20120901-salt-noop.diff
> Remove unnecessary attempt to fill up the salt variable in UnixCrypt
> --------------------------------------------------------------------
>
> Key: CODEC-151
> URL: https://issues.apache.org/jira/browse/CODEC-151
> Project: Commons Codec
> Issue Type: Improvement
> Affects Versions: 1.6
> Reporter: Christian Hammers
> Priority: Minor
> Labels: patch
> Attachments: commons-codec-chammers-20120901-salt-noop.diff
>
>
> https://github.com/lathspell/commons-codec/commit/878618cecb7b2f03550f0bb0e6642ca96187cbae
> Note that the "// NOOP" comment was misleading, the salt variable
> actually was modified inside the for loop.
> This functionality will never be used though as the salt is checked
> some lines above to an exact length of 2. Filling it with "A" would
> be a very bad idea anyway as e.g. glibc fills a one-char salt with
> exactly the same char so our implementation would have been incompatible!
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira