Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PATCH] evdns has issues in 64-bit #25

Open
GoogleCodeExporter opened this issue Mar 14, 2015 · 0 comments
Open

[PATCH] evdns has issues in 64-bit #25

GoogleCodeExporter opened this issue Mar 14, 2015 · 0 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Use evdns in a 64-bit environment
2. In certain cases, an exception may be thrown wherein id is overflowed
3. The call will fail with above exception

What is the expected output? What do you see instead?
The call is expected to succeed, but instead fails.

What version of the product are you using? On what operating system?
pyevent SVN - using 64-bit Linux.

Please provide any additional information below.
The bug is due to an assumption in the pyrex evdns file where id is assumed to 
never overflow an int. On 64-bit platforms, this assumption fails, as ints are 
still only 32-bits wide (and id returns the address of the object). The 
enclosed patch should resolve the issue by changing the type to a long int, 
which will be the necessary 64 bits on a 64-bit platform.

Original issue reported on code.google.com by [email protected] on 31 Aug 2010 at 2:17

Attachments:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant