Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Freezes the screen xfreerdp in LinuxMint when switching from full screen mode (Ctrl+Alt+Enter) #3666

Open
ava57r opened this issue Dec 24, 2016 · 16 comments

Comments

@ava57r
Copy link

ava57r commented Dec 24, 2016

Use - version This is FreeRDP version 2.0.0~dev (git n/a)
OS - Linux Mint 18.1 Serena
Desktop - Cinnamon 3.2.7

Connect to Windows server 2008 RDP gateway with option /f
When switching from full screen mode pressing (Ctrl+Alt+Enter) 3 times

  1. Full screen mode -> window mode
  2. Window mode -> Full screen mode
  3. Full screen mode - window mode
    FreeRDP freezes
    Lost icons and panels, window freerdp becomes transparent

syslog - empty
run terminal (Ctrl+Alt+F1) and kill proccess

@akamuza
Copy link

akamuza commented Sep 4, 2017

Ubuntu 16.04, Intel graphics G33 , FreeRDP version 2.0.0-dev (latest nightly build).
Same problem.
After switching several times to/from fullscreen - screen just freezes and nothing helps but reboot.
Using +toggle-fullscreen doesn't help.

@wolfstefan
Copy link

I figured out that this issue was introduced with the commits 48be081 and 7ddd15d. To be more precise, calling XMoveWindow before xf_SendClientEvent seems to break it.

The attached patch removes all calls of XMoveWindow in this case and fixes the problem as far as I have tested it. Moreover, it prevents the window from getting lower after each switch between fullscreen and non-fullscreen. However, I think that there is a reason for the XMoveWindow calls and therefore this patch might break something else. It would be nice if one of the developers could take a look at it.

P.S.: thanks to all developers for the amazing work on FreeRDP :)

fullscreen.diff.txt

@akamuza
Copy link

akamuza commented Sep 7, 2017

In my case XMoveWindow is a good idea, because when I exit fullscreen for the very first time then the top bar of my rdp window is hidden by Ubuntu's top toolbar, and thus I can not see my close,minimize buttons of the rdp window.
And because of that I have to enter/exit fullscreen once more in order to move rdp window a bit lower.

Of cause that's not a big deal, jast an additional information on XMoveWindow

@wolfstefan
Copy link

That's a good point. In this case it should be enough to call XMoveWindow after xf_SendClientEvent when leaving fullscreen. Doing this would give back the behavior that the window gets lower each time switching and it shouldn't break switching more than once on Mint.

@wolfstefan
Copy link

This issue still occurs on the current master and up-to-date Linux Mint 18.2. I don't know enough about X11 to say whether that should work and might be rather a bug in Cinnamon. Therefore, it would be nice if one of the developers who is more aware of X11 could comment on this issue.

@akallabeth akallabeth added this to the 2.0-rc1 milestone Nov 20, 2017
@akallabeth
Copy link
Member

Could not reproduce with current nightlies, could you give some more details?

@akallabeth
Copy link
Member

It would also be helpful to get a broken run with /log-level:trace output attached.

@wolfstefan
Copy link

Have you tried it with Linux Mint and Cinnamon as desktop? Linux Mint with Cinnamon is the only system on which the problem occurred. I couldn't reproduce the issue on the other systems I tested (Fedora with Gnome and Ubuntu with Unity).

I attached the output but I cut the NLA part since I don't know if it includes any credentials.
rdp_log.txt

@akallabeth
Copy link
Member

@wolfstefan Tried with cinnamon on debian (don't have mint ready to use), no such issues at all. Also nothing suspicious in the log...

@wolfstefan
Copy link

Can you please tell me which Cinnamon version (cinnamon --version in terminal) you use? I will then test it with your version on Linux Mint.

@akallabeth
Copy link
Member

3.2.7-4 is the cinnamon debian package (stretch/stable)

@akallabeth akallabeth modified the milestones: 2.0-rc1, 2.0-rc2 Nov 23, 2017
@wolfstefan
Copy link

@akallabeth Are you sure that you switched between fullscreen and non-fullscreen at least twice? Switching only once doesn't trigger the issue. I'm asking since I can also reproduce the issue with a newly installed Debian 9 with Cinnamon.

I found out that the issue was introduced on Cinnamon side between version 2.4 and 2.6 with the following commit: linuxmint/muffin@c055801 . That commit changes the window manager's default placement mode (dconf value org.cinnamon.muffin.placement-mode) from automatic to center. So everyone who is affected by the issue can set that value back to automatic as a workaround.

@akallabeth
Copy link
Member

@wolfstefan Did test that with at least 5x but will check the settings. (Maybe some older settings were still around in my user home)

@ava57r
Copy link
Author

ava57r commented Dec 11, 2017

$ uname -a
Linux pc.localdomain 4.13.16-202.fc26.x86_64 #1 SMP Thu Nov 30 15:39:32 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

$ lsb_release -a
LSB Version:	:core-4.1-amd64:core-4.1-noarch
Distributor ID:	Fedora
Description:	Fedora release 26 (Twenty Six)
Release:	26
Codename:	TwentySix

$ cinnamon --version
Cinnamon 3.6.6

This problem is reproduced
OS installed in may 2017

In the mode of trace window shows that the xfreerdp to become transparent and logging of actions continues in the terminal
rdp_trace_11122017.txt

@akallabeth
Copy link
Member

Ok, finally got it to reproduce.
Special settings necessary:

  • Use default settings of cinnamon (create new user)
  • Single monitor setup (could not reproduce with secondary monitor attached)

@bmiklautz bmiklautz modified the milestones: 2.0-rc2, 2.0-rc3 Mar 26, 2018
mmattes pushed a commit to mmattes/FreeRDP that referenced this issue Jul 22, 2018
mmattes pushed a commit to mmattes/FreeRDP that referenced this issue Jul 22, 2018
@akallabeth akallabeth modified the milestones: 2.0-rc3, 2.0.0 Jul 31, 2018
@akallabeth
Copy link
Member

Looks like no easy solution is available (thank you, nice little fullscreen mess with X11 window managers) without breaking something else.
Will delay this for the time being.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants