Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test__xxsubinterpreters leaked references and memory blocks #80628

Closed
pablogsal opened this issue Mar 27, 2019 · 3 comments
Closed

test__xxsubinterpreters leaked references and memory blocks #80628

pablogsal opened this issue Mar 27, 2019 · 3 comments
Labels
3.8 only security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs) performance Performance or resource usage tests Tests in the Lib/test dir

Comments

@pablogsal
Copy link
Member

BPO 36447
Nosy @vstinner, @ericsnowcurrently, @pablogsal
PRs
  • bpo-36447: Fix refleak in sysmodule.c when calling SET_SYS_FROM_STRING_BORROW #12586
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = <Date 2019-03-27.12:44:16.411>
    created_at = <Date 2019-03-27.10:00:53.114>
    labels = ['interpreter-core', '3.8', 'tests', 'performance']
    title = 'test__xxsubinterpreters leaked references and memory blocks'
    updated_at = <Date 2019-03-27.13:00:37.325>
    user = 'https://github.com/pablogsal'

    bugs.python.org fields:

    activity = <Date 2019-03-27.13:00:37.325>
    actor = 'ZackerySpytz'
    assignee = 'none'
    closed = True
    closed_date = <Date 2019-03-27.12:44:16.411>
    closer = 'vstinner'
    components = ['Interpreter Core', 'Tests']
    creation = <Date 2019-03-27.10:00:53.114>
    creator = 'pablogsal'
    dependencies = []
    files = []
    hgrepos = []
    issue_num = 36447
    keywords = ['patch']
    message_count = 3.0
    messages = ['338949', '338951', '338957']
    nosy_count = 3.0
    nosy_names = ['vstinner', 'eric.snow', 'pablogsal']
    pr_nums = ['12586']
    priority = 'normal'
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = 'resource usage'
    url = 'https://bugs.python.org/issue36447'
    versions = ['Python 3.8']

    @pablogsal
    Copy link
    Member Author

    The buildbot x86 Gentoo Refleaks 3.x/546 had detected that test__xxxsubinterpreters is leaking references and blocks:

    test__xxsubinterpreters leaked [138, 138, 138] references, sum=414
    test__xxsubinterpreters leaked [69, 69, 69] memory blocks, sum=207
    4 tests failed again:
    test__xxsubinterpreters test_atexit test_capi test_threading

    https://buildbot.python.org/all/#/builders/1/builds/546

    @pablogsal pablogsal added 3.8 only security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs) tests Tests in the Lib/test dir performance Performance or resource usage labels Mar 27, 2019
    @pablogsal
    Copy link
    Member Author

    Bisecting shows that:

    8b9dbc0 is the first bad commit
    commit 8b9dbc0
    Author: Victor Stinner <vstinner@redhat.com>
    Date: Wed Mar 27 01:36:16 2019 +0100

    bpo-36444: Remove _PyMainInterpreterConfig (GH-12571)
    

    @vstinner
    Copy link
    Member

    New changeset 34ef64f by Victor Stinner (Pablo Galindo) in branch 'master':
    bpo-36447, bpo-36447: Fix refleak in _PySys_InitMain() (GH-12586)
    34ef64f

    @ZackerySpytz ZackerySpytz mannequin changed the title test__xxsubinterpreters leaked regards and memory blocks test__xxsubinterpreters leaked references and memory blocks Mar 27, 2019
    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    3.8 only security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs) performance Performance or resource usage tests Tests in the Lib/test dir
    Projects
    None yet
    Development

    No branches or pull requests

    2 participants