Bugfix: memory leaks are prevented

rscada / python-mbus

Python wrapper for libmbus

BSD 3-Clause "New" or "Revised" License

25 stars 28 forks source link

Bugfix: memory leaks are prevented #18

Closed cgrigis closed 7 years ago

cgrigis commented 7 years ago

As part of the development of a long-running service based on python-mbus, we realized that the library was leaking. This was traced to three issues:

the MBus context was not freed after deletion
the frame data was not freed and frame_data_free was not exposed
the frame_data_xml did not free the returned buffer

This pull request addresses all three issues and corrects the example accordingly. If you have any comment or remark, we'll be happy to address those as part of this PR. Thank you.

cgrigis commented 7 years ago

As a separate note, we're wondering if you'd be interested in sharing this package through PyPI. We'd be happy to assist in that direction.

rscada commented 7 years ago

Thanks for the fixes!

Also, having this package on PyPI is fine with me.