diff options
author | Axel Spoerl <axel.spoerl@qt.io> | 2023-07-03 16:01:25 +0200 |
---|---|---|
committer | Axel Spoerl <axel.spoerl@qt.io> | 2023-07-04 23:31:51 +0200 |
commit | a7deddba519fc1f6fd637496e92ca5daccf6d453 (patch) | |
tree | fb72c418aa2db57ff2d1c46bf639417bb07318ce /src/plugins/sqldrivers/ibase/qsql_ibase.cpp | |
parent | 3748b194d4de790540aa74db8d65b602e097f415 (diff) |
QIbaseResult: qWarning, when disallowed null parameter becomes zero
QIbaseResult turns null into zero when database field doesn't
accept a null argument.
This patch adds a warning in that case. It would be better to return
with an error, but that breaks existing behavior with code relying on
it.
Fixes: QTBUG-114683
Pick-to: 6.6 6.5
Change-Id: Ib50b7b6b4dd6c51489ba8b355f7baa8b1b14dc15
Reviewed-by: Ievgenii Meshcheriakov <ievgenii.meshcheriakov@qt.io>
Reviewed-by: Christian Ehrlicher <ch.ehrlicher@gmx.de>
Diffstat (limited to 'src/plugins/sqldrivers/ibase/qsql_ibase.cpp')
-rw-r--r-- | src/plugins/sqldrivers/ibase/qsql_ibase.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/plugins/sqldrivers/ibase/qsql_ibase.cpp b/src/plugins/sqldrivers/ibase/qsql_ibase.cpp index 047b64d4d0..6e86640d31 100644 --- a/src/plugins/sqldrivers/ibase/qsql_ibase.cpp +++ b/src/plugins/sqldrivers/ibase/qsql_ibase.cpp @@ -1051,6 +1051,9 @@ 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; } switch(d->inda->sqlvar[para].sqltype & ~1) { case SQL_INT64: |