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

Memory leak with statements (dpiConn_prepareStmt()). #27

Open
jaman opened this issue May 18, 2024 · 0 comments
Open

Memory leak with statements (dpiConn_prepareStmt()). #27

jaman opened this issue May 18, 2024 · 0 comments

Comments

@jaman
Copy link

jaman commented May 18, 2024

I've been working to try to find a memory leak in oranif, and believe I've found it.
I hit the leak as I hit 50+ databases every five seconds or so. This makes the leak jump out, as it grows 160 bytes per query.

The leak is in that the dpiConn_prepareStmt() is called, along with dpiStmt_close, but never the dpiStmt_release().
Without the release, the 160 bytes of the prepared statement stays around, unused, and uncleared for each query.
In my copy of oranif, I've added a dpiStmt_release(stmtRes->stmt); after the dpiStmt_close call (line 419 of dpiStmt_nif.c).

This looks to have possibly rectified memory issues in my environment, though I'll be testing for the next couple of days to see if I run into the memory leak over time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant