Skip to content

v12: Fixed a memory leak in mkIAU#1199

Merged
sdrabenh merged 2 commits intofeature/sdrabenh/gcm_v12from
bugfix/atrayano/memleak_fix_mkiau-v12
Mar 12, 2026
Merged

v12: Fixed a memory leak in mkIAU#1199
sdrabenh merged 2 commits intofeature/sdrabenh/gcm_v12from
bugfix/atrayano/memleak_fix_mkiau-v12

Conversation

@mathomp4
Copy link
Copy Markdown
Member

This is the v12 equivalent of #1198 by @atrayano. Per #1198

This PR fixes a memory leak caused by wrongly passing a filename to MAPL_read_bundle instead of a file template. Then MAPL cannot properly close the netCDF after done reading it.

This PR needs MAPL v2.65.0. Otherwise, there will be run-time failures due to improper formatting issues of pFlogger

NOTE: GEOSgcm feature/sdrabenh/gcm_v12 does now have MAPL v2.65.0 as I just merged in GEOS-ESM/GEOSgcm#1025 after @sdrabenh tagged GCMv12-rc23

@mathomp4 mathomp4 self-assigned this Jan 30, 2026
@mathomp4 mathomp4 requested a review from a team as a code owner January 30, 2026 20:28
@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Jan 30, 2026
@sdrabenh sdrabenh merged commit a3c195a into feature/sdrabenh/gcm_v12 Mar 12, 2026
7 of 9 checks passed
@sdrabenh sdrabenh deleted the bugfix/atrayano/memleak_fix_mkiau-v12 branch March 12, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants