From 2461783ae96bb40820fcaa6fa03732f383987e43 Mon Sep 17 00:00:00 2001 From: Axel Spoerl Date: Wed, 5 Jul 2023 08:51:49 +0200 Subject: [PATCH] QIbaseResult: null parameter follow up fix a7deddba519fc1f6fd637496e92ca5daccf6d453 implemented a warning for disallowed null parameters. A check, was missing, if the argument is actually null. This patch adds the missing check. Fixes: QTBUG-114683 Change-Id: Iecbd636599a28284a0a9afe2987e48552f3658ff Reviewed-by: Ievgenii Meshcheriakov (cherry picked from commit 634ef449bffad3e97a7141ca6c2df0795407b9fc) Reviewed-by: Qt Cherry-pick Bot --- src/plugins/sqldrivers/ibase/qsql_ibase.cpp | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/plugins/sqldrivers/ibase/qsql_ibase.cpp b/src/plugins/sqldrivers/ibase/qsql_ibase.cpp index 6e86640d316..0e5b515c777 100644 --- a/src/plugins/sqldrivers/ibase/qsql_ibase.cpp +++ b/src/plugins/sqldrivers/ibase/qsql_ibase.cpp @@ -1052,8 +1052,11 @@ bool QIBaseResult::exec() // a value of 0 means non-null. *(d->inda->sqlvar[para].sqlind) = 0; } else { - qWarning() << "QIBaseResult::exec: Null value replaced by zero for"_L1 - << d->inda->sqlvar[para].ownname; + if (QSqlResultPrivate::isVariantNull(val)) { + qWarning() << "QIBaseResult::exec: Null value replaced by default (zero)"_L1 + << "value for type of column"_L1 << d->inda->sqlvar[para].ownname + << ", which is not nullable."_L1; + } } switch(d->inda->sqlvar[para].sqltype & ~1) { case SQL_INT64: