Skip to content

Commit

Permalink
SST ASAN buffer fix (#3972)
Browse files Browse the repository at this point in the history
* Release data buffer as well

* Suppress fewer tests
  • Loading branch information
eisenhauer committed Dec 28, 2023
1 parent 18205ce commit 13723e8
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 29 deletions.
39 changes: 10 additions & 29 deletions CTestConfig.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -14,35 +14,16 @@ set(MEMORYCHECK_SUPPRESSIONS_FILE ${CMAKE_SOURCE_DIR}/scripts/dashboard/nightly/

# Ignore tests that are currently failing, remove tests here as they are fixed
list(APPEND CTEST_CUSTOM_MEMCHECK_IGNORE
Staging.TimeoutReader.1x1.CommMin.BP5.SST
Staging.1x1Struct.BP5
Staging.WriteMemorySelectionRead.1x1.CommMin.BP.SST
Staging.1x1.Local2.CommMin.BP.SST
Staging.OnDemandSingle.1x1.CommMin.BP5.SST
Staging.AllToAllDistribution.1x1x3.CommMin.BP5.SST
Staging.RoundRobinDistribution.1x1x3.CommMin.BP5.SST
Staging.1x1.LocalMultiblock.CommMin.BP5.SST
Staging.1x1.SstRUDP.CommMin.BP5.SST
Staging.WriteMemorySelectionRead.1x1.CommMin.BP5.SST
Staging.1x1LockGeometry.CommMin.BP5.SST
Staging.1x1.ForcePreload.CommMin.BP5.SST
Staging.1x1.NoPreload.CommMin.BP5.SST
Staging.DiscardWriter.1x1.CommMin.BP5.SST
Staging.LatestReaderHold.1x1.CommMin.BP5.SST
Staging.LatestReader.1x1.CommMin.BP5.SST
Engine.BP.BPWriteReadAsStreamTestADIOS2.ReaderWriterDefineVariable.BP5.Serial
Staging.ZFPCompression.3x5.CommMin.BP5.SST
Staging.ZFPCompression.1x1.CommMin.BP5.SST
Staging.1x1Joined.CommMin.BP5.SST
Staging.1x1Struct.CommMin.BP5.SST
Staging.1x1.Local2.CommMin.BP5.SST
Staging.1x1VarDestruction.CommMin.BP5.SST
Staging.1x1.ModAttrs.CommMin.BP5.SST
Staging.1x1.Attrs.CommMin.BP5.SST
Staging.1x1.CommMin.BP5.SST
Engine.SST.SstWriteFails.InvalidBeginStep.Serial
remoteServerCleanup
Remote.BPWriteMemorySelectionRead.FileRemote
Remote.BPWriteMemorySelectionRead.GetRemote
Remote.BPWriteReadADIOS2stdio.GetRemote
Remote.BPWriteMemorySelectionRead.GetRemote
Remote.BPWriteMemorySelectionRead.FileRemote
remoteServerCleanup
Engine.SST.SstWriteFails.InvalidBeginStep.Serial
Staging.1x1.Local2.CommMin.BP5.SST
Staging.1x1Struct.CommMin.BP5.SST
Staging.WriteMemorySelectionRead.1x1.CommMin.BP5.SST
Staging.1x1.Local2.CommMin.BP.SST
Staging.WriteMemorySelectionRead.1x1.CommMin.BP.SST
Staging.1x1Struct.BP5
)
1 change: 1 addition & 0 deletions source/adios2/engine/sst/SstWriter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -293,6 +293,7 @@ void SstWriter::EndStep()
// Free data and metadata blocks here. BlockToFree is the newblock
// value in the enclosing function.
free(BlockToFree->MetaMetaBlocks);
delete BlockToFree->TSInfo->DataBuffer;
delete BlockToFree->TSInfo;
delete BlockToFree;
};
Expand Down

0 comments on commit 13723e8

Please sign in to comment.