Skip to content

Memory leak in string construction #8

Open

Description

@pjb7687

Hello,

I am inspired by your code and currently developing a cffi-based pysam alternative which works with Pypy. While testing your code I found that

ffi.new('kstring_t *', {'m': 0, 'l': 0, 's': ffi.NULL})

causes memory leak, so I replaced it with Pythonic way. You can find the change here:

HiDiHlabs@cd35d6f

Maybe there would be better way to do this? Thank you so much for your contribution!

Best regards,
Jeongbin Park

Activity

  1. brentp commented on Oct 19, 2016

    @brentp
    Owner

    Thanks for reporting.
    I didn't know about the memory leak. I think the better solution is to just free kstr.s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions