null check dynamic casts

stellar · Nov 21, 2024 · bdd99db · bdd99db
1 parent 1f5ab3e
commit bdd99db
Show file tree

Hide file tree

Showing 10 changed files with 10 additions and 0 deletions.
diff --git a/src/ledger/LedgerTxnAccountSQL.cpp b/src/ledger/LedgerTxnAccountSQL.cpp
@@ -628,6 +628,7 @@ class BulkLoadAccountsOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnClaimableBalanceSQL.cpp b/src/ledger/LedgerTxnClaimableBalanceSQL.cpp
@@ -109,6 +109,7 @@ class BulkLoadClaimableBalanceOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnConfigSettingSQL.cpp b/src/ledger/LedgerTxnConfigSettingSQL.cpp
@@ -112,6 +112,7 @@ class bulkLoadConfigSettingsOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnContractCodeSQL.cpp b/src/ledger/LedgerTxnContractCodeSQL.cpp
@@ -117,6 +117,7 @@ class BulkLoadContractCodeOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnContractDataSQL.cpp b/src/ledger/LedgerTxnContractDataSQL.cpp
@@ -150,6 +150,7 @@ class BulkLoadContractDataOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnDataSQL.cpp b/src/ledger/LedgerTxnDataSQL.cpp
@@ -452,6 +452,7 @@ class BulkLoadDataOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnLiquidityPoolSQL.cpp b/src/ledger/LedgerTxnLiquidityPoolSQL.cpp
@@ -129,6 +129,7 @@ class BulkLoadLiquidityPoolOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnOfferSQL.cpp b/src/ledger/LedgerTxnOfferSQL.cpp
@@ -798,6 +798,7 @@ class BulkLoadOffersOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnTTLSQL.cpp b/src/ledger/LedgerTxnTTLSQL.cpp
@@ -116,6 +116,7 @@ class BulkLoadTTLOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);

diff --git a/src/ledger/LedgerTxnTrustLineSQL.cpp b/src/ledger/LedgerTxnTrustLineSQL.cpp
@@ -465,6 +465,7 @@ class BulkLoadTrustLinesOperation
         }
         auto sqliteStatement =
             dynamic_cast<soci::sqlite3_statement_backend*>(be);
+        releaseAssertOrThrow(sqliteStatement);
         auto st = sqliteStatement->stmt_;
 
         sqlite3_reset(st);