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

Undefined Behaviour in qsimpledrag.cpp line 207

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • P2: Important
    • 6.2.4, 6.3.0 Beta3
    • 5.5.0
    • GUI: Drag and Drop
    • None
    • Source code inspection
    • 29a7489769bca59630782061789561fb07d77892 c5792dcfd6 (qt/tqtc-qtbase/dev) 32b278b13b (qt/qtbase/6.2.4) 32b278b13b (qt/tqtc-qtbase/6.2.4)

    Description

      I have an scientific application and enabled floating point exceptions. This results in drag&drop operations (at least in my treeview) always giving a floating point invalid-operation exception in line 207 of qtbase/src/gui/kernel/qsimpledrag.cpp. The offending line is:

        if (pos.x() == int(qInf())) { ... }

      that is a conversion of floating point +INFINITY to int, which is I believe is Undefined Behaviour.

      (grepping out of curiousity shows that the same dubious construct is also used in ./qtquickcontrols/src/controls/qquickmenu.cpp, line 405)

      Attachments

        Issue Links

          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-99056 FPE on unknown adreess with ASAN during drag and drop

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

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-45045 SIGFPE in QQuickMenu

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

          Task - A task that needs to be done. QTBUG-69433 Support multiple seats (several users with separate screens/mice/keyboards)

          • Not Evaluated - Not yet evaluated/prioritized
          • Reported

          Task - A task that needs to be done. QTBUG-52430 support multiple pointing device cursors in Qt 6

          • P4: Low - Would be nice to fix, but it's not very important.
          • Reported

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-69167 Floating point exception in QPointF::toPoint() with qInf()s

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-61783 SwipeDelegate left and right components not reacting on touchscreen press

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          For Gerrit Dashboard: QTBUG-52472
          # Subject Branch Project Status CR V
          393144,2 Stop using QGuiApplicationPrivate::lastCursorPosition.toPoint() dev qt/qtdeclarative Status: MERGED -1 0
          393209,9 Make QGuiApplicationPrivate::lastCursorPosition.toPoint() safe to use dev qt/qtbase Status: MERGED +2 0
          394092,2 Stop using QGuiApplicationPrivate::lastCursorPosition.toPoint() 6.3 qt/qtdeclarative Status: MERGED +2 0
          394093,2 Stop using QGuiApplicationPrivate::lastCursorPosition.toPoint() 6.2 qt/qtdeclarative Status: MERGED +2 0
          397257,2 Make QGuiApplicationPrivate::lastCursorPosition.toPoint() safe to use 6.2 qt/qtbase Status: MERGED +2 0
          397259,2 Make QGuiApplicationPrivate::lastCursorPosition.toPoint() safe to use 6.3 qt/qtbase Status: MERGED +2 0
          397260,2 Make QGuiApplicationPrivate::lastCursorPosition.toPoint() safe to use 6.2.4 qt/qtbase Status: MERGED +2 0

          Activity

            People

              srutledg Shawn Rutledge
              jablo Jacob Lorensen
              Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: