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.
Created on 2014年11月08日 14:54 by serhiy.storchaka, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| fcntl_arg_type.patch | serhiy.storchaka, 2014年11月08日 15:15 | review | ||
| Messages (5) | |||
|---|---|---|---|
| msg230861 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2014年11月08日 14:54 | |
Arguments of wrong type is passed to C function fcntl() in the fcntl module. Third argument of fcntl() should be either pointer to binary structure or C int. But C long is passed instead. All works on platforms where sizeof(long) == sizeof(int) or on little-endian platforms, but on big-endian platform with sizeof(long) != sizeof(int) this will pass wrong value. |
|||
| msg230863 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2014年11月08日 15:15 | |
Here is a patch. It is much easier than I expected. |
|||
| msg230929 - (view) | Author: Brett Cannon (brett.cannon) * (Python committer) | Date: 2014年11月10日 01:25 | |
Fixed in 3.5 as part of 6e6532d313a1 as it was easier to integrate it as part of the Clinic patch. |
|||
| msg230945 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2014年11月10日 09:29 | |
New changeset 61e99438c237 by Serhiy Storchaka in branch '2.7': Issue #22821: Fixed fcntl() with integer argument on 64-bit big-endian https://hg.python.org/cpython/rev/61e99438c237 New changeset 45e8aed69767 by Serhiy Storchaka in branch '3.4': Issue #22821: Fixed fcntl() with integer argument on 64-bit big-endian https://hg.python.org/cpython/rev/45e8aed69767 New changeset 2d203a0b7908 by Serhiy Storchaka in branch 'default': Issue #22821: Fixed fcntl() with integer argument on 64-bit big-endian https://hg.python.org/cpython/rev/2d203a0b7908 |
|||
| msg230946 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2014年11月10日 09:31 | |
> Fixed in 3.5 as part of 6e6532d313a1 as it was easier to integrate it as > part of the Clinic patch. 6e6532d313a1 has introduced other bug ("l" was parsed to int). Changed to "I" for reasons described in the comment in fcntl_ioctl_impl(). |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:10 | admin | set | github: 67010 |
| 2014年11月10日 09:33:08 | serhiy.storchaka | set | status: open -> closed assignee: serhiy.storchaka stage: commit review -> resolved resolution: fixed versions: + Python 3.5 |
| 2014年11月10日 09:31:58 | serhiy.storchaka | set | messages: + msg230946 |
| 2014年11月10日 09:29:54 | python-dev | set | nosy:
+ python-dev messages: + msg230945 |
| 2014年11月10日 01:25:30 | brett.cannon | set | stage: patch review -> commit review messages: + msg230929 versions: - Python 3.5 |
| 2014年11月08日 15:15:20 | serhiy.storchaka | set | files:
+ fcntl_arg_type.patch keywords: + patch messages: + msg230863 stage: needs patch -> patch review |
| 2014年11月08日 14:54:38 | serhiy.storchaka | create | |