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

Fix uninitialized member #4037

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Fix uninitialized member #4037

merged 1 commit into from
Nov 21, 2023

Conversation

EduPonz
Copy link

@EduPonz EduPonz commented Nov 21, 2023 •
edited by MiguelCompany
Loading

Description

Fixes ununitialized error reported by valgrind. Kudos @pablogs9

@Mergifyio backport master

Contributor Checklist

  • Commit messages follow the project guidelines.
  • The code follows the style guidelines of this project.
  • N/A: Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
  • N/A: Any new/modified methods have been properly documented using Doxygen.
  • Changes are ABI compatible.
  • Changes are API compatible.
  • N/A: New feature has been added to the versions.md file (if applicable).
  • N/A: New feature has been documented/Current behavior is correctly described in the documentation.
  • Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • Check contributor checklist is correct.
  • N/A Check CI results: changes do not issue any warning.
  • N/A Check CI results: failing tests are unrelated with the changes.

@pablogs9
Fix uninitialized member
683be98 
Signed-off-by: Pablo Garrido <pablogs9@gmail.com>
@EduPonz EduPonz added the skip-ci Automatically pass CI label Nov 21, 2023
@MiguelCompany MiguelCompany added this to the v2.12.2 milestone Nov 21, 2023
@EduPonz
Copy link
Author

EduPonz commented Nov 21, 2023

@Mergifyio backport master

@mergify Mergify
Copy link
Contributor

mergify bot commented Nov 21, 2023 •
edited
Loading

backport master

✅ Backports have been created

@EduPonz EduPonz merged commit b7fc545 into 2.12.x Nov 21, 2023
8 checks passed
@EduPonz EduPonz deleted the pgarrido/fix_uninitialized_error branch November 21, 2023 13:59
@mergify mergify bot mentioned this pull request Nov 21, 2023
Merged
EduPonz added a commit that referenced this pull request Dec 7, 2023
@EduPonz
Fix uninitialized member (#4037)
bc05ce6 
Signed-off-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
(cherry picked from commit b7fc545)
EduPonz added a commit that referenced this pull request Dec 11, 2023
@mergify @EduPonz
Fix uninitialized member (#4037) (#4038)
0073aa6 
Signed-off-by: Pablo Garrido <pablogs9@gmail.com>
Co-authored-by: Pablo Garrido <pablogs9@gmail.com>
(cherry picked from commit b7fc545)

Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MiguelCompany MiguelCompany MiguelCompany approved these changes

Assignees
No one assigned
Labels
skip-ci Automatically pass CI
Projects
None yet
Milestone
v2.12.2
Development

Successfully merging this pull request may close these issues.
3 participants
@EduPonz @MiguelCompany @pablogs9