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 queryables memory leak #880

Merged
merged 7 commits into from
Feb 3, 2025
Merged

Conversation

jean-roland
Copy link
Contributor

Fixes #879

Fix a raised assertion introduced with #877 on z_bytes_to_string api call.

Copy link

github-actions bot commented Feb 3, 2025

PR missing one of the required labels: {'documentation', 'breaking-change', 'new feature', 'dependencies', 'bug', 'internal', 'enhancement'}

@jean-roland jean-roland added the bug Something isn't working label Feb 3, 2025
@jean-roland jean-roland requested a review from sashacmc February 3, 2025 14:19
Copy link
Member

@sashacmc sashacmc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed some unexpected changes, other LGTM

CMakeLists.txt Outdated Show resolved Hide resolved
include/zenoh-pico/config.h Outdated Show resolved Hide resolved
@gabrik gabrik merged commit 07700e4 into eclipse-zenoh:main Feb 3, 2025
67 checks passed
@jean-roland jean-roland deleted the fix_memory_leak branch February 4, 2025 10:44
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

Successfully merging this pull request may close these issues.

[Bug] _z_trigger_queryables_inner() does not release infos for unhandled queries
3 participants