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

Improve safety/debuggability of item change listeners

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Open
    • Priority: P2: Important
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Quick: Core Declarative QML
    • Labels:
      None

      Description

      It's very common to have ASAN report heap-use-after-frees for code that uses item change listeners. Debugging them (at least for me) typically involves a lot of effort, as it requires printing each and every listener added/updated/removed, and then finding which object or item that listener corresponds to.

      It would be helpful to have some macro we could define that enables some form of safety check, such that a warning would be printed when a listener is about to be destroyed without first removing all of its listen "targets".

        Attachments

          Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-51927 [REG:5.6.0-alpha1->5.6.0] adding items to GridLayout with default property alias causes a crash during destruction

          • 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-97423 heap-use-after-free in SwipeView::test_orientation

          • 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-68964 heap-use-after-free in QQuickPathView

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

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-70729 heap-use-after-free when running qmlbench with Menu as delegate

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

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-98769 QML StackView heap-use-after-free on pop()

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Reported
          For Gerrit Dashboard: QTBUG-99629
          # Subject Branch Project Status CR V
          389010,3 Change behavior of and rename removeItemChangeListener dev qt/qtdeclarative Status: NEW 0 0
          388948,5 Remove addItemChangeListener in favor of updateOrAddItemChangeListener dev qt/qtdeclarative Status: DEFERRED -1 0

            Activity

              People

              Assignee:
              mitch_curtis Mitch Curtis
              Reporter:
              mitch_curtis Mitch Curtis
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:

                  Gerrit Reviews

                  There is 1 open Gerrit change