Interbase: Handle EXECUTE BLOCK statements correctly

Since an EXECUTE BLOCK statement can have a mix of ? and :var syntax
then a special case for this needs to be added so that it does not try
to convert the :var parts into positional placeholders as they need to
kept as-is when preparing such a statement.

Fixes: QTBUG-83152
Change-Id: Iff891207ad6dea1681a1b3a335acbbbb668b465d
Reviewed-by: Christian Ehrlicher <ch.ehrlicher@gmx.de>
(cherry picked from commit 65afcef2173cabe297778d19dda3198595820cfa)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>

1 files changed, 7 insertions, 0 deletions

diff --git a/src/sql/kernel/qsqlresult.cpp b/src/sql/kernel/qsqlresult.cpp
index 69c9dcbac9..9a580e8bae 100644
--- a/src/sql/kernel/qsqlresult.cpp
+++ b/src/sql/kernel/qsqlresult.cpp
@@ -126,6 +126,13 @@ QString QSqlResultPrivate::positionalToNamedBinding(const QString &query) const
 
 QString QSqlResultPrivate::namedToPositionalBinding(const QString &query)
 {
+    // In the Interbase case if it is an EXECUTE BLOCK then it is up to the
+    // caller to make sure that it is not using named bindings for the wrong
+    // parts of the query since Interbase uses them literally
+    if (sqldriver->dbmsType() == QSqlDriver::Interbase &&
+        query.trimmed().startsWith(QLatin1String("EXECUTE BLOCK"), Qt::CaseInsensitive))
+        return query;
+
     int n = query.size();
 
     QString result;