Skip to content

Commit

Permalink
RDPNego: do not request Standard RDP Security when not allowed (WAB-1…
Browse files Browse the repository at this point in the history
…0220)

Standard RDP security was always requested, with the new options it is
not required anymore to do so.
  • Loading branch information
mengtan committed May 30, 2024
1 parent 1fda00f commit cce4ffd
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions src/core/RDP/nego.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@
struct RdpNegoProtocols
{
enum {
None = 0x00000000,
Rdp = 0x00000001,
Tls = 0x00000002,
Nla = 0x00000004
Expand All @@ -60,8 +61,9 @@ RdpNego::RdpNego(
, rdp_legacy_fallback(rdp_legacy)
, krb(nla && krb)
, restricted_admin_mode(admin_mode)
, selected_protocol(RdpNegoProtocols::Rdp)
, enabled_protocols(RdpNegoProtocols::Rdp
, selected_protocol(RdpNegoProtocols::None)
, enabled_protocols(
(this->rdp_legacy_fallback ? RdpNegoProtocols::Rdp : 0)
| (this->tls ? RdpNegoProtocols::Tls : 0)
| (this->nla ? RdpNegoProtocols::Nla : 0))
, username(username)
Expand All @@ -76,9 +78,10 @@ RdpNego::RdpNego(
, tls_config(tls_config)
, verbose(verbose)
{
LOG(LOG_INFO, "RdpNego: TLS=%s NLA=%s adminMode=%s",
LOG(LOG_INFO, "RdpNego: TLS=%s NLA=%s Legacy=%s adminMode=%s",
((this->enabled_protocols & RdpNegoProtocols::Tls) ? "Enabled" : "Disabled"),
((this->enabled_protocols & RdpNegoProtocols::Nla) ? "Enabled" : "Disabled"),
((this->enabled_protocols & RdpNegoProtocols::Rdp) ? "Enabled" : "Disabled"),
(this->restricted_admin_mode ? "Enabled" : "Disabled")
);

Expand Down

0 comments on commit cce4ffd

Please sign in to comment.