Skip to content
This repository has been archived by the owner on Jul 5, 2022. It is now read-only.

Difference in color palette (256 colors) between FreeRDP (Windows) and native rdp client #27

Open
ToM-MaS opened this issue May 28, 2011 · 7 comments
Milestone

Comments

@ToM-MaS
Copy link

ToM-MaS commented May 28, 2011

Hi everyone,
I'm currently developing the new Remote Desktop Client for ReactOS ( http://www.reactos.org ).
That means I'm using the FreeRDP Windows version.
I'm connecting from Windows 7 to a Windows 2000 Server for this example. (means 8 bit, 256 colors)
I've noticed a color difference between FreeRDP (git version 2011-05-02) and the native mstsc.
This image shows the difference (left freerdp, right mstsc - look at the mspaint color palette at the bottom):
http://www.thomasbrandt.net/ReactOS/FreeRDP/Freerdp_palette_problem.png

That's all.
Keep up the good work.
(Please don't neglect xrdp too much, ReactOS will need it some day :) )

@awakecoding
Copy link
Member

Hi,

The ReactOS project is welcome to replace rdesktop by FreeRDP for their built-in RDP client ;)

I'm willing to give it a try and see if I can find the source of the problem, but I do not have a Windows 2000 Server machine at my disposal. Were you able to reproduce the same issue with a more recent version of Windows?

@ToM-MaS
Copy link
Author

ToM-MaS commented May 30, 2011

Hi!
Yeah, it's the same when I connect to Windows XP SP3 with 8 bit color depth preset in both clients.
http://www.thomasbrandt.net/ReactOS/FreeRDP/Freerdp_palette_problem_XP.png

@otavio
Copy link
Contributor

otavio commented Jun 20, 2011

Can't this issue has been fixed at with the change done in #50?

@awakecoding
Copy link
Member

@otavio: no, it has to do with palette handling, gdi_mono_image_convert() shouldn't be related to this

@dorianj
Copy link

dorianj commented Jun 21, 2011

I can confirm this on Unix using libfreerdp-gdi, I see it with a WinXP SP3 host in 8bit mode.

@otavio
Copy link
Contributor

otavio commented Jun 21, 2011

If you can come up with a fix for it, it would be nice :-)

@dorianj
Copy link

dorianj commented Jun 21, 2011

I'm going to work on it. I have worked with the color palettes in rdesktop before.

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

No branches or pull requests

4 participants