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.

Sign up for GitHub

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

Jump to bottom

[defect]: Potential memory leak in src/modules/rlm_sql/drivers/rlm_sql_sqlite/rlm_sql_sqlite.c #5392

Closed
icy17 opened this issue Aug 1, 2024 · 0 comments
Closed

[defect]: Potential memory leak in src/modules/rlm_sql/drivers/rlm_sql_sqlite/rlm_sql_sqlite.c #5392

icy17 opened this issue Aug 1, 2024 · 0 comments
Labels
defect category: a defect or misbehaviour

Comments

@icy17
Copy link

icy17 commented Aug 1, 2024

What type of defect/bug is this?

incorrect 3rd party API usage

How can the issue be reproduced?

Potential memory leak in rlm_sql_sqlite.c line 745. Calling sqlite3_open_v2() without calling sqlite3_close() to free the memory will cause a memory leak (in line 774).

image

Log output from the FreeRADIUS daemon

None

Relevant log output from client utilities

No response

Backtrace from LLDB or GDB

No response
@icy17 icy17 added the defect category: a defect or misbehaviour label Aug 1, 2024
alandekok added a commit that referenced this issue Aug 1, 2024
@alandekok
add more close. Fixes #5392
e5a1305
@arr2036 arr2036 closed this as completed in d736cfa Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
defect category: a defect or misbehaviour
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@icy17