yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

wayland_server: Only check if there's an executable when we check permissions

Browse source 
There's not much point checking that we can convert an executable into a
file if we are not going to use that file.

Signed-off-by: Victoria Fischer <victoria.fischer@mbition.io>
This commit is contained in:
Aleix Pol Gonzalez 2024-04-02 18:26:04 +02:00 committed by Aleix Pol Gonzalez
parent 8c6ebee318
commit 2b0b04235c
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/wayland_server.cpp
View file

@ -158,13 +158,12 @@ public:
return true; return true;
} }
if (client->executablePath().isEmpty()) {
qCDebug(KWIN_CORE) << "Could not identify process with pid" << client->processId();
return false;
}
static bool permissionCheckDisabled = qEnvironmentVariableIntValue("KWIN_WAYLAND_NO_PERMISSION_CHECKS") == 1; static bool permissionCheckDisabled = qEnvironmentVariableIntValue("KWIN_WAYLAND_NO_PERMISSION_CHECKS") == 1;
if (!permissionCheckDisabled) { if (!permissionCheckDisabled) {
if (client->executablePath().isEmpty()) {
qCDebug(KWIN_CORE) << "Could not identify process with pid" << client->processId();
return false;
}
auto requestedInterfaces = client->property("requestedInterfaces"); auto requestedInterfaces = client->property("requestedInterfaces");
if (requestedInterfaces.isNull()) { if (requestedInterfaces.isNull()) {
requestedInterfaces = fetchRequestedInterfaces(client); requestedInterfaces = fetchRequestedInterfaces(client);