Issue 33767: Improper use of SystemError in the mmap module (original) (raw)

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/77948

classification

Title: Improper use of SystemError in the mmap module
Type: behavior Stage: resolved
Components: Extension Modules Versions: Python 3.8, Python 3.7, Python 3.6, Python 2.7

process

Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: ZackerySpytz, miss-islington, serhiy.storchaka
Priority: normal Keywords: patch

Created on 2018-06-04 04:40 by ZackerySpytz, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 7381 merged ZackerySpytz,2018-06-04 04:44
PR 7426 merged miss-islington,2018-06-05 13:00
PR 7427 merged miss-islington,2018-06-05 13:02
PR 7432 merged serhiy.storchaka,2018-06-05 13:24
Messages (7)
msg318623 - (view) Author: Zackery Spytz (ZackerySpytz) * (Python triager) Date: 2018-06-04 04:40
Both mmap_concat() and mmap_repeat() raise a SystemError when invoked.
msg318630 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2018-06-04 04:59
Thank you for your PR Zackery! I concur that an alternative would be to remove mmap_concat() and mmap_repeat(). Why not use this simpler way? Are there any drawbacks?
msg318635 - (view) Author: Zackery Spytz (ZackerySpytz) * (Python triager) Date: 2018-06-04 05:47
It doesn't look like it. I've updated the PR.
msg318749 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2018-06-05 12:59
New changeset e9e397605789b2a67b67558fbbe756b7b88934f5 by Serhiy Storchaka (Zackery Spytz) in branch 'master': bpo-33767: Fix improper use of SystemError by mmap.mmap objects (GH-7381) https://github.com/python/cpython/commit/e9e397605789b2a67b67558fbbe756b7b88934f5
msg318753 - (view) Author: miss-islington (miss-islington) Date: 2018-06-05 13:21
New changeset 631fe1fa423c42197d533103dcf349ca19baed0c by Miss Islington (bot) in branch '3.7': bpo-33767: Fix improper use of SystemError by mmap.mmap objects (GH-7381) https://github.com/python/cpython/commit/631fe1fa423c42197d533103dcf349ca19baed0c
msg318754 - (view) Author: miss-islington (miss-islington) Date: 2018-06-05 13:25
New changeset ae55d29fe04ee1aeb1bcb2e9b84e5bc9d4ecb070 by Miss Islington (bot) in branch '3.6': bpo-33767: Fix improper use of SystemError by mmap.mmap objects (GH-7381) https://github.com/python/cpython/commit/ae55d29fe04ee1aeb1bcb2e9b84e5bc9d4ecb070
msg318767 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2018-06-05 16:51
New changeset 3014d6eb7f6ed8cc61b9b26fe1c4454760dc8621 by Serhiy Storchaka in branch '2.7': [2.7] bpo-33767: Fix improper use of SystemError by mmap.mmap objects (GH-7381) (GH-7432) https://github.com/python/cpython/commit/3014d6eb7f6ed8cc61b9b26fe1c4454760dc8621
History
Date User Action Args
2022-04-11 14:59:01 admin set github: 77948
2018-06-05 16:52:00 serhiy.storchaka set status: open -> closedresolution: fixedstage: patch review -> resolved
2018-06-05 16:51:46 serhiy.storchaka set messages: +
2018-06-05 13:25:24 miss-islington set messages: +
2018-06-05 13:24:54 serhiy.storchaka set pull_requests: + <pull%5Frequest7058>
2018-06-05 13:21:07 miss-islington set nosy: + miss-islingtonmessages: +
2018-06-05 13:02:51 miss-islington set pull_requests: + <pull%5Frequest7053>
2018-06-05 13:00:53 miss-islington set pull_requests: + <pull%5Frequest7052>
2018-06-05 12:59:47 serhiy.storchaka set messages: +
2018-06-04 05:47:22 ZackerySpytz set messages: +
2018-06-04 04:59:09 serhiy.storchaka set nosy: + serhiy.storchakamessages: +
2018-06-04 04:44:04 ZackerySpytz set keywords: + patchstage: patch reviewpull_requests: + <pull%5Frequest7008>
2018-06-04 04:40:18 ZackerySpytz create