[pycrypto] documentation for PKCS1_PSS.py is incorrect, plus minor issue w/ _slowmath.py
Dwayne C. Litzenberger
dlitz at dlitz.net
Thu May 24 18:57:45 EDT 2012
On Wed, May 23, 2012 at 10:48:03AM -0700, Gurney, John-Mark wrote:
>In the file on lines 41 & 42 it has:
>>>> signer = PKCS1_PSS.new(key)
>>>> signature = PKCS1_PSS.sign(key)
>The second line is incorrect. It should be:
>>>> signature = signer.sign(h)
I've committed a fix. Thanks!
>Oh, there is also a bogus assertion in _slowmath.py:
>On line 85, it asserts that e is long. If you have an e of 3, it is reasonable for e to be a normal int, and not a long. Maybe the asserts should be changed to:
>n = long(n)
>e = long(e)
>So that python will convert the types properly.
No, those assertions in _slowmath are there so that we catch usage that
would break when using _fastmath.
_fastmath.rsa_construct (which is written in C) requires long parameters in
order to keep the C code simple. If we do the implicit conversions in
_slowmath, we should also have do the conversions in _fastmath. Since
_fastmath is an internal module, I see no reason for the added complexity
(and risk of bugs).
Dwayne C. Litzenberger <dlitz at dlitz.net>
OpenPGP: 19E1 1FE8 B3CF F273 ED17 4A24 928C EC13 39C2 5CF7
More information about the pycrypto