Skip to content

Remove extraneous format specifier associated with random ctest segfaults#21

Merged
markleader merged 1 commit intonasa:mainfrom
dfnca:cbindingTestFix
Feb 3, 2026
Merged

Remove extraneous format specifier associated with random ctest segfaults#21
markleader merged 1 commit intonasa:mainfrom
dfnca:cbindingTestFix

Conversation

@dfnca
Copy link
Copy Markdown
Contributor

@dfnca dfnca commented Feb 3, 2026

Summary

There was an unused %s format specifier in C binding test example 2 that apparently could cause memory corruption.
This addresses Issue #20

Changes

Removed extraneous format specifier (see diff)

Testing

  1. made change on a pristine branch
  2. rebuilt release version, same compiler and settings (gcc 13.3.0)
  3. Invoked ctest repeatedly, always 100% pass where previously example 2 would sometimes cause a segmentation fault

Compatibility / Numerical behavior

  • [ X] No expected changes to numerical results
  • Expected changes (explain and provide validation)

@markleader
Copy link
Copy Markdown
Contributor

Thanks @dfnca , merging

@markleader markleader merged commit 64d1fd4 into nasa:main Feb 3, 2026
8 of 12 checks passed
@dfnca dfnca deleted the cbindingTestFix branch February 3, 2026 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants