Skip to content

Commit 5b141cb

Browse files
authored
Update socketapi.cpp
Signed-off-by: Rello <Rello@users.noreply.github.com>
1 parent 22bbb59 commit 5b141cb

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

src/gui/socketapi/socketapi.cpp

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -407,6 +407,14 @@ void SocketApi::slotNewConnection()
407407
}
408408

409409
#ifdef Q_OS_WIN
410+
// SECURITY NOTE:
411+
// This is the central trust boundary for incoming Windows SocketAPI clients.
412+
// We gate new connections here (before listener registration and command
413+
// dispatch) using named-pipe client process/session validation.
414+
//
415+
// This check improves local IPC hardening but is not a complete security
416+
// boundary against a fully compromised same-user environment. Command-level
417+
// validation and user interaction requirements remain important.
410418
QString trustError;
411419
if (!isTrustedWindowsPipeClient(socket, &trustError)) {
412420
qCWarning(lcSocketApi) << "Rejecting untrusted SocketAPI client:" << trustError;

0 commit comments

Comments
 (0)