Commit f6269cad authored by David Edmundson's avatar David Edmundson

[libkworkspace] Fix if getCurrentSeat needs to fallback to old approach

The test for whether "/auto" is supported was bogus.

QDBusAbstractInterface::isValid effectively just confirms the service
exists and the path is a legally valid name, not that it has any
interfaces there.

This means the fallback path is not run appropriately.

BUG: 423526
parent ec6ec848
......@@ -432,7 +432,7 @@ static bool getCurrentSeat(QDBusObjectPath *currentSession, QDBusObjectPath *cur
if (man.isValid()) {
*currentSeat = QDBusObjectPath(_SYSTEMD_SEAT_BASE_PATH "/auto");
SystemdSeat seat(*currentSeat);
if (seat.isValid()) {
if (seat.property("Id").isValid()) { //query an arbitrary property to confirm the path is valid
return true;
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment