Details
-
Bug
-
Resolution: Done
-
P0: Blocker
-
5.11.0 Beta 4
-
None
-
7f0bdc21a8f1a5fbe2afdba61f037a71f00c1d61
Description
Have a QPaintedTextureImage and do an update() from a QFrameAction. (very common in Qt 3D Studio, to update text field contents). This simply does not work due to gltexture.cpp asserting in getOrCreateGlTexture() when trying to retrieve the image data for an already released generator:
for (const Image &img : qAsConst(m_images)) {
const QTextureImageDataPtr imgData = m_textureImageDataManager->getData(img.generator);
Q_ASSERT(imgData);
a bit of logging clearly shows that the TextureImage backend node releases and registers the new generator:
} else if (propertyChange->propertyName() == QByteArrayLiteral("dataGenerator")) {
// Release ref to generator
if (m_generator)
m_textureImageDataManager->releaseData(m_generator, peerId());
m_generator = propertyChange->value().value<QTextureImageDataGeneratorPtr>();
// Request functor upload
if (m_generator)
m_textureImageDataManager->requestData(m_generator, peerId());
}
however gltexture then tries to query with the old (just released) generator.
Note that the key to reproduce is to do updates fast enough, i.e. at least two updates within three frames.
Attachments
Issue Links
- relates to
-
QT3DS-660
Qt 3D bugs and features required for Runtime 2
-
- Withdrawn
-
- resulted from
-
QT3DS-1413
Changing Text.textstring in onUpdate (or via datainput!) crashes in Qt3D
-
- Closed
-
| For Gerrit Dashboard: QTBUG-67989 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 228139,1 | Revert "Load textures in a single frame" | 5.11.0 | qt/qt3d | Status: ABANDONED | 0 | 0 |
| 228147,3 | Remove race in texture data upload | 5.11.0 | qt/qt3d | Status: MERGED | +2 | 0 |