Skip to content

Fixed the SHM segment smashing while reading with security <master> #1646

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

Merged
merged 7 commits into from
Jan 11, 2021

Conversation

rsanchez15
Copy link
Contributor

This is a port of PR #1644 from 2.0.x to master

@rsanchez15 rsanchez15 force-pushed the bugfix/security-shm/master branch from 14ce462 to 43002ba Compare December 18, 2020 12:21
@richiware
Copy link
Member

Build status:

  • Linux Build Status
  • Linux aarch64Build Status
  • Mac Build Status
  • Windows Build Status

@richiware
Copy link
Member

Build status:

  • Linux Build Status
  • Linux aarch64Build Status
  • Mac Build Status
  • Windows Build Status

@MiguelCompany MiguelCompany merged commit 8ddd10a into master Jan 11, 2021
@MiguelCompany MiguelCompany deleted the bugfix/security-shm/master branch January 11, 2021 10:56
MiguelBarro pushed a commit that referenced this pull request Jan 15, 2021
* Refs #9475: Fixed encrypted RTPS message and submessage decoding

Signed-off-by: RaulSanchez <[email protected]>

* Refs #9475: Added secure communication test for encrypted messages

Signed-off-by: RaulSanchez <[email protected]>

* Refs #9475: Uncrustify

Signed-off-by: RaulSanchez <[email protected]>

* Refs #9475: Added buffer to process decoded submessages

Signed-off-by: RaulSanchez <[email protected]>

* Refs #9475: Added more comments

Signed-off-by: RaulSanchez <[email protected]>

* Refs #9475: Applied suggested changes

Signed-off-by: RaulSanchez <[email protected]>

* Refs #9475: Applied suggested changes on port to 2.1.x

Signed-off-by: RaulSanchez <[email protected]>
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.

3 participants