SRombauts / SQLiteCpp

100%

master: 100%

LAST BUILD ON BRANCH fix-savepoint-destructor-exception-safety

branch: fix-savepoint-destructor-exception-safety

CHANGE BRANCH

x

Reset

• fix-savepoint-destructor-exception-safety
• 181-feature-statement-iterator
• 2.0.0
• 2.1.0
• 2.2.0
• 2.3.0
• 2.4.0
• 2.5.0
• 3.0.0
• 3.1.0
• 3.1.1
• add-bind-no-copy-pull-request-86
• add-cmake-project-version-to-fix-warning
• add-implicit-cast-char-short
• asan-tests
• code-review
• coverity_scan
• create-function-unit-test
• database-backup
• feature-ci-coverage
• feature-coverity-github-actions
• feature-improve-test-coverage
• fix-column-stream-utf8
• fix-coveralls
• fix-executemany-clearbindings
• fix-getheaderinfo-shift-ub
• fix-isunencrypted-raw-read
• fix-lp64-ambiguous-bind
• fix-meson-disable-std-filesystem-flag
• fix-null-pointer-ub-guards
• fix-nullptr
• fix-vs2013-column-test
• fix-warning-extern-variable-has-an-initializer
• gcc-7
• gcc-9-fix-bind
• gcov-inline-functions
• libsqlite3
• master
• move-constructor
• sqlite-3.34
• sqlitecpp-2.x
• sqlitecpp-3.x
• statement-unit-tests
• subdir_example
• test-building-with-mingw-on-appveyor
• test-travis-ci-configuration
• travis-valgrind
• try_execute
• update-sqlite3.19.3
• vs2019

Build # 28482655233

Build Type

Pull #559

github

Committed by SRombauts

Commit Message

Fix Transaction destructor exception-safety

The destructor caught only SQLite::Exception, so a std::bad_alloc thrown
while building the "ROLLBACK TRANSACTION" string (or any other non-SQLite
exception) escaped the implicitly noexcept destructor and called
std::terminate. Broaden the handler to catch(...), matching the Savepoint
fix (afa51d3).

Fixes TXN-08 from the code review.

Pull Request Pull Request #559: Fix Savepoint destructor exception-safety and rollback bookkeeping

Coverage Stats

5 of 5 new or added lines in 2 files covered. (100.0%)

641 of 643 relevant lines covered (99.69%)

34.41 hits per line