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年06月18日 17:52 by serhiy.storchaka, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| bufferedrwpair_close.patch | serhiy.storchaka, 2014年06月18日 17:52 | review | ||
| bufferedrwpair_close_2.patch | serhiy.storchaka, 2014年11月25日 07:41 | review | ||
| Messages (6) | |||
|---|---|---|---|
| msg220945 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2014年06月18日 17:52 | |
Current implementation of BufferedRWPair.close() is: def close(self): self.writer.close() self.reader.close() When self.writer.close() raises an exception, self.reader left non-closed. This can cause file description leak unless GC sweep it. Proposed patch fixes this issue. With applied patch for issue21715 it would be a little simpler. |
|||
| msg224000 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2014年07月25日 20:47 | |
Antoine, what do you think about this? |
|||
| msg231595 - (view) | Author: Antoine Pitrou (pitrou) * (Python committer) | Date: 2014年11月24日 10:00 | |
Why don't you reuse the API from issue21715? |
|||
| msg231641 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2014年11月25日 07:41 | |
Because this API was not still committed. Here is a patch which uses it. |
|||
| msg234268 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2015年01月18日 20:05 | |
Ping. If this patch will be accepted I'll provide larger patch for similar issues in close methods of other classes. |
|||
| msg239182 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2015年03月24日 21:24 | |
New changeset d4694c672c36 by Serhiy Storchaka in branch '3.4': Issue #21802: The reader in BufferedRWPair now is closed even when closing https://hg.python.org/cpython/rev/d4694c672c36 New changeset f515c7ccff57 by Serhiy Storchaka in branch 'default': Issue #21802: The reader in BufferedRWPair now is closed even when closing https://hg.python.org/cpython/rev/f515c7ccff57 New changeset dcf25060cae8 by Serhiy Storchaka in branch '2.7': Issue #21802: The reader in BufferedRWPair now is closed even when closing https://hg.python.org/cpython/rev/dcf25060cae8 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:05 | admin | set | github: 66001 |
| 2015年03月24日 21:24:54 | serhiy.storchaka | set | status: open -> closed resolution: fixed stage: patch review -> resolved |
| 2015年03月24日 21:24:33 | python-dev | set | nosy:
+ python-dev messages: + msg239182 |
| 2015年03月24日 21:01:59 | serhiy.storchaka | set | assignee: serhiy.storchaka |
| 2015年03月20日 02:47:50 | martin.panter | set | nosy:
+ martin.panter |
| 2015年01月18日 20:05:49 | serhiy.storchaka | set | messages: + msg234268 |
| 2014年11月25日 07:41:18 | serhiy.storchaka | set | files:
+ bufferedrwpair_close_2.patch messages: + msg231641 |
| 2014年11月24日 10:00:20 | pitrou | set | messages: + msg231595 |
| 2014年07月25日 20:47:34 | serhiy.storchaka | set | messages: + msg224000 |
| 2014年06月18日 17:52:22 | serhiy.storchaka | create | |