summaryrefslogtreecommitdiff
path: root/django
diff options
context:
space:
mode:
authorFlorian Apolloner <florian@apolloner.eu>2013-09-17 22:59:56 +0200
committerFlorian Apolloner <florian@apolloner.eu>2013-09-24 21:10:21 +0200
commite5dc08f2dbdd8bf67960f0448810d71ba58990b4 (patch)
tree63c9998c2a33e526fc9af4ecf53cdf14739a50dc /django
parent50a811a170dfeddd9c27d21aee36801277d21bf3 (diff)
[1.6.x] Fixed #21138 -- Increased the performance of our PBKDF2 implementation.
Thanks go to Michael Gebetsroither for pointing out this issue and help on the patch. Backport of 68540fe4df44492571bc610a0a043d3d02b3d320 from master.
Diffstat (limited to 'django')
-rw-r--r--django/utils/crypto.py10
1 files changed, 7 insertions, 3 deletions
diff --git a/django/utils/crypto.py b/django/utils/crypto.py
index d3b14cdd4d..2dc571087b 100644
--- a/django/utils/crypto.py
+++ b/django/utils/crypto.py
@@ -123,9 +123,8 @@ def _fast_hmac(key, msg, digest):
This function operates on bytes.
"""
dig1, dig2 = digest(), digest()
- if len(key) > dig1.block_size:
- key = digest(key).digest()
- key += b'\x00' * (dig1.block_size - len(key))
+ if len(key) != dig1.block_size:
+ raise ValueError('Key size needs to match the block_size of the digest.')
dig1.update(key.translate(hmac.trans_36))
dig1.update(msg)
dig2.update(key.translate(hmac.trans_5C))
@@ -161,6 +160,11 @@ def pbkdf2(password, salt, iterations, dklen=0, digest=None):
hex_format_string = "%%0%ix" % (hlen * 2)
+ inner_digest_size = digest().block_size
+ if len(password) > inner_digest_size:
+ password = digest(password).digest()
+ password += b'\x00' * (inner_digest_size - len(password))
+
def F(i):
def U():
u = salt + struct.pack(b'>I', i)