[qt/qt/qtquick3d-assimp]: Summary of bulk changes made
KDE Git Services - Bulk Change <[email protected]>
| Newsgroups | gmane.comp.kde.cvs |
|---|---|
| Message-ID | <[email protected]> |
Git repository change summary for qt/qt/qtquick3d-assimp Pushed by mirror-service into branch 'upstream/master'. Changed from eb21a7c49874408e950e901009705f8b96df27ed to ba8f4f456446e79594a6131c11674a559a7e07b1 Acknowledgement was received that this change introduces only existing code that has been pushed to another public open source repository. This change contains the following new commits: Git commit ba8f4f456446e79594a6131c11674a559a7e07b1 by GitHub (on behalf of alb3e3) on 17/08/2026 at 12:21.. Fix out-of-bounds copy of the uv transformation property (#6748) * Add texture transform tests for the uv transformation property Covers the regular property and a property whose data is shorter than an aiUVTransform. The latter trips the ai_assert on master; with asserts disabled the memcpy reads 20 bytes out of the shorter property. Export TextureTransformStep so the tests can drive the step directly, as is already done for the other post-processing steps under test. Assisted-by: Claude Code (Claude Opus 5) * Fix out-of-bounds copy of the uv transformation property TextureTransformStep::Execute() copied five floats through &info.mTranslation.x, which writes past that member and over the two following members of the aiUVTransform base - undefined behaviour that a sanitizer flags as an intra-object overflow (assimp#6733). The size was also hard-coded to sizeof(float) * 5, so it read the wrong amount when ai_real is double, and the source length was only checked by an ai_assert that is compiled out in release builds. Copy into a complete aiUVTransform of the correct size instead and skip properties whose data is too short. Assisted-by: Claude Code (Claude Opus 5) --------- Co-authored-by: Kim Kulling <[email protected]> https://invent.kde.org/qt/qt/qtquick3d-assimp/-/commit/ba8f4f456446e79594a6131c11674a559a7e07b1