[qt/qt/qtsvg]: 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/qtsvg
Pushed by mirror-service into branch '6.12'.
Changed from d197de7e6d802a0b470d16489b886db8db1751c9 to 5b03bef6fb3740a1d142d0e23689a04061389ef7
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 313fe745b09474b74faa917272e48930d1996e2c by Qt Cherry-pick Bot (on behalf of Marc Mutz) on 13/08/2026 at 22:46..
tst_QSvgHandler: fix some mem-leaks in testCreateAnimateTransformNode()

The test deleted `node` at the end of the test function, but didn't
take into account that there are various QEXPECT_FAIL(..., Abort)
in-between, which thus leak the node.

Fix by holding the node in a unique_ptr.

This doesn't get rid of all leaks in the function, but those are the
leaks that the test function itself is responsible for. The rest will
be fixed in a follow-up.

Amends c9b94e6bdb2b0a031cdba35375572ecf9e94e455 (6.12).

Change-Id: I6bd7ad313d7d521f6bfca64f6d54c485732433e0
Reviewed-by: Robert Löhning <[email protected]>
(cherry picked from commit 2dda9a88b262897bd1ad8090d10df85886d266cd)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
https://invent.kde.org/qt/qt/qtsvg/-/commit/313fe745b09474b74faa917272e48930d1996e2c

Git commit 5b03bef6fb3740a1d142d0e23689a04061389ef7 by Qt Cherry-pick Bot (on behalf of Marc Mutz) on 13/08/2026 at 22:46..
QSvgHandler: make clear when document() ownership is transferred

Previously, QSvgHandler assumed that someone would call document() and
assume ownership of the QSvgDocument. As a result, while QSvgHandler
deleted the document in error cases (potentially causing earlier
callers of document() that held on to the pointer to dangle), it
didn't in its destructor.

As a direct consequence, if nobody took ownership of document() before
~QSvgHandler() runs, the document was leaked. That this is not just a
theoretical issue was shown by tst_QSvgHandler itself (of all cases),
where testCreateAnimateTransformNode() didn't take ownership, and
therefore produced leaks.

As a fix, make QSvgHandler own the QSvgDocument and document() just
observe the stored object. Add a new function, takeDocument(), which
actually transfers ownership of the document to the caller, resetting
QSvgHandler::m_doc, so any further use doesn't use a potentially
dangling pointer.

This fixes the remaining asan-reported leaks in tst_QSvgHandler.

Amends the start of the public history.

Pick-to: 6.11 6.8
Change-Id: I1b207378ba1fec0d355f0bbce5376de4c5c4ed65
Reviewed-by: Robert Löhning <[email protected]>
(cherry picked from commit 390b0b91e98ba2f5a3b0a4d616c0b3930f2f3207)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
https://invent.kde.org/qt/qt/qtsvg/-/commit/5b03bef6fb3740a1d142d0e23689a04061389ef7
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.