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

Fix memory leak errors raised by Address Sanitizer from Python tests #1189

Open
anuchak opened this issue Jan 17, 2023 · 1 comment
Open
Assignees
Labels
bug Something isn't working

Comments

@anuchak
Copy link
Collaborator

anuchak commented Jan 17, 2023

6_cat asan log.txt

Currently Address Sanitizer is raising memory leak issues due to which the Python tests are failing:

  • In kuzu_binding.cpp the bindEnumTypes function has a memory leak.
  • In py_connection.cpp the transformPythonParameters and transformPythonParameter function are taking parameters by value, need to be changed to take the args by const reference.

These are the two I noticed but there can be more.

@acquamarin
Copy link
Collaborator

As desribed here: pybind/pybind11#3865
There is a memory leakage bug when binding enum class.

@andyfengHKU andyfengHKU added the bug Something isn't working label Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants