[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
This is an untested patch for 4.12 and maybe earlier versions. ** Patch added: "0001-Wait-for-lockcommand-to-exit-on-suspend-hibernate.patch" https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/+attachment/5337244/+files/0001-Wait-for-lockcommand-to-exit-on-suspend-hibernate.patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
This is an untested patch for xfce 4.14. ** Patch added: "0001-Wait-for-lockcommand-to-exit.-Bug-10089.patch" https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/+attachment/5337188/+files/0001-Wait-for-lockcommand-to-exit.-Bug-10089.patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Martin Castillo (2-acc-von), oh I thought the function would reserve some memory for the variable. I do not have the source checked out either, but maybe later. Thanks for your contribution. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Why didn't the compiler warn about the uninitialized variable? Don't you use -Wuninitialized (or -Wall for developing)? My first corrected patch is still not checking the return value of g_spawn_command_line_sync(). https://developer.gnome.org/glib/stable/glib-Spawning-Processes.html#g-spawn-command-line-sync Fixed this now. Again, I don't have the source checked out, so I _manually_ edited the patch. It may not apply cleanly. Note for the upstream bug[1], that this function has changed in master. It now calls https://git.xfce.org/xfce/xfce4-session/tree/xfce4-session/xfce-screensaver.c#n484. That function still has that bug in the SCREENSAVER_TYPE_OTHER case. The fix should look similar to what I attached. I might write a patch in a few weeks if you don't beat me to it. :) [1]: https://bugzilla.xfce.org/show_bug.cgi?id=10089 ** Patch added: "check return value of g_spawn_command_line_sync()" https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/+attachment/5258042/+files/xfsm-systemd.c.patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Jarno, just from looking at it, I think your patch is wrong. You passed an uninitialized pointer to g_spawn_command_line_sync(). This function will write to whatever address was passed. That's a memory corruption and probably the source of the error message in comment #47. Please try the attached patch. ** Patch added: "xfsm-systemd.c.patch" https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/+attachment/5258038/+files/xfsm-systemd.c.patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Sean, I think then there is a bug in the locker of your choice (which?), if it does not hide the screen before its command exits. Anyway, I am glad to know that you could use the patch in Xubuntu 18.10. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Just some additional thoughts brought to you by the work of the fella's over at the LXQt project on this same issue: TL;DR: systemd-logind may not be inhibiting long enough for screen- lockers to complete. Variable adjustments can be made. 1. https://github.com/lxqt/lxqt/issues/1515#issuecomment-425371720 and 2. https://github.com/lxqt/lxqt/issues/1515#issuecomment-435624731 ** Bug watch added: github.com/lxqt/lxqt/issues #1515 https://github.com/lxqt/lxqt/issues/1515 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
I've tried this patch in Xubuntu 18.10, and my desktop is still briefly visible when coming back from suspend. I do believe the amount of time it is visible is reduced, but it's still visible nonetheless. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
The "error" you tell in #47 is actually not an error, but warning. It should not stop suspend from working. Theo, did bluesabre test the patch? Did the test fail even if xflock4 returned 0? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Maybe it would work in Xenial that has pm-utils. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
I suspend the system via xfce4-session-logout > suspend. It works with the default xfce4-session package. It works if I rebuild the package with no changes. The problem only occurs if I rebuild the package with your patch added to the Debian patch series. I have asked bluesabre from the Xubuntu team to test your patch too. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Oh, it seems to be dropped from Xubuntu. How did you launch suspend? Try "xfce4-session-logout --suspend" in terminal. xfsm-shutdown-helper seems still to rely on pm-utils on Linux. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
The package pm-utils is not installed. It happens with any screen locker. This is tested in a live session environment inside a VM with the xfce4-session package rebuilt to include your patch. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Do you have pm-utils installed? What if you use another locker in xflock4? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Still not working. Oddly, the log entry is from xfsm-shutdown-helper. After disabling 'lock on suspend' I can suspend just fine. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
You could try replacing
"g_spawn_command_line_sync ("xflock4", NULL, NULL, exit_status, error);"
by
"g_spawn_command_line_sync ("xflock4", NULL, NULL, exit_status, NULL);"
(In git in calls of g_spawn_command_line_async NULL is used for the
error parameter (in xfce-sreensaver.c). Error variable might have some
additional information about the error, though.
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1054299
Title:
Race condition in suspend scripts reveals desktop
To manage notifications about this bug go to:
https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions
--
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
This error is logged in .xsession-errors: (xfsm-shutdown-helper:28111): GLib-WARNING **: 14:20:07.868: GError set over the top of a previous GError or uninitialized memory. This indicates a bug in someone's code. You must ensure an error is NULL before it's set. The overwriting error message was: Failed to execute child process ?/usr/sbin/pm-suspend? (No such file or directory) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Oh, how does it break suspend? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
The approval by upstream was mainly meant for xflock4. Also, I assumed that /general/LockCommand is already included in 4.12.1, but it is not. Your patch for xfsm-systemd.c appears to have the expected effect. However, it breaks suspend in my test environment. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Why does it need to be approved upstream? You asked what if the locker does not exit immediately. It is pointless question with current xflock4 (in Xubuntu releases), as it does not have such lockers (or they are run in background). xlock is not available for Ubuntu. slock is so tiny and fast locker that there is no issue with it. The more complex xflock4 has more features, if desired. Those features can be easily backported to Xubuntu, if desired. I suppose the xflock4 does not need to be approved upstream. In Xubuntu xflock4 has line "light-locker-command --lock \", but it has newer been accepted upstream. So have you been able to test my patch you asked for? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Alright. I see that you have covered most use cases while turning xflock4 into a rather complex shell script. This needs to be approved upstream, but I am not sure if anything will happen.. "The thing is that xflock4 is not fixable, it's a hack that's broken by design and needs to go away." -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
That is handled in xflock4 I attached in https://bugzilla.xfce.org/show_bug.cgi?id=10217 : If such a fallback locker is used, they will be called in background in xflock4. If you have defined such a locker in xfce4-session -> /general/LockCommand configuration variable, you should not set xfce4-session -> /general/LockWaitComplete to true. Otherwise, when you suspend (with expected locking) it will just lock, but suspends only after you unlock. If xflock4 expects a locker not to return before unlock, it will be run in background and its success is determined by checking, if it is still running after short delay. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
What if the locker command does not exit immediately (waits until unlock)? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
The attachment "call xflock4 synchronously and return according to its exit status" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team. [This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.] ** Tags added: patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Please test this patch, if you are able to build xfce4-session. (I created the patch using diff -Nrud) ** Patch added: "call xflock4 synchronously and return according to its exit status" https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/+attachment/5189082/+files/xfsm-systemd.c.patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Oh, yes, albeit the xfce-screensaver.c is present only in newer xfce4-session than what is used by Xubuntu. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
The relevant code should be in the following two files: xfce4-session/xfsm-systemd.c xfce4-session/xfce-screensaver.c -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Theo, sorry, but I can not even find the part of code where the locking is executed as part of suspend to RAM function. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Jarno, would you be able to contribute a patch to improve the functionality? This is a long-standing issue and still not fully fixed. The upstream developers may not be aware of its severity. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Theo, Xfce Bugzilla #10089 should be fixed. Testing this is easiest with xscreensaver: $ pkill light-locker(or whatever screen saver daemon is running.) $ xscreensaver (Install it first, if you have to.) Setup long "fade to black when blancking" duration in the Advanced tab of Screensaver Preferences. Make sure lock when going to sleep is chosen in the security settings of Xfce4 power manager. Suspend and resume If desktop is shown some time before asking password, the bug exists. That said, light-locker seems to be the only locker that Bug #904006 does not occur with (in Xubuntu). light-locker does not show desktop that long time, I suppose. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
** Changed in: xfce4-power-manager Importance: Wishlist => Undecided ** Changed in: xfce4-power-manager Status: Fix Released => New ** Changed in: xfce4-power-manager Remote watch: Xfce Bugzilla #6413 => None -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Jarno, does the race condition still exist in 18.04? If yes, what changes are needed to finally fix this issue? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Launchpad has imported 10 comments from the remote bug at https://bugzilla.xfce.org/show_bug.cgi?id=6413. If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. On 2010-04-26T19:42:54+00:00 Yves-Alexis Perez wrote: Hey, a Debian user asked for a feature for xfpm (see http://bugs.debian.org/579267). Basically what would be nice would be to warn the user if she ticks the “lock screen on suspend” checkbox while there is no screensaver running (or maybe even run it in the background). One problem is that there's no easy way to detect any screensaver, and some of them don't need to run (for example xlockmore). What do you think? Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/0 On 2014-01-02T16:14:35+00:00 Jarno Suni wrote: I think one way to deal with this problem is to not let suspend/hibernate happen, if xflock4 fails (i.e. returns a nonzero exit code). An error message should be shown about it. (It should work similarly, when suspend/hibernate is triggered via Log Out dialog or Action Buttons.) That could be problematic, if user request suspend/hibernate when laptop lid is closed or when battery is low, though. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/14 On 2014-01-02T16:56:10+00:00 Jarno Suni wrote: The error dialog in conjunction with suspend/hibernate request should be such that it gives user certain time to cancel the operation, if xflock4 fails. If user does not cancel, suspend or hibernate state (or maybe shutdown?) should be entered. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/15 On 2014-01-02T17:06:33+00:00 Jarno Suni wrote: Yet, another solution is quit using xflock4 script in Xfce4, and start depending on xautolock for which to add GUI for configuration, but it might be an overkill, if you are going to use xscreensaver or new light- locker anyway. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/16 On 2014-04-29T14:45:53+00:00 Eric Koegel wrote: I went with the simple approach of popping up a dialog box with the option to continue the suspend operation. My reasoning is that you'll see the dialog the first time this happens and install/fix the lock tool issue. If someone feels strongly about this not being enough, let me know (or write a patch :) The change is http://git.xfce.org/xfce/xfce4-power-manager/commit/?id=73ed5e362f7e0754b466d7d0e824ab14fec9cd17 Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/20 On 2014-04-29T19:27:37+00:00 Jarno Suni wrote: If screen is requested to be locked after laptop lid is closed, user can not see the dialog. I suppose some audio alarm could be given that would (hopefully) make user open the lid and see the message. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/21 On 2014-10-12T08:19:36+00:00 Jarno Suni wrote: On the other hand, user may notice that suspend/hibernate does not happen from a laptop LED or hard disk activity without an audio alarm. Besides this is supposed to happen only in rare exception, so I think it is not worth adding any additional alarm. As for the implementation of xfpm_lock_screen, I think it is obsolete to use gnome-screensaver in it explicitly. I guess no distribution uses gnome-screensaver by default. Well, I could not find implementation of xfpm_lock_screen in GIT, so maybe this is not an issue anymore. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/25 On 2014-10-12T09:15:59+00:00 Jarno Suni wrote: Oh, found xfpm_lock_screen :) And gnome-screensaver is not alone :) The other locking methods are used in case xfce4-power-manager is installed without xfce4-session. If gnome-screensaver is kept in the list, I would suggest it is tried as the last option, since if it is installed, it will start even if its daemon is not running and even if xscreensaver's daemon is running. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/26 On 2016-01-28T22:14:54+00:00 Simon Steinbeiß wrote:
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
I just added Unity to this bug. I looked for a way to report this directly to the Unity project, but it wasn't immediately apparent. I'm seeing identical behaviour using stock unity on Ubuntu 15.10. ** Also affects: unity Importance: Undecided Status: New -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
** Also affects: xfce4-power-manager via https://bugzilla.xfce.org/show_bug.cgi?id=6413 Importance: Unknown Status: Unknown -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-power-manager/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Launchpad has imported 5 comments from the remote bug at https://bugzilla.xfce.org/show_bug.cgi?id=10089. If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. On 2013-05-15T05:20:39+00:00 sikke wrote: Running XFCE on Xubuntu 13.04 and noticed that suspend + lock screen doesn't always work as expected. When I'm suspending the system and the option Lock screen when going for suspend/hibernate is used, the system seeems to suspend normally. However, when waking up the system yet again, I see a short glance of the desktop that was on when suspending. Only after that I get the lock screen password prompt. This happens randomly, not always. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/7 On 2013-05-15T05:22:58+00:00 sikke wrote: Running 'xflock4 xfce4-session-logout -s' is a workaround Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/8 On 2014-01-03T12:39:39+00:00 Jarno Suni wrote: Downstream report: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299 The caller of xflock4 should wait until xflock4 exits, i.e. not run xflock4 in background. This is an issue with xfce4-power-manager, as well. Please, see https://bugzilla.xfce.org/show_bug.cgi?id=6413 Especially comments 1 and 2. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/13 On 2014-02-27T17:32:43+00:00 Jarno Suni wrote: Please note that even the workaround given in Comment 1 does not work in principle, if xflock4 uses one of the lockers that start in background. Those include xlock, slock, slimlock or any locker configured with xautolock, when considering the proposed xflock4: http://bug-attachment.xfce.org/attachment.cgi?id=5359 (featured within https://bugzilla.xfce.org/show_bug.cgi?id=10217) Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/14 On 2014-09-21T22:46:48+00:00 Jarno Suni wrote: (In reply to siukola.antti from comment #1) Running 'xflock4 xfce4-session-logout -s' is a workaround Yes, if xflock4 uses xscreensaver, but if it uses light-locker, the script doesn't work according to my experience in Ubuntu Studio 14.04: the logout command requests password in background. Reply at: https://bugs.launchpad.net/ubuntu/+source/xfce4-session/+bug/1054299/comments/18 ** Changed in: xfce4-session Status: Unknown = Confirmed ** Changed in: xfce4-session Importance: Unknown = Medium ** Bug watch added: Xfce Bugzilla #6413 https://bugzilla.xfce.org/show_bug.cgi?id=6413 ** Bug watch added: Xfce Bugzilla #10217 https://bugzilla.xfce.org/show_bug.cgi?id=10217 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Yes, but the race condition still exists, if the system does not wait until screen has been locked before suspending. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Any news on this issue? The bug was originally reported against xscreensaver, and it has been fixed in utopic (bug 1229486). -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
** Changed in: acpi-support (Debian) Status: New = Fix Released -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
Status changed to 'Confirmed' because the bug affects multiple users. ** Changed in: xfce4-power-manager (Ubuntu) Status: New = Confirmed -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
** Project changed: xfce4-power-manager = xfce4-power-manager (Ubuntu) ** Changed in: xfce4-power-manager (Ubuntu) Status: New = Incomplete -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
** Changed in: xfce4-power-manager (Ubuntu) Importance: Undecided = High ** Changed in: xfce4-power-manager (Ubuntu) Status: Incomplete = Triaged -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
** Also affects: xfce4-power-manager Importance: Undecided Status: New -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-power-manager/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: Race condition in suspend scripts reveals desktop
** Summary changed: - race condition in suspend scripts reveals desktop + Race condition in suspend scripts reveals desktop -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: Race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: race condition in suspend scripts reveals desktop
** Summary changed: - screensaver doesn't hide screen during unsuspend + race condition in suspend scripts reveals desktop ** Bug watch added: Debian Bug tracker #593303 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=593303 ** Changed in: acpi-support (Debian) Status: New = Unknown ** Changed in: acpi-support (Debian) Remote watch: Debian Bug tracker #700696 = Debian Bug tracker #593303 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1054299] Re: race condition in suspend scripts reveals desktop
** Changed in: acpi-support (Debian) Status: Unknown = New -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1054299 Title: race condition in suspend scripts reveals desktop To manage notifications about this bug go to: https://bugs.launchpad.net/xfce4-session/+bug/1054299/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
