You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Gary D. Gregory (JIRA)" <ji...@apache.org> on 2012/09/01 18:44:07 UTC
[jira] [Resolved] (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 ]
Gary D. Gregory resolved CODEC-151.
-----------------------------------
Resolution: Fixed
Fix Version/s: 1.7
> 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
> Fix For: 1.7
>
> 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