wohaph / keyczar

Automatically exported from code.google.com/p/keyczar
0 stars 0 forks source link

RSA_PRIV keylength is too small (512) in the python test suite #61

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. run the python test suite of keyczar
2. watch it fail in the testpubkey part
3.

What is the expected output? What do you see instead?
The test should pass, but it raises an exception instead.

What version of the product are you using? On what operating system?
pycrpyto-2.3
keyczar-0.6b

Please provide any additional information below.
This is the full message that I get. I suggest to change the size of the key to 
1024 as per patch attached.

======================================================================
ERROR: testPubKey (keyczart_test.KeyczartTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/guillaume/Downloads/python-keyczar-0.6b/tests/keyczar/keyczart_test.py", line 60, in testPubKey
    pubmock.AddKey(33, keyinfo.PRIMARY, 512)  # small key size for fast tests
  File "/usr/lib/python2.6/site-packages/keyczar/readers.py", line 114, in AddKey
    key = keys.GenKey(self.kmd.type, size)
  File "/usr/lib/python2.6/site-packages/keyczar/keys.py", line 72, in GenKey
    keyinfo.RSA_PRIV: RsaPrivateKey.Generate}[type](size)
  File "/usr/lib/python2.6/site-packages/keyczar/keys.py", line 538, in Generate
    key = RSA.generate(size, util.RandBytes)
  File "/usr/lib/python2.6/site-packages/Crypto/PublicKey/RSA.py", line 207, in generate
    raise ValueError("RSA modulus length must be a multiple of 256 and > 1024")
ValueError: RSA modulus length must be a multiple of 256 and > 1024

----------------------------------------------------------------------
Ran 30 tests in 0.434s

FAILED (errors=1)

Original issue reported on code.google.com by guillaum...@gmail.com on 13 Nov 2010 at 6:09

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks for your report Guillaume, here is the related commit 
http://code.google.com/p/keyczar/source/detail?r=502

Original comment by sebastien.martini on 28 Nov 2010 at 9:58