Skip to content

Fix transaction wrapper mutex release on exception - #22

Open
Solution-lab wants to merge 1 commit into
seclerp:masterfrom
Solution-lab:fix/transaction-wrapper-release
Open

Fix transaction wrapper mutex release on exception#22
Solution-lab wants to merge 1 commit into
seclerp:masterfrom
Solution-lab:fix/transaction-wrapper-release

Conversation

@Solution-lab

Copy link
Copy Markdown

Summary

Fixes a mutex leak when a FileFsClient operation throws before EndTransaction is called.

Changes

  • Replaced manual BeginTransaction / EndTransaction calls with callback-based Execute.
  • Ensured the mutex is always released after an operation.
  • Updated client tests to use NullTransactionWrapper.
  • Added a regression test for mutex release after an exception.

Known limitation

FileFsClient.Copy was already implemented without a transaction and remains unchanged in this PR.

Verification

  • 497 tests passed
  • Full solution build completed with 0 errors

Sign up for freeto 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.

1 participant

@Solution-lab