Skip to content

Commit 011c3f5

Browse files
committed
Checks return code before nullifying db pointer.
1 parent 426ed4c commit 011c3f5

3 files changed

Lines changed: 9 additions & 3 deletions

File tree

boost_src/sqlite3pp.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,9 @@ namespace sqlite3pp
9797
int rc = SQLITE_OK;
9898
if (db_) {
9999
rc = sqlite3_close(db_);
100-
db_ = 0;
100+
if (rc == SQLITE_OK) {
101+
db_ = 0;
102+
}
101103
}
102104

103105
return rc;

headeronly_src/sqlite3pp.ipp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,9 @@ namespace sqlite3pp
114114
auto rc = SQLITE_OK;
115115
if (db_) {
116116
rc = sqlite3_close(db_);
117-
db_ = nullptr;
117+
if (rc == SQLITE_OK) {
118+
db_ = nullptr;
119+
}
118120
}
119121

120122
return rc;

src/sqlite3pp.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,9 @@ namespace sqlite3pp
116116
auto rc = SQLITE_OK;
117117
if (db_) {
118118
rc = sqlite3_close(db_);
119-
db_ = nullptr;
119+
if (rc == SQLITE_OK) {
120+
db_ = nullptr;
121+
}
120122
}
121123

122124
return rc;

0 commit comments

Comments
 (0)