-
Notifications
You must be signed in to change notification settings - Fork 765
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Protect from uncaught exception during SHM Segment creation. (backport …
…#3293) (#3333) * Protect from uncaught exception during SHM Segment creation (#3293) * Added regression test Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Fix Segfault when SharedSegment::compute_per_allocation_extra_size throws Signed-off-by: Eduardo Ponz <eduardoponz@eprosima.com> * Refs #16360: Added comments and remove some unneeded code Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Applied suggestions Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Uncrustify Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Fixed Windows warnings Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Added NOMINMAX flag to test Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> --------- Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> Signed-off-by: Eduardo Ponz <eduardoponz@eprosima.com> Co-authored-by: Eduardo Ponz <eduardoponz@eprosima.com> (cherry picked from commit 72d1ba1) # Conflicts: # src/cpp/rtps/DataSharing/WriterPool.hpp * Fixed conflicts Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Fixed log macro name Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Fixed log macro name Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> * Refs #16360: Please uncrustify Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Modified SharedMemSegment mock to be in line with 2.6.x version of the same class Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> --------- Signed-off-by: Javier Santiago <javiersantiago@eprosima.com> Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> Co-authored-by: jsantiago-eProsima <90755661+jsantiago-eProsima@users.noreply.github.com> Co-authored-by: Javier Santiago <javiersantiago@eprosima.com> Co-authored-by: JesusPoderoso <jesuspoderoso@eprosima.com>
- Loading branch information
1 parent
b06c377
commit 578b5ed
Showing
9 changed files
with
621 additions
and
48 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.