Only prepare for describe if statement type is 'SELECT' for Oracle ba…

…ckend
SOCI · Jan 9, 2024 · f426f87 · f426f87
1 parent 65a7e4d
commit f426f87
Showing 1 changed file with 21 additions and 13 deletions.
diff --git a/src/backends/oracle/statement.cpp b/src/backends/oracle/statement.cpp
@@ -164,21 +164,29 @@ std::string oracle_statement_backend::rewrite_for_procedure_call(
 
 int oracle_statement_backend::prepare_for_describe()
 {
-    sword res = OCIStmtExecute(session_.svchp_, stmtp_, session_.errhp_,
-        1, 0, 0, 0, OCI_DESCRIBE_ONLY);
-    if (res != OCI_SUCCESS)
-    {
-        throw_oracle_soci_error(res, session_.errhp_);
-    }
+    ub2 statementType = 0;
+    sword res = OCIAttrGet(static_cast<dvoid*>(stmtp_),
+        static_cast<ub4>(OCI_HTYPE_STMT), static_cast<dvoid*>(&statementType),
+        0, static_cast<ub4>(OCI_ATTR_STMT_TYPE), session_.errhp_);
+
+    int cols = 0;
+    if (statementType == OCI_STMT_SELECT) {
+        res = OCIStmtExecute(session_.svchp_, stmtp_, session_.errhp_,
+            1, 0, 0, 0, OCI_DESCRIBE_ONLY);
+        if (res != OCI_SUCCESS)
+        {
+            throw_oracle_soci_error(res, session_.errhp_);
+        }
 
-    int cols;
-    res = OCIAttrGet(static_cast<dvoid*>(stmtp_),
-        static_cast<ub4>(OCI_HTYPE_STMT), static_cast<dvoid*>(&cols),
-        0, static_cast<ub4>(OCI_ATTR_PARAM_COUNT), session_.errhp_);
+        int cols;
+        res = OCIAttrGet(static_cast<dvoid*>(stmtp_),
+            static_cast<ub4>(OCI_HTYPE_STMT), static_cast<dvoid*>(&cols),
+            0, static_cast<ub4>(OCI_ATTR_PARAM_COUNT), session_.errhp_);
 
-    if (res != OCI_SUCCESS)
-    {
-        throw_oracle_soci_error(res, session_.errhp_);
+        if (res != OCI_SUCCESS)
+        {
+            throw_oracle_soci_error(res, session_.errhp_);
+        }
     }
 
     return cols;