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

Fixes memory leak in move-value filter when Settings is selected as the Move Value Type. #217

Merged

Conversation

FiniteSingularity
Copy link
Contributor

The obs_data_t *setting in the Move Value filter's load_properties function was not being released in all cases. Added in an increment to the reference counter if setting is assigned to item2, and then releases setting at the end of the function call.

In addition, item was not being released in the while loop in move_value_load_properties.

Finally, this PR adds version.h and resource.rc to the .gitignore file for devs who are using the git command line.

Fixes #216

@exeldro exeldro merged commit 6b9d5cb into exeldro:master Jul 2, 2024
7 of 8 checks passed
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

Successfully merging this pull request may close these issues.

[Bug] Minor memory leak in Move Value filter when "Settings" selected as Move Value Type.
2 participants