[PATCH] evdns has issues in 64-bit
Nobody has claimed this yet.
- Dominant language
- C
- Stars
- 4
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
```
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 `elizab...@sporksirc.net` on 31 Aug 2010 at 2:17
Attachments:
- [evdns-64-bit-fix.patch](https://storage.googleapis.com/google-code-attachments/pyevent/issue-25/comment-0/evdns-64-bit-fix.patch)
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the Pyrex evdns file and the attached evdns-64-bit-fix.patch, then reproduce the reported failure on 64-bit Linux. The work is done when the overflowing id is handled without an exception and the evdns call succeeds.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c, python
- Domain
- networking
- Issue type
- Bug
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100