[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 'dev'.
Changed from 1ba217a64de33ebcae67dc8b0e6db48624157217 to 390b0b91e98ba2f5a3b0a4d616c0b3930f2f3207
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 2dda9a88b262897bd1ad8090d10df85886d266cd by Marc Mutz on 13/08/2026 at 22:17..
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).

Pick-to: 6.12
Change-Id: I6bd7ad313d7d521f6bfca64f6d54c485732433e0
Reviewed-by: Robert Löhning <[email protected]>
https://invent.kde.org/qt/qt/qtsvg/-/commit/2dda9a88b262897bd1ad8090d10df85886d266cd

Git commit 390b0b91e98ba2f5a3b0a4d616c0b3930f2f3207 by Marc Mutz on 13/08/2026 at 22:18..
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.12 6.11 6.8
Change-Id: I1b207378ba1fec0d355f0bbce5376de4c5c4ed65
Reviewed-by: Robert Löhning <[email protected]>
https://invent.kde.org/qt/qt/qtsvg/-/commit/390b0b91e98ba2f5a3b0a4d616c0b3930f2f3207
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.