diff options
author | Edward Welbourne <edward.welbourne@qt.io> | 2021-06-21 12:03:10 +0200 |
---|---|---|
committer | Edward Welbourne <edward.welbourne@qt.io> | 2021-06-22 20:09:36 +0200 |
commit | c42ebd5ba6f1fb861cabdbe44ab10df55b9b3ada (patch) | |
tree | 41236590fc1152bf0d8c8f0de323cf875d0571a8 /util | |
parent | d201c90c1ca19f9831a6a6590fd9dbaaef103bb0 (diff) |
Fix leak of transaction context handle in QSql's OCI driver
Transaction handles were allocated but nowhere freed. Thanks to Stefan
Latsch for pointing this out and suggesting the fix. Make the handle
yet another member of QOCIDriverPrivate so that close() can free
it. At the same time, also free the service context handle d->svc when
failing do open(); and shuffle the order of frees to be the reverse of
the order of allocations.
Fixes: QTBUG-94246
Pick-to: 6.2 6.1 5.15
Change-Id: I45818ada6d884b06028056d28635390a300e2def
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
Diffstat (limited to 'util')
0 files changed, 0 insertions, 0 deletions