[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