ddidderr/zstd-rs
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Ensure BMK_timedFnState is always freed in benchMem

Browse Source 
When an error occurs in BMK_isSuccessful_runOutcome, the code
previously skipped the call to BMK_freeTimedFnState(tfs),
leaking the allocated tfs object.
Fiexed by calling BMK_freeTimedFnState(tfs) before goto _cleanOut.
This commit is contained in:
jinyaoguo
2025-06-12 19:52:58 -04:00
parent 5e6bdf5e3d
commit cad0b72ad8
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/fullbench.c
+1
View File
@@ -890,6 +890,7 @@ static int benchMem(unsigned scenarioID,
if (!BMK_isSuccessful_runOutcome(bOutcome)) { if (!BMK_isSuccessful_runOutcome(bOutcome)) {
DISPLAY("ERROR: Scenario %u: %s \n", scenarioID, ZSTD_getErrorName(BMK_extract_errorResult(bOutcome))); DISPLAY("ERROR: Scenario %u: %s \n", scenarioID, ZSTD_getErrorName(BMK_extract_errorResult(bOutcome)));
errorcode = 1; errorcode = 1;
BMK_freeTimedFnState(tfs);
goto _cleanOut; goto _cleanOut;
} }