Memory leak in string construction

[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

[brentp]

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