Skip to content

Comments

Fix #232, Address sanitizer issue#235

Merged
dzbaker merged 1 commit intonasa:mainfrom
avan989:Fix-232-address-sanitizer-issue
Feb 25, 2025
Merged

Fix #232, Address sanitizer issue#235
dzbaker merged 1 commit intonasa:mainfrom
avan989:Fix-232-address-sanitizer-issue

Conversation

@avan989
Copy link
Contributor

@avan989 avan989 commented Feb 19, 2025

Fix #232, set data buffer for CFE_TBL_GetAddress

Describe the contribution
A clear and concise description of what the contribution is.
Fix #232, Update coverage test for address sanitizer issue.

Testing performed
Steps taken to test the contribution:
Add in the following options, then run unit test.
a. add_compile_options(-fsanitize=address -g)
b. add_link_options(-fsanitize=address)

Expected behavior changes
None

System(s) tested on

  • Hardware: [e.g. PC, SP0, MCP750]
  • OS: [e.g. Ubuntu 18.04, RTEMS 4.11, VxWorks 6.9]
  • Versions: [e.g. cFE 6.6, OSAL 4.2, PSP 1.3 for mcp750, any related apps or tools]

Additional context
Add any other context about the contribution here.

Third party code
If included, identify any third party code and provide text file of license

Contributor Info - All information REQUIRED for consideration of pull request
Full name and company/organization/center of all contributors ("Personal" if individual work)

  • If NASA Civil Servant Employee or GSFC Contractor on SES II
    • Address/email/phone and contract/task information (if applicable) must be on file
  • Else if Company
    • HAND SIGNED Company CLA must be on file (once per release): Company CLA
  • Else if Individual
    • HAND SIGNED Individual CLA must be on file (once per release): Individual CLA
      Anh Van, GSFC

Fix nasa#232, set data buffer for CFE_TBL_GetAddress
@dzbaker dzbaker added the CCB:Approved Indicates code approval by CCB label Feb 20, 2025
dzbaker added a commit to nasa/cFS that referenced this pull request Feb 25, 2025
*Combines:*

osal equuleus-rc1+dev97
sample_app equuleus-rc1+dev50

**Includes:**

*osal*
-nasa/osal#1493

*sample_app*
-nasa/sample_app#235

Co-authored by: Leigh Garbs <[email protected]>
Co-authored by: Anh Van <[email protected]>
@dzbaker dzbaker mentioned this pull request Feb 25, 2025
2 tasks
dzbaker added a commit to nasa/cFS that referenced this pull request Feb 25, 2025
*Combines:*

osal equuleus-rc1+dev97
sample_app equuleus-rc1+dev50
cFS-GroundSystem equuleus-rc1+dev18

**Includes:**

*osal*
-nasa/osal#1493

*sample_app*
-nasa/sample_app#235

*cFS-GroundSystem*
-nasa/cFS-GroundSystem#249

Co-authored by: Leigh Garbs <[email protected]>
Co-authored by: Anh Van <[email protected]>
Co-authored by: Voicu Ioan Vladut <[email protected]>
@dzbaker dzbaker merged commit 7c2e0d9 into nasa:main Feb 25, 2025
10 checks passed
dzbaker added a commit to nasa/cFS that referenced this pull request Feb 25, 2025
*Combines:*

osal equuleus-rc1+dev97
sample_app equuleus-rc1+dev50
cFS-GroundSystem equuleus-rc1+dev18

**Includes:**

*osal*
-nasa/osal#1493

*sample_app*
-nasa/sample_app#235

*cFS-GroundSystem*
-nasa/cFS-GroundSystem#249

Co-authored by: Leigh Garbs <[email protected]>
Co-authored by: Anh Van <[email protected]>
Co-authored by: Voicu Ioan Vladut <[email protected]>
@dzbaker dzbaker added this to the v7.0.0 milestone Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates code approval by CCB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

addressSanitizer issue with coverage-sample_app-sample_app_cmds-testrunner

3 participants