Skip to content

[Blueberry Release v2.0.2] originId not set in in ChunkSender if chunk is recycled #1850

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

Closed
mossmaurice opened this issue Jan 16, 2023 · 2 comments
Assignees
Labels
bug Something isn't working duplicate This issue or pull request already exists
Milestone

Comments

@mossmaurice
Copy link
Contributor

Required information

Branch:
release_2.0

Operating system:
Ubuntu 20.04.3 LTS

Compiler version:
GCC 8.4.0

Observed result or behaviour:

On ros2/rmw_iceoryx#84 (6546111) when starting the client for the 2nd time the originId is never valid after reading it via chunk_header->originId() acquired via popo::UntypedClient.

Expected result or behaviour:
The originId should always be set and valid.

Conditions where it occurred / Performed steps:

./install/bin/iox-roudi -l verbose -m off
RMW_IMPLEMENTATION=rmw_iceoryx_cpp ros2 run demo_nodes_cpp add_two_ints_server
RMW_IMPLEMENTATION=rmw_iceoryx_cpp ros2 run demo_nodes_cpp add_two_ints_client # 'originId' is valid and can be read, client gracefully stops
RMW_IMPLEMENTATION=rmw_iceoryx_cpp ros2 run demo_nodes_cpp add_two_ints_client # Now, 'originId' is invalid and seems uninitialized 
@mossmaurice mossmaurice added the bug Something isn't working label Jan 16, 2023
@mossmaurice mossmaurice added this to the High prio milestone Jan 16, 2023
@mossmaurice mossmaurice self-assigned this Jan 16, 2023
mossmaurice pushed a commit to ApexAI/iceoryx that referenced this issue Jan 16, 2023
@elBoberido
Copy link
Member

@mossmaurice it seems it was fixed by #1668 on master

@mossmaurice mossmaurice added the duplicate This issue or pull request already exists label Jan 16, 2023
@mossmaurice
Copy link
Contributor Author

Duplicate of #1668

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants