[qt/qt/qtserialport]: 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/qtserialport Pushed by mirror-service into branch 'dev'. Changed from 960e9da04ca07178cb246ea0a02d175093c26fa1 to e72e30c43bff184d404c438e12d86fc731eee4ca 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 ef91504c20fc99a84f813c84c4380242279aae4b by Ivan Solovev on 06/08/2026 at 08:25.. QSerialPortInfo FreeBSD: improve error handling in OID parsing The sysctl() call can theoretically return a zero length or a length that does not represent a whole amount of ints. In such case, the next sysctl() would have accessed the first element of an empty array or passed an incorrect length due to truncating. In both cases that could lead to accessing the memory past the end of the allocated QList. Note that it is unlikely to happen in practice, unless there's a bug in the kernel, because on success at least one component should be available. Still, adding these guards won't hurt. But I'm not picking this patch to ESM branches. Task-number: QTBUG-144899 Pick-to: 6.12 6.11 6.8 Change-Id: I95d00da9eaa2e448283f810ae67bba48cf822af1 Reviewed-by: Tatiana Borisova <[email protected]> https://invent.kde.org/qt/qt/qtserialport/-/commit/ef91504c20fc99a84f813c84c4380242279aae4b Git commit f11174f614ab961965cdf142f2a4c1d57f0e9a34 by Ivan Solovev on 06/08/2026 at 08:25.. QSerialPortInfo FreeBSD: assert the preconditions in a helper function The fact that previousOid should not be empty is a precondition for the nextOid() helper function. This is currently checked at the call sites. Add a Q_ASSERT() to guarantee that a future refactoring does not skip the condition. Task-number: QTBUG-144899 Change-Id: I2133e5bb312d258ec311ebe5d36f8bec03cbc355 Reviewed-by: Tatiana Borisova <[email protected]> https://invent.kde.org/qt/qt/qtserialport/-/commit/f11174f614ab961965cdf142f2a4c1d57f0e9a34 Git commit 37ab9f41f0d164791757aa050e114b537c34fda1 by Ivan Solovev on 06/08/2026 at 08:25.. QSerialPort docs: clarify negative values in set{Read,Write}BufferSize Generally speaking, passing negative value as a buffer size does not make sense. However, in practice negative parameters are handled differently for read and write buffers. Document it to avoid confusion. Task-number: QTBUG-144899 Pick-to: 6.12 6.11 Change-Id: Ic608ebb00b9fe44c1e53f61d9e7172907b9e3110 Reviewed-by: Tatiana Borisova <[email protected]> https://invent.kde.org/qt/qt/qtserialport/-/commit/37ab9f41f0d164791757aa050e114b537c34fda1 Git commit e72e30c43bff184d404c438e12d86fc731eee4ca by Ivan Solovev on 06/08/2026 at 08:25.. QSerialPortInfo FreeBSD: avoid variable shadowing The availablePorts() implementation has two loops over nodes, where the second one shadows the outer variable: for (const NodeInfo &node : nodes) { // ... for (const NodeInfo &node : nodes) { // ... } } Rename the inner variable to descriptionNode, because it's used to populate the device description and manufacturer information. Pick-to: 6.12 Change-Id: I5c1f89ac49b118bcb2c78f4c2bc31e4410f89a46 Reviewed-by: Tatiana Borisova <[email protected]> https://invent.kde.org/qt/qt/qtserialport/-/commit/e72e30c43bff184d404c438e12d86fc731eee4ca