diff options
author | Jan Arve Sæther <jan-arve.saether@qt.io> | 2018-02-15 14:44:56 +0100 |
---|---|---|
committer | Jan Arve Sæther <jan-arve.saether@qt.io> | 2018-02-16 07:56:08 +0000 |
commit | 6ffb358822db2e0d30fb34853c3222cd866d57c5 (patch) | |
tree | 835072073f0615991e89eb11c6db0fd29403ec1b /src/testlib/doc/src/qttest-index.qdoc | |
parent | 0fb8271a467202990c90321066e40faed640a7a8 (diff) |
un-crash QPlainTestLogger::printMessage()
Commit cf4a6111150d866424ee07bda80a1d38f24ea02d refactored out test
identifier buildup into a standalone function, but it returned the
QTestCharBuffer as a value type, which ultimately caused it to crash:
Unfortunately QTestCharBuffer is not copied correctly: Since it uses the
default copy ctor it will copy the buf pointer and create a deep copy of
the staticBuf pointer. When the dtor was later called it would then end up
calling free(buf) (where buf pointed to the staticBuf of the original
QTestCharBuffer).
Change-Id: Ifa290658be6f077a0d6613451c26aeeffc8df41c
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
Diffstat (limited to 'src/testlib/doc/src/qttest-index.qdoc')
0 files changed, 0 insertions, 0 deletions