Package: xrdp
Version: 0.5.0-2
Severity: normal
Tags: patch
User: debian-...@lists.debian.org
Usertags: debian-edu
If a user first connect to xrdp using rdesktop with one geometry, and
then reconnect using rdesktop with another geometry, a new VNC session
is created. This is really not needed, as RDP support resizing the
desktop. The attached patch, written by my college Dag-Erling Smørgrav,
change xrdp to not include the geometry in the session hash, and thus
allowing users to reconnect to their RDP session no matter what geometry
their RDP client uses.
This patch work best with another patch to improve the desktop resize
handling in xrdp.
Please include this patch in a future version of xrdp.
--
Happy hacking
Petter Reinholdtsen
--- xrdp-HEAD-a9cfc23.orig/sesman/session.c 2012-10-08 07:17:08.258314709 +0200
+++ xrdp-HEAD-a9cfc23/sesman/session.c 2012-10-08 09:35:49.003847933 +0200
@@ -78,8 +78,8 @@
while (tmp != 0)
{
if (g_strncmp(name, tmp->item->name, 255) == 0 &&
- tmp->item->width == width &&
- tmp->item->height == height &&
+ // tmp->item->width == width &&
+ // tmp->item->height == height &&
tmp->item->bpp == bpp &&
tmp->item->type == type)
{
@@ -500,8 +500,8 @@
/* these are the must have parameters */
list_add_item(xserver_params, (long)g_strdup("Xvnc"));
list_add_item(xserver_params, (long)g_strdup(screen));
- list_add_item(xserver_params, (long)g_strdup("-geometry"));
- list_add_item(xserver_params, (long)g_strdup(geometry));
+ // list_add_item(xserver_params, (long)g_strdup("-geometry"));
+ // list_add_item(xserver_params, (long)g_strdup(geometry));
list_add_item(xserver_params, (long)g_strdup("-depth"));
list_add_item(xserver_params, (long)g_strdup(depth));
#ifndef WIN32
