Skip to content

Commit 7fd154c

Browse files
committed
Set unsupported to true on error in FreeDesktopScreenSaver
This avoids possible memory leaks when the editor exits.
1 parent b5bdb88 commit 7fd154c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

platform/linuxbsd/freedesktop_screensaver.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ void FreeDesktopScreenSaver::inhibit() {
7979
if (dbus_error_is_set(&error)) {
8080
dbus_error_free(&error);
8181
dbus_connection_unref(bus);
82-
unsupported = false;
82+
unsupported = true;
8383
return;
8484
}
8585

@@ -116,6 +116,7 @@ void FreeDesktopScreenSaver::uninhibit() {
116116
DBUS_TYPE_INVALID);
117117

118118
DBusMessage *reply = dbus_connection_send_with_reply_and_block(bus, message, 50, &error);
119+
dbus_message_unref(message);
119120
if (dbus_error_is_set(&error)) {
120121
dbus_error_free(&error);
121122
dbus_connection_unref(bus);
@@ -125,7 +126,6 @@ void FreeDesktopScreenSaver::uninhibit() {
125126

126127
print_verbose("FreeDesktopScreenSaver: Released screensaver inhibition cookie: " + uitos(cookie));
127128

128-
dbus_message_unref(message);
129129
dbus_message_unref(reply);
130130
dbus_connection_unref(bus);
131131
}

0 commit comments

Comments
 (0)