nasa / osal

The Core Flight System (cFS) Operating System Abstraction Layer (OSAL)
Apache License 2.0
556 stars 217 forks source link

Fix pointer error in OS_GetErrorName #1436

Closed ahatstat closed 10 months ago

ahatstat commented 11 months ago

Checklist (Please check before submitting)

Describe the contribution Fixes #1435

Testing performed Build and run all coverage tests including coverage-shared-errors-testrunner on ARM/Linux and PC/Ubunutu. All tests pass and do not seg fault.

Expected behavior changes Fewer seg faults when running unit tests

System(s) tested on ARM9/Embedded Linux PC/Ubuntu 22.04LTS

Additional context N/A

Third party code N/A

Contributor Info - All information REQUIRED for consideration of pull request Andrew Hatstat, Geost

jphickey commented 10 months ago

This looks good but I had the question of how it could have been in there for so long and never noticed. We should create a follow-on issue to investigate why this does not fail in our existing workflows