Merge pull request #512 from speidy/devel

xrdp_encoder: do not initialize for non-UNIX rdp clients
master
metalefty 8 years ago committed by GitHub
commit 943dec86ca

@ -145,6 +145,9 @@ struct xrdp_client_info
int disableSSLv3; /* 0 = no, 1 = yes */ int disableSSLv3; /* 0 = no, 1 = yes */
char tls_ciphers[64]; char tls_ciphers[64];
int client_os_major;
int client_os_minor;
}; };
#endif #endif

@ -217,8 +217,25 @@
/* RDP capabilities */ /* RDP capabilities */
#define RDP_CAPSET_GENERAL 1 #define RDP_CAPSET_GENERAL 1
#define RDP_CAPLEN_GENERAL 0x18 #define RDP_CAPLEN_GENERAL 0x18
#define OS_MAJOR_TYPE_UNIX 4 #define OSMAJORTYPE_UNSPECIFIED 0x0000
#define OS_MINOR_TYPE_XSERVER 7 #define OSMAJORTYPE_WINDOWS 0x0001
#define OSMAJORTYPE_OS2 0x0002
#define OSMAJORTYPE_MACINTOSH 0x0003
#define OSMAJORTYPE_UNIX 0x0004
#define OSMAJORTYPE_IOS 0x0005
#define OSMAJORTYPE_OSX 0x0006
#define OSMAJORTYPE_ANDROID 0x0007
#define OSMAJORTYPE_CHROME_OS 0x0008
#define OSMINORTYPE_UNSPECIFIED 0x0000
#define OSMINORTYPE_WINDOWS_31X 0x0001
#define OSMINORTYPE_WINDOWS_95 0x0002
#define OSMINORTYPE_WINDOWS_NT 0x0003
#define OSMINORTYPE_OS2_V21 0x0004
#define OSMINORTYPE_POWER_PC 0x0005
#define OSMINORTYPE_MACINTOSH 0x0006
#define OSMINORTYPE_NATIVE_XSERVER 0x0007
#define OSMINORTYPE_PSEUDO_XSERVER 0x0008
#define OSMINORTYPE_WINDOWS_RT 0x0009
#define RDP_CAPSET_BITMAP 2 #define RDP_CAPSET_BITMAP 2
#define RDP_CAPLEN_BITMAP 0x1C #define RDP_CAPLEN_BITMAP 0x1C
@ -499,22 +516,6 @@
#define CB_ITEMCHANGE 300 #define CB_ITEMCHANGE 300
#define OSMAJORTYPE_UNSPECIFIED 0x0000
#define OSMAJORTYPE_WINDOWS 0x0001
#define OSMAJORTYPE_OS2 0x0002
#define OSMAJORTYPE_MACINTOSH 0x0003
#define OSMAJORTYPE_UNIX 0x0004
#define OSMINORTYPE_UNSPECIFIED 0x0000
#define OSMINORTYPE_WINDOWS_31X 0x0001
#define TS_OSMINORTYPE_WINDOWS_95 0x0002
#define TS_OSMINORTYPE_WINDOWS_NT 0x0003
#define TS_OSMINORTYPE_OS2_V21 0x0004
#define TS_OSMINORTYPE_POWER_PC 0x0005
#define TS_OSMINORTYPE_MACINTOSH 0x0006
#define TS_OSMINORTYPE_NATIVE_XSERVER 0x0007
#define TS_OSMINORTYPE_PSEUDO_XSERVER 0x0008
#define TS_CAPS_PROTOCOLVERSION 0x0200 #define TS_CAPS_PROTOCOLVERSION 0x0200
#define FASTPATH_OUTPUT_SUPPORTED 0x0001 #define FASTPATH_OUTPUT_SUPPORTED 0x0001

@ -74,8 +74,12 @@ xrdp_caps_process_general(struct xrdp_rdp *self, struct stream *s,
g_writeln("xrdp_caps_process_general: error"); g_writeln("xrdp_caps_process_general: error");
return 1; return 1;
} }
in_uint8s(s, 10);
in_uint16_le(s, extraFlags); in_uint16_le(s, self->client_info.client_os_major); /* osMajorType (2 bytes) */
in_uint16_le(s, self->client_info.client_os_minor); /* osMinorType (2 bytes) */
in_uint8s(s, 6);
in_uint16_le(s, extraFlags); /* extraFlags (2 bytes) */
/* use_compact_packets is pretty much 'use rdp5' */ /* use_compact_packets is pretty much 'use rdp5' */
self->client_info.use_compact_packets = (extraFlags != 0); self->client_info.use_compact_packets = (extraFlags != 0);
/* op2 is a boolean to use compact bitmap headers in bitmap cache */ /* op2 is a boolean to use compact bitmap headers in bitmap cache */

