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

Mem-leak in iceoryx2_bb_posix::Directory::contents() on failure #287

Closed
elfenpiff opened this issue Jul 12, 2024 · 0 comments · Fixed by #288
Closed

Mem-leak in iceoryx2_bb_posix::Directory::contents() on failure #287

elfenpiff opened this issue Jul 12, 2024 · 0 comments · Fixed by #288
Assignees
@elfenpiff
Labels
bug Something isn't working

Comments

@elfenpiff
Copy link
Contributor

Required information

When the contents call fails, it will immediately return with fail! and skip the release of the acquired memory posix::free for the file names.
@elfenpiff elfenpiff added the bug Something isn't working label Jul 12, 2024
@elfenpiff elfenpiff self-assigned this Jul 12, 2024
elfenpiff added a commit to elfenpiff/iceoryx2 that referenced this issue Jul 12, 2024
@elfenpiff
[eclipse-iceoryx#287] Fix directory mem leak; skip empty file names i…
dda538b 
…n contents list
elfenpiff added a commit to elfenpiff/iceoryx2 that referenced this issue Jul 12, 2024
@elfenpiff
[eclipse-iceoryx#287] Add release note
9cde861
@elfenpiff elfenpiff mentioned this issue Jul 12, 2024
Merged
17 tasks
elfenpiff added a commit to elfenpiff/iceoryx2 that referenced this issue Jul 12, 2024
@elfenpiff
[eclipse-iceoryx#287] Add release note
24bf49f
elfenpiff added a commit that referenced this issue Jul 12, 2024
@elfenpiff
Merge pull request #288 from elfenpiff/iox2-287-fix-directory-mem-leak
dd20e65 
[#287] fix directory mem leak
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@elfenpiff elfenpiff

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#287] fix directory mem leak elfenpiff/iceoryx2
1 participant
@elfenpiff