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

Initialize mvd_r and mvd_c before first use in Thompson microphysics scheme #1152

Merged

Conversation

mgduda
Copy link
Contributor

@mgduda mgduda commented Mar 21, 2024

This PR initializes two local variables, mvd_r and mvd_c, in the mp_thompson routine to avoid the potential use of uninitialized memory.

Note: Identical changes to those included in this PR have separately been introduced into newer versions of the Thompson microphysics scheme, and so the changes here can be considered a back-port of these fixes.

…scheme

This commit initializes two local variables, mvd_r and mvd_c, in the mp_thompson
routine to avoid the potential use of uninitialized memory.

Note: Identical changes to those included in this commit have been introduced
into newer versions of the Thompson microphysics scheme, and so the changes here
can be considered a back-port of fixes.
@mgduda
Copy link
Contributor Author

mgduda commented Mar 21, 2024

The changes in this PR fix Issue #1150.

@ldfowler58
Copy link
Contributor

I completely agree with the description of the added initializations and approve this PR.

@mgduda mgduda merged commit 098e9c2 into MPAS-Dev:hotfix-v8.0.2 Mar 23, 2024
@mgduda mgduda deleted the atmosphere/thompson_uninit_mvd_fix branch March 23, 2024 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SIGFPE: Floating-point exception in Thompson microphysics v8.0.1 when DEBUG=true
2 participants