@ -52,15 +52,22 @@ struct xrdp_encoder *APP_CC
xrdp_encoder_create(struct xrdp_mm *mm) xrdp_encoder_create(struct xrdp_mm *mm)
{ {
struct xrdp_encoder *self; struct xrdp_encoder *self;
struct xrdp_client_info *client_info;
char buf[1024]; char buf[1024];
int pid; int pid;
if (mm->wm->client_info->mcs_connection_type != 6) /* LAN */ client_info = mm->wm->client_info;
if (client_info->client_os_major != OSMAJORTYPE_UNIX)
{ {
/* temp workaround to avoid black screen in non-supported clients */
return 0; return 0;
} }
if (client_info->mcs_connection_type != 6) /* LAN */
if (mm->wm->client_info->bpp < 24) {
return 0;
}
if (client_info->bpp < 24)
{ {
return 0; return 0;
} }
@ -68,24 +75,24 @@ xrdp_encoder_create(struct xrdp_mm *mm)
self = (struct xrdp_encoder *)g_malloc(sizeof(struct xrdp_encoder), 1); self = (struct xrdp_encoder *)g_malloc(sizeof(struct xrdp_encoder), 1);
self->mm = mm; self->mm = mm;
if (mm->wm->client_info->jpeg_codec_id != 0) if (client_info->jpeg_codec_id != 0)
{ {
LLOGLN(0, ("xrdp_encoder_create: starting jpeg codec session")); LLOGLN(0, ("xrdp_encoder_create: starting jpeg codec session"));
self->codec_id = mm->wm->client_info->jpeg_codec_id; self->codec_id = client_info->jpeg_codec_id;
self->in_codec_mode = 1; self->in_codec_mode = 1;
self->codec_quality = mm->wm->client_info->jpeg_prop[0]; self->codec_quality = client_info->jpeg_prop[0];
mm->wm->client_info->capture_code = 0; client_info->capture_code = 0;
mm->wm->client_info->capture_format = client_info->capture_format =
/* XRDP_a8b8g8r8 */ /* XRDP_a8b8g8r8 */
(32 << 24) | (3 << 16) | (8 << 12) | (8 << 8) | (8 << 4) | 8; (32 << 24) | (3 << 16) | (8 << 12) | (8 << 8) | (8 << 4) | 8;
self->process_enc = process_enc_jpg; self->process_enc = process_enc_jpg;
} }
else if (mm->wm->client_info->rfx_codec_id != 0) else if (client_info->rfx_codec_id != 0)
{ {
LLOGLN(0, ("xrdp_encoder_create: starting rfx codec session")); LLOGLN(0, ("xrdp_encoder_create: starting rfx codec session"));
self->codec_id = mm->wm->client_info->rfx_codec_id; self->codec_id = client_info->rfx_codec_id;
self->in_codec_mode = 1; self->in_codec_mode = 1;
mm->wm->client_info->capture_code = 2; client_info->capture_code = 2;
self->process_enc = process_enc_rfx; self->process_enc = process_enc_rfx;
#ifdef XRDP_RFXCODEC #ifdef XRDP_RFXCODEC
self->codec_handle = self->codec_handle =
@ -94,13 +101,13 @@ xrdp_encoder_create(struct xrdp_mm *mm)
RFX_FORMAT_YUV, 0); RFX_FORMAT_YUV, 0);
#endif #endif
} }
else if (mm->wm->client_info->h264_codec_id != 0) else if (client_info->h264_codec_id != 0)
{ {
LLOGLN(0, ("xrdp_encoder_create: starting h264 codec session")); LLOGLN(0, ("xrdp_encoder_create: starting h264 codec session"));
self->codec_id = mm->wm->client_info->h264_codec_id; self->codec_id = client_info->h264_codec_id;
self->in_codec_mode = 1; self->in_codec_mode = 1;
mm->wm->client_info->capture_code = 3; client_info->capture_code = 3;
mm->wm->client_info->capture_format = client_info->capture_format =
/* XRDP_nv12 */ /* XRDP_nv12 */
(12 << 24) | (64 << 16) | (0 << 12) | (0 << 8) | (0 << 4) | 0; (12 << 24) | (64 << 16) | (0 << 12) | (0 << 8) | (0 << 4) | 0;
self->process_enc = process_enc_h264; self->process_enc = process_enc_h264;

Loading…
Cancel
Save