This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author brendan-donegan
Recipients Brian Nenninger, brendan-donegan, rhettinger, steven.daprano
Date 2016-12-24.09:39:22
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
Ok, here it is. My first code patch in Python. 

Basically the existing code was depending on bit_length to DTRT and raise a ValueError, but negative numbers have a positive bit length. Then when it hits:

234             while r >= n:                                                        
235                 r = getrandbits(k)  

It just spins on that as r is always going to be greater than a negative number.

I tried not to be too clever so just put a guard early in the function. This has the added advantage of giving us a clearer error message.
Date User Action Args
2016-12-24 09:39:22brendan-donegansetrecipients: + brendan-donegan, rhettinger, steven.daprano, Brian Nenninger
2016-12-24 09:39:22brendan-donegansetmessageid: <>
2016-12-24 09:39:22brendan-doneganlinkissue29061 messages
2016-12-24 09:39:22brendan-donegancreate