Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-105034

QDataStream cannot serialize containers > 2Gi elements

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • P2: Important
    • 6.7.0 FF
    • 6.2.4, 6.3.1, 6.4.0 Beta2
    • Core: Serialization
    • None
    • 13
    • fd48ce0b7 (dev)
    • Foundation Sprint 92, Foundation Sprint 93

    Description

      The code fails on 64-bit platforms:

      QByteArray ba;
const QString str =  ~~~ more than 2Gi characters ~~~;
QDataStream(&ba) << str;
QString restored;
QDataStream(ba) >> restored;
assert(restored == str);

      Ditto all other containers.

      The problem seems to be that the serialisation format only allocates a four-octet length field.

      There doesn't appear to be something you can do, unless you change the format to include a longer length indicator.

      Attachments

        Issue Links

          is required for

          Task - A task that needs to be done. QTBUG-103541 Finish migration to qsizetype [qtbase/src/corelib/serialization]

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Open
          resulted from

          Task - A task that needs to be done. QTBUG-103531 Finish migration to qsizetype [qtbase/src/corelib/text]

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          mentioned in

          Page Loading...

          For Gerrit Dashboard: QTBUG-105034
          # Subject Branch Project Status CR V
          427159,1 QString: remove narrowing cast from qsizetype/size_t to uint dev qt/qtbase Status: ABANDONED +2 0
          510834,13 Add support for containers > 4 Gi elements in QDataStream dev qt/qtbase Status: MERGED -1 0

          Activity

            People

              oyheskes Øystein Heskestad
              mmutz Marc Mutz
              Vladimir Minenko Vladimir Minenko
              Alex Blasche Alex Blasche
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes