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

Crash in Compositor::Observer::compositor() with nullptr access

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • P1: Critical
    • 6.5.4, 6.6.1, 6.7.0 FF
    • 6.5.3
    • WebEngine
    • None
    • 888f786ff (dev), e6c7af122 (6.6), 81e9907fc (6.5)

    Description

      Have no reproducer, just see such crashes among our users crash reports.

      Unhandled exception thrown: read access violation.
      this->**m_binding** was nullptr.
      

      Call stack:

      >	Qt6WebEngineCore.dll!QtWebEngineCore::Compositor::Observer::compositor() Line 100	C++
       	Qt6WebEngineCore.dll!QtWebEngineCore::RenderWidgetHostViewQtDelegateItem::onBeforeRendering() Line 401	C++
       	[Inline Frame] Qt6Core.dll!QtPrivate::QSlotObjectBase::call(QObject *) Line 374	C++
       	Qt6Core.dll!doActivate<0>(QObject * sender, int signal_index, void * * argv) Line 4036	C++
       	Qt6Core.dll!QMetaObject::activate(QObject * sender, const QMetaObject * m, int local_signal_index, void * * argv) Line 4097	C++
       	[Inline Frame] Qt6Quick.dll!QQuickWindow::beforeRendering() Line 853	C++
       	Qt6Quick.dll!QQuickWindowPrivate::renderSceneGraph() Line 637	C++
       	Qt6Quick.dll!QSGRenderThread::syncAndRender() Line 736	C++
       	Qt6Quick.dll!QSGRenderThread::run() Line 942	C++
       	Qt6Core.dll!QThreadPrivate::start(void * arg) Line 292	C++
       	[External Code]	
      

       

      Looks like a race. I guess

          if (!m_binding)
              return nullptr;
      

      should be guarded by the mutex as well.

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          People

            qt_webengine_team Qt WebEngine Team
            studiosus Vladimir Belyavsky
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes