Skip to content

fix(daemon): suppress GCC 16 -Warray-bounds false positive #113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
zccrs merged 1 commit into from
Oct 10, 2026
Merged

fix(daemon): suppress GCC 16 -Warray-bounds false positive #113

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions src/daemon/DdeSeatdControl.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -388,7 +388,14 @@ void DdeSeatdControl::handleEventSocket() {
m_eventBuffer.append(chunk);
while (m_eventBuffer.size() >= static_cast<int>(sizeof(ControlHeader))) {
ControlHeader header {};
// The loop condition above guarantees the buffer holds at least sizeof(ControlHeader)
// bytes, so this copy is in bounds. GCC 15/16 nevertheless reports -Warray-bounds here:
// it cannot relate QByteArray::size() to the pointer returned by constData(), which may
// be the 1-byte static QByteArray::_empty for an empty byte array.
QT_WARNING_PUSH
QT_WARNING_DISABLE_GCC("-Warray-bounds")
memcpy(&header, m_eventBuffer.constData(), sizeof(header));
QT_WARNING_POP
if (header.size > maxControlPayloadSize) {
qWarning("Invalid dde-seatd control message size %u", header.size);
disconnectEventSocket();
Expand Down
Loading