diff options
| author | Thiago Macieira <thiago.macieira@intel.com> | 2016-01-12 13:55:01 -0800 |
|---|---|---|
| committer | Thiago Macieira <thiago.macieira@intel.com> | 2016-01-15 06:38:53 +0000 |
| commit | 434302eae1a0c5d3016fe07c7d343bdbad118a8a (patch) | |
| tree | 61e07e4cc73872cf7923ffc47d0d0d9475554eca /src/corelib/io | |
| parent | 61169b72c24b336af23702fda1e86d1d1c2c8095 (diff) | |
Fix non-Latin1 strings in QJsonDocument on big-endian platforms
QJsonDocument stores the entire JSON tree in its binary format. Since
the binary format is the same on-disk as in-memory, it has a fixed
endianness. But when converting from QString to the little-endian
format, the code accidentally converted twice (from little endian, to
little endian), which resulted in a no-op and the string got stored as
big-endian.
It's like encrypting with double-ROT13.
No new testcase because tst_QtJson::removeNonLatinKey was already
failing and gets fixed by this commit.
Task-number: QTBUG-50419
Change-Id: I408dcb81ba654c929f25ffff1428cc79472bbe13
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>
Reviewed-by: Lars Knoll <lars.knoll@theqtcompany.com>
Diffstat (limited to 'src/corelib/io')
0 files changed, 0 insertions, 0 deletions