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

C binding storage sizes do not match for multiple OS's and architectures #1218

Closed
elBoberido opened this issue Mar 3, 2022 · 0 comments · Fixed by #1220 or #1229
Closed

C binding storage sizes do not match for multiple OS's and architectures #1218

elBoberido opened this issue Mar 3, 2022 · 0 comments · Fixed by #1220 or #1229
Assignees
@elBoberido
Labels
bug Something isn't working

Comments

@elBoberido
Copy link
Member

Required information

The C binding uses arrays with magic numbers to provide a storage for C++ constructs. This is more brittle than expected and breaks on multiple platforms and architectures.

In order to fix this, the heap should be used until there is a proper solution for the magic number problems. This might be solved by #308
@elBoberido elBoberido added the bug Something isn't working label Mar 3, 2022
@elBoberido elBoberido self-assigned this Mar 3, 2022
elBoberido added a commit to ApexAI/iceoryx that referenced this issue Mar 3, 2022
@elBoberido
iox-eclipse-iceoryx#1218 Use the heap for C bindings contructs
83db754
elBoberido added a commit to ApexAI/iceoryx that referenced this issue Mar 3, 2022
@elBoberido
iox-eclipse-iceoryx#1218 Fix waitset in C example
b69e113
elBoberido added a commit to ApexAI/iceoryx that referenced this issue Mar 3, 2022
@elBoberido
iox-eclipse-iceoryx#1218 Set storage size to 8 byte
e837bcd
This was referenced Mar 3, 2022
Merged
Open
elBoberido added a commit to ApexAI/iceoryx that referenced this issue Mar 3, 2022
@elBoberido
iox-eclipse-iceoryx#1218 Add workaround to ease CyclondDDS transition…
64cd6bf 
… to the C binding using the heap
elBoberido added a commit to ApexAI/iceoryx that referenced this issue Mar 3, 2022
@elBoberido
iox-eclipse-iceoryx#1218 Reenable FreeBSD CI job
1298dbd
elBoberido added a commit to ApexAI/iceoryx that referenced this issue Mar 3, 2022
@elBoberido
iox-eclipse-iceoryx#1218 Remove iceoryx_binding_c_deployment
a55056a
elBoberido added a commit to ApexAI/iceoryx that referenced this issue Mar 3, 2022
@elBoberido
iox-eclipse-iceoryx#1218 Update release notes
302e648
elBoberido added a commit that referenced this issue Mar 3, 2022
@elBoberido
Merge pull request #1220 from ApexAI/iox-#1218-use-heap-for-c-binding…
1e62939 
…-constructs

Iox #1218 use heap for c binding constructs
@elBoberido elBoberido reopened this Mar 4, 2022
elBoberido added a commit to ApexAI/iceoryx that referenced this issue Mar 4, 2022
@elBoberido
iox-eclipse-iceoryx#1218 Remove potential dangerous usage of storage …
8b5350c 
…in binding C tests
@elBoberido elBoberido mentioned this issue Mar 4, 2022
Merged
21 tasks
elBoberido added a commit that referenced this issue Mar 4, 2022
@elBoberido
Merge pull request #1229 from ApexAI/iox-#1218-potentially-dangerous-…
4054a3e 
…usage-of-storage-in-binding-c-tests

iox-#1218 Remove potential dangerous usage of storage in binding C tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@elBoberido elBoberido

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@elBoberido