Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drhdf5 fix #1816

Closed
wants to merge 55 commits into from
Closed

Drhdf5 fix #1816

wants to merge 55 commits into from

Conversation

KatuunuXVI
Copy link

The Macros in the DRHDF5 file modify the size of the DRHDF5 in only this translation unit upon compilation. This can result in buffer overflow (encountered during tests with checkpoint loading).

Jose A. Perez and others added 30 commits October 16, 2023 12:14
Added IP&A additions

See merge request simulation/trick!1
Added get_class_map function to scheduler

See merge request simulation/trick!2
…aster'

Resolve AXST-250128 "Feature/ create simulation profiler"

Closes AXST-250128

See merge request simulation/axtools/trick!3
…into 'master'

External link to Standard Messenger

See merge request simulation/axtools/trick!4
…lently' into 'master'"

This reverts commit 0ed0033, reversing
changes made to 49f6067.
…aster'

Resolve AXST-321270 "Bugfix/ axsim memory manager issues"

Closes AXST-321270

See merge request simulation/axtools/trick!5
Record Toggle

See merge request simulation/axtools/trick!8
Hard Fail on Bad Memory

See merge request simulation/axtools/trick!9
KatuunuXVI and others added 25 commits October 4, 2024 15:05
Record Enable moved to Member of Trick Data Record Object

See merge request simulation/axtools/trick!11
DR Check

See merge request simulation/axtools/trick!10
Data Record Fix

See merge request simulation/axtools/trick!12
Double Free Removal

See merge request simulation/axtools/trick!13
…ulation/axtools/trick into hotfix/fix_MTV_GUI
@coveralls
Copy link

Coverage Status

coverage: 55.881% (-0.05%) from 55.926%
when pulling 53346ca on Axiom-Space:drhdf5-fix
into 1330c42 on nasa:master.

@sharmeye
Copy link
Contributor

I think this will require more investigation to ensure that any proposed solution doesn't introduce unwanted behavior elsewhere in the system, and as such our team will investigate and resolve this internally. Thank you for the information and valuable input you have provided.

@sharmeye sharmeye closed this Dec 12, 2024
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.

5 participants