➜
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 2011-01-23 22:02 by vstinner, last changed 2022-04-11 14:57 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| ssl.patch | vstinner, 2011-01-27 23:10 | |||
| Messages (8) | |||
|---|---|---|---|
| msg126901 - (view) | Author: STINNER Victor (vstinner) *  |
Date: 2011-01-23 22:02 | |
ssl.SSLContext(True).load_verify_locations(None, True) does segfault. Py_DECREF(cafile_bytes) in Modules/_ssl.c:1686 should be replaced by Py_XDECREF(cafile_bytes). |
|||
| msg126902 - (view) | Author: Antoine Pitrou (pitrou) * |
Date: 2011-01-23 22:07 | |
Needs a patch + tests :) |
|||
| msg127237 - (view) | Author: STINNER Victor (vstinner) * |
Date: 2011-01-27 23:10 | |
Here is a patch including a test. The test pass on regrtest with -R 3:3: (no reference leak). |
|||
| msg127238 - (view) | Author: Antoine Pitrou (pitrou) * |
Date: 2011-01-27 23:27 | |
Looks good to me. |
|||
| msg127365 - (view) | Author: STINNER Victor (vstinner) * |
Date: 2011-01-29 01:36 | |
Should it be fixed in Python 3.2 or not? (load_verify_locations was introduced with SSLContext in Python 3.2) |
|||
| msg127367 - (view) | Author: Antoine Pitrou (pitrou) * |
Date: 2011-01-29 01:37 | |
Well, I think you can commit. |
|||
| msg127386 - (view) | Author: Georg Brandl (georg.brandl) * |
Date: 2011-01-29 09:21 | |
It's reviewed by Antoine, so you can commit. |
|||
| msg127405 - (view) | Author: STINNER Victor (vstinner) * |
Date: 2011-01-29 11:32 | |
Ok, fixed in r88228. Not backport needed, SSLContext was introduced in Python 3.2. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:57:11 | admin | set | github: 55198 |
| 2011-01-29 11:32:55 | vstinner | set | status: open -> closed messages: + msg127405 resolution: accepted -> fixed nosy: georg.brandl, pitrou, vstinner |
| 2011-01-29 09:21:22 | georg.brandl | set | nosy:
georg.brandl, pitrou, vstinner messages: + msg127386 |
| 2011-01-29 01:37:59 | pitrou | set | nosy:
georg.brandl, pitrou, vstinner messages: + msg127367 |
| 2011-01-29 01:36:59 | vstinner | set | priority: high -> release blocker nosy: georg.brandl, pitrou, vstinner messages: + msg127365 |
| 2011-01-27 23:27:51 | pitrou | set | nosy:
georg.brandl, pitrou, vstinner messages: + msg127238 resolution: accepted stage: needs patch -> commit review |
| 2011-01-27 23:10:56 | vstinner | set | files:
+ ssl.patch messages: + msg127237 keywords: + patch nosy: georg.brandl, pitrou, vstinner |
| 2011-01-23 22:07:18 | pitrou | set | priority: normal -> high nosy: + georg.brandl messages: + msg126902 type: crash stage: needs patch |
| 2011-01-23 22:02:01 | vstinner | create | |