Skip to content

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 <leighgarbs@users.noreply.github.com>
Co-authored by: Anh Van <avan989@users.noreply.github.com>
@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 <leighgarbs@users.noreply.github.com>
Co-authored by: Anh Van <avan989@users.noreply.github.com>
Co-authored by: Voicu Ioan Vladut <vladvoicux64@users.noreply.github.com>
@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 <leighgarbs@users.noreply.github.com>
Co-authored by: Anh Van <avan989@users.noreply.github.com>
Co-authored by: Voicu Ioan Vladut <vladvoicux64@users.noreply.github.com>
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