[issue41654] Segfault when raising MemoryError

STINNER Victor added the comment: New changeset 1f0cde678406749524d11e852a16bf243cef5c5f by Miss Islington (bot) in branch '3.9': bpo-41654: Fix compiler warning in MemoryError_dealloc() (GH-22387) (GH-24894) https://github.com/python/cpython/commit/1f0cde678406749524d11e852a16bf243cef5c5f

[issue41654] Segfault when raising MemoryError

Change by miss-islington : -- nosy: +miss-islington nosy_count: 5.0 -> 6.0 pull_requests: +23656 pull_request: https://github.com/python/cpython/pull/24894 ___ Python tracker _

[issue41654] Segfault when raising MemoryError

STINNER Victor added the comment: New changeset bbeb223e9a5e9f9374df384efa386b4068a65c0e by Victor Stinner in branch 'master': bpo-41654: Fix compiler warning in MemoryError_dealloc() (GH-22387) https://github.com/python/cpython/commit/bbeb223e9a5e9f9374df384efa386b4068a65c0e -- __

[issue41654] Segfault when raising MemoryError

Change by STINNER Victor : -- pull_requests: +21429 pull_request: https://github.com/python/cpython/pull/22387 ___ Python tracker ___ __

[issue41654] Segfault when raising MemoryError

Pablo Galindo Salgado added the comment: New changeset 6ae61959ec51a6b3dddc8e665ce6a7b8aeb26c04 by Pablo Galindo in branch '3.8': bpo-41654: Explicitly cast PyExc_MemoryError to PyTypeObject to avoid warning (GH-22102) https://github.com/python/cpython/commit/6ae61959ec51a6b3dddc8e665ce6a7b8

[issue41654] Segfault when raising MemoryError

Change by Pablo Galindo Salgado : -- pull_requests: +21188 pull_request: https://github.com/python/cpython/pull/22102 ___ Python tracker ___ ___

[issue41654] Segfault when raising MemoryError

STINNER Victor added the comment: "AMD64 Arch Linux Asan 3.8" buildbot logs a compiler warning: https://buildbot.python.org/all/#builders/580/builds/4 Objects/exceptions.c: In function ‘MemoryError_dealloc’: Objects/exceptions.c:2298:23: warning: comparison of distinct pointer types lacks a ca

[issue41654] Segfault when raising MemoryError

STINNER Victor added the comment: Thanks for the fix Pablo! Thanks for the funny bug report Oleg Hoefling! I didn't expect that anyone would ever subclass MemoryError. -- ___ Python tracker

[issue41654] Segfault when raising MemoryError

Change by Pablo Galindo Salgado : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue41654] Segfault when raising MemoryError

Pablo Galindo Salgado added the comment: New changeset 77f4000ae0d43a2685face80e7f14d4aba053973 by Pablo Galindo in branch '3.8': [3.8] [3.9] bpo-41654: Fix deallocator of MemoryError to account for subclasses (GH-22020) (GH-22046) https://github.com/python/cpython/commit/77f4000ae0d43a2685f

[issue41654] Segfault when raising MemoryError

Pablo Galindo Salgado added the comment: New changeset d14775ddbb067bcfa6eca516d3cbe968a8c1334e by Pablo Galindo in branch '3.9': [3.9] bpo-41654: Fix deallocator of MemoryError to account for subclasses (GH-22020) (GH-22045) https://github.com/python/cpython/commit/d14775ddbb067bcfa6eca516d

[issue41654] Segfault when raising MemoryError

Change by Pablo Galindo Salgado : -- pull_requests: +21142 pull_request: https://github.com/python/cpython/pull/22045 ___ Python tracker ___ ___

[issue41654] Segfault when raising MemoryError

Change by Pablo Galindo Salgado : -- pull_requests: +21143 pull_request: https://github.com/python/cpython/pull/22046 ___ Python tracker ___ ___

[issue41654] Segfault when raising MemoryError

Pablo Galindo Salgado added the comment: New changeset 9b648a95ccb4c3b14f1e87158f5c9f5dbb2f62c0 by Pablo Galindo in branch 'master': bpo-41654: Fix deallocator of MemoryError to account for subclasses (GH-22020) https://github.com/python/cpython/commit/9b648a95ccb4c3b14f1e87158f5c9f5dbb2f62c0

[issue41654] Segfault when raising MemoryError

Pablo Galindo Salgado added the comment: The problem is that the deallocator of MemoryError class is not taking into account subclasses for the freelist management, but the allocator (tp_new) is. -- ___ Python tracker

[issue41654] Segfault when raising MemoryError

Change by Pablo Galindo Salgado : -- keywords: +patch nosy: +pablogsal nosy_count: 4.0 -> 5.0 pull_requests: +21120 stage: -> patch review pull_request: https://github.com/python/cpython/pull/22020 ___ Python tracker

[issue41654] Segfault when raising MemoryError

hai shi added the comment: Hm, Looks like we need check the double free risk of `Py_DECREF()`. -- ___ Python tracker ___ ___ Python

[issue41654] Segfault when raising MemoryError

Change by Dong-hee Na : -- nosy: +corona10 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue41654] Segfault when raising MemoryError

Change by hai shi : -- nosy: +shihai1991 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.

[issue41654] Segfault when raising MemoryError

STINNER Victor added the comment: Aha, interesting bug report. I attached a simplified reproducer. Output with a Python built in debug mode: --- Objects/frameobject.c:590: _Py_NegativeRefcount: Assertion failed: object has negative ref count Fatal Python error: _PyObject_AssertFailed: _PyObj

[issue41654] Segfault when raising MemoryError

Oleg Hoefling added the comment: If this is of any help, I've set up an example repository containing the snippet: https://github.com/hoefling/bpo-issue-41654 Here are the results of running the snippet in Travis with Python 3.{5-10} and Pypy 3.6: https://travis-ci.com/github/hoefling/bpo-is

[issue41654] Segfault when raising MemoryError

New submission from Oleg Hoefling : First of all, I guess this is a somewhat obscure error that is unlikely to occur in a usual context, nevertheless IMO worth reporting. We observed this when unit-testing custom exception reporting mechanism, raising different exceptions in different context