Skip to content

Fix #320, return CFE_PSP_ERROR on failure in CFE_PSP_GetCFETextSegmentInfo#460

Merged
dzbaker merged 1 commit intonasa:mainfrom
thnkslprpt:fix-320-return-ERROR-on-failure-in-CFE_PSP_GetCFETextSegmentInfo
Dec 17, 2025
Merged

Fix #320, return CFE_PSP_ERROR on failure in CFE_PSP_GetCFETextSegmentInfo#460
dzbaker merged 1 commit intonasa:mainfrom
thnkslprpt:fix-320-return-ERROR-on-failure-in-CFE_PSP_GetCFETextSegmentInfo

Conversation

@thnkslprpt
Copy link
Contributor

Checklist

Describe the contribution

Testing performed
GitHub CI actions all passing successfully (incl. Build + Run, Unit/Functional Tests etc.).

Expected behavior changes
Returns CFE_PSP_ERROR when moduleInfoGet fails - as expected.

System(s) tested on
Debian 12 using the current main branch of cFS bundle.

Contributor Info
Avi Weiss   @thnkslprpt

Copy link
Contributor

@jphickey jphickey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is a bug! Thanks for the fix.

@jphickey jphickey added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Sep 30, 2025
@dzbaker dzbaker added CCB:Approved Indicates Approval by CCB and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Dec 17, 2025
dzbaker added a commit to nasa/cFS that referenced this pull request Dec 17, 2025
*Combines:*

cFE equuleus-rc1+dev275
psp equuleus-rc1+dev85

**Includes:**

*cFE*
- nasa/cFE#463
- nasa/cFE#2611
- nasa/cFE#2613
- nasa/cFE#2619
- nasa/cFE#2621
- nasa/cFE#2624
- nasa/cFE#2670

*osal*

*PSP*
- nasa/PSP#459
- nasa/PSP#460
- nasa/PSP#463

Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com>
Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
@dzbaker dzbaker mentioned this pull request Dec 17, 2025
2 tasks
@dzbaker dzbaker merged commit b16cda2 into nasa:main Dec 17, 2025
10 checks passed
dzbaker added a commit to nasa/cFS that referenced this pull request Dec 17, 2025
*Combines:*

cFE equuleus-rc1+dev275
psp equuleus-rc1+dev85

**Includes:**

*cFE*
- nasa/cFE#2347
- nasa/cFE#2611
- nasa/cFE#2613
- nasa/cFE#2619
- nasa/cFE#2621
- nasa/cFE#2624
- nasa/cFE#2670

*osal*

*PSP*
- nasa/PSP#459
- nasa/PSP#460
- nasa/PSP#463

Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com>
Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
@thnkslprpt thnkslprpt deleted the fix-320-return-ERROR-on-failure-in-CFE_PSP_GetCFETextSegmentInfo branch December 18, 2025 10:31
@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 Approval by CCB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CFE_PSP_GetCFETextSegmentInfo returns success even though it failed to get module info

3 participants