Removed duplcate NULL checks.

Fixed NULL access checks.

Fixed freerdp_assistance_parse_address_list

* Return a proper error in case parsing fails.
* Don't duplicate string as we already operate on a duplicate.

Fixed test_msrsc_incident_file_type1 return checks.

Fixed memory leak in reallocate.

Updated windows shadow server assistance usage.

Use blocking sockets in freerdp_tcp_connect_multi

The non blocking connect in freerdp_tcp_connect_multi did not
work reliably.
Fall back to blocking connect (which might take longer until the connection
is established) instead of not being able to connect at all.

Fixed socket cleanup in connect multi.

Disable NLA for remote assistance.

Fixed version check for remote assistance channel.

Fixed assistance unit test.

Fixed event check in freerdp_tcp_connect_multi

Set username from assistance file.

Unified address parsing for assistance files.

assistance v2 support

Merge pull request #5051 from akallabeth/libressl_fix

Fix #5049: LibreSSL does not have SSL_CTX_set_security_level

Fix #5049: LibreSSL does not have SSL_CTX_set_security_level

Merge pull request #5045 from akallabeth/license_double_free_fix

Fixed double free in license_free_binary_blob

Merge pull request #5016 from akallabeth/windows_server_build_fix

Windows server build fix

Fixed double free in license_free_binary_blob

Merge pull request #5041 from akallabeth/compile_fix

Fix #5037: Fix calls to ntlm_print_av_pair_list

Merge pull request #5039 from mcsong/master

Fixed recreate activity issue with bluetooth devices.

Fixed broken DEBUG_RDP log message.

Fix #5037: Fix calls to ntlm_print_av_pair_list

Fixed recreate activity issue when a bluetooth device connected or disconnected

Fixed recreate activity issue when a bluetooth device connected or disconnected.

Merge pull request #5008 from wintersandroid/master

Android library build script fixes, and Android thread lock on remote disconnect.

Merge pull request #5033 from bmiklautz/dev5

new: development cycle dev5

new: development cycle dev5

Merge pull request #5032 from bmiklautz/rc4

new: version 2.0.0-rc4

new: version 2.0.0-rc4

ChangeLog and version updated

Merge pull request #5031 from akallabeth/cve_fixes_final

Cve fixes

Refactored ntlm_av_pairs API

Tightened checks, cleaned up code and improved redability.

Fixed CVE-2018-8784

Thanks to Eyal Itkin from Check Point Software Technologies.

Fixed CVE-2018-8788

Thanks to Eyal Itkin from Check Point Software Technologies.

Fixed CVE-2018-8787

Thanks to Eyal Itkin from Check Point Software Technologies.

Fixed CVE-2018-8786

Thanks to Eyal Itkin from Check Point Software Technologies.

Fixed CVE-2018-8785

Thanks to Eyal Itkin from Check Point Software Technologies.

Fixed CVE-2018-8789

Thanks to Eyal Itkin from Check Point Software Technologies.

Merge pull request #4953 from akallabeth/region_signed

signed gdi regions to adjust negative coordinates

Merge pull request #5030 from akallabeth/pointer_cache_optional

Fix #4864: Register pointer cache after PostConnect is called

Fix #4864: Register pointer cache after PostConnect is called

With #4950 client side pointer implementation was made optional.
This addresses an issue that each client had to call
pointer_cache_register_callbacks on its own.

Fixed invalid NULL check

Merge pull request #5029 from mcsong/master

added korean translation for aFreeRDP

added korean translation

Clip coordinates for lineTo and polyLine

Fixed signed/unsigned warnings.

Fixed glyph cache bounds setting.

Fixed gdi_BitBlt coordinate clipping.

Fixed opaque rect bounding rectangle.

signed gdi regions to adjust negative coordinates

Merge pull request #5021 from akallabeth/profiler_api_rework

Fixed profiler API

Merge pull request #4940 from akallabeth/check_window_order_support

Added checks for order type window support.

Merge pull request #5025 from DustPuppyNet/openssl_1_1_1

fix for connection with openssl 1.1.1

Revert store password variable

* remove obsolete SSLv23_client_method in tls_connect()
* set min TLS Version

Merge pull request #5023 from ileGITimo/master

call xf_SetWindowTitle before XMapWindow

call xf_SetWindowTitle before XMapWindow, so window manager can act on window name

Fixed profiler API

* Duplicate the name internally to avoid issues with stack
* Made API opaque and removed direct dereferencing of struct.

Merge pull request #5018 from hardening/windowpos

cmdline: add window-position argument to set initial window position

Merge pull request #4989 from akallabeth/dsp_encode_resample

Added resampling to freerdp_dsp_encode.

Merge pull request #4976 from hardening/rfx_flush

remotefx: fix rlgr last bits of encoder

Merge pull request #4964 from akallabeth/rdg_fixes

Rdg/RPC and gateway fixes

cmdline: add window-position argument to set initial window position

This is useful if you want the window to be at a given position. The patch also mutualizes
the parsing of <xpos>x<ypos> or <width>x<height> arguments.

Fixed order mask

Allow window orders with AllowUnanouncedOrdersFromServer

Added checks for order type window support.

Merge pull request #5012 from akallabeth/rail_app_icons

Rail app icons

Reenabled and fixed windows server compile.

Fixed windows build warnings.

Merge pull request #5015 from chipitsine/cleanup

cleanup based on PVS analyzer findings

remove not needed check

PVS analyzer: A part of conditional expression is always true: DomainMatch.

remove not needed check

PVS analyzer: A part of conditional expression is always true: bWaitAll.

Fixes as per merge request

Merge pull request #4979 from hardening/CAL

license: support CAL license

Merge pull request #5009 from akallabeth/rdp_version_update

Updated RDP_VERSION definitions.

Merge pull request #5013 from bmiklautz/sspi_symbols

fix [winpr/sspi]: export symbols on all systems

Fixed source format for freerdp_dsp_encode in rdpsnd_server_send_wave_pdu

Add soxr library and include paths.

Added missing define guards.

Allow building without libsoxr support.

Marked custom ADPCM codecs experimental.

Added fallback for supported sample rates if original not supported.
Added format compatibility check between recording input and destination.

Fixed possible NULL pointer dereference

Added libsoxr based resampling.

Added CMake detection support for libsoxr

Refactored freerdp_dsp_resample, checks now done internally.

Fixed format issues with server sound channel (review by @llysz)

Fixed unused function warnings when FFMPEG backend is used.

Added resampling to freerdp_dsp_encode.

Merge pull request #4973 from Devolutions/fix_drive_redirection

Allow fixed drive redirection (Windows) and fix macos drive duplication.

Merge pull request #5001 from akallabeth/mac_clipboard_fix

Fixed #4995: Don't forget to send the NULL of the clipboard string.

fix [winpr/sspi]: export symbols on all systems

Better description and formatting for update_read_icon_info

Made all internal functions static.

Fixed color conversion, log level and buffer types

Fixed rail_string_to_unicode_string size calculation.

xfreerdp: set _NET_WM_ICON to RAIL app icon

Icons on X11 windows are configured using the _NET_WM_ICON property
described in Extended Window Manager Hints. Here we implement converison
from DIB bitmaps used by RAIL to the format expected by _NET_WM_ICON,
and actually set the icon for RAIL app windows.

Both DIB format and _NET_WM_ICON (or rather, Xlib) are weird. Let's
start with RAIL's format. That's the one used in BMP and ICO formats
on Windows. It has some strange properties but thankfully FreeRDP's
freerdp_image_copy() can handle most of them for us. (With an exception
of monochrome and 16-color formats that it does not support. Sorry, but
I'm too lazy to fix them. They are not seem to be used by any real
application either.) The one thing that it can't do is to apply the
alpha transparency bitmask so we have to do it manually. This instantly
reminds us that DIB format has HISTORY: it's vertically flipped and
each must be padded to 4 bytes. Both these quirks having reasonable
(for a certain definition of 'reason') explanations. Such is life.
(Also, 8-bit images require a color palette which we must fill in.)

So okay, now comes _NET_WM_ICON. It is more sane (or rather, easier to
deal with). The bitmap is represented with a tiny [width, height] header
followed by an array of pixels in ARGB format. There is no padding, no
weird color formats. But here's a catch: you can't simply take the
output of freerdp_image_copy() and cast to (unsigned char*) of colors.
We have to allocate an array of C's longs and copy the pixels there,
because that's what Xlib expects (and this is mentioned in the spec).
Simply casting an array of bytes causes crashes on 64-bit systems.
So don't try to cheat or "optimize" and read the docs, kids.

Note that XFlush() call after XChangeProperty(). It's there because it
seems to helps see the icon quicker with Unity on Ubuntu 14.04. I don't
know why. (And Unity does not support _NET_WM_ICON officially. But it
sorta kinda works sometimes.)

Oh, and while we're here, delete some old, unused, and commented out
code that was setting window icons in the past. It's not needed anymore.

xfreerdp: add RAIL icon cache

This commit adds a cache for RAIL application icons. It is (surpisingly)
used to cache icons for remote applications. This mechanism is described
in MS-RDPERP 3.1.1.2 Icon Cache Support and related items.

Note that some (actually, most) of the icons are not cached. These are
marked with CacheId == 0xFF. In order to keep the code clean we do not
introduce a special case and instead use a 'scratch' icon and simply
pretend to return an xfRailIcon from cache.

We're going to set icons via _NET_WM_ICON property which explains
why we use "long" values to store pixel data. The icon conversion
is not implemented in this commit, it's only stubs.

However, we do implement processing of window information orders that
contain new icons or cached references to previously sent icons. Note
that it is important to not fail (i.e., to not return FALSE) if we
cannot find a window for the icon by its ID. The server occasionally
likes to be slow or something and send icon updates for nonexistent
windows. This behavior is mandated by the spec, too:

    MS-RDPERP 3.2.5.1.6 Processing Window Information Orders

    Upon receipt of a Window Information Order for an icon or
    cached icon, as specified in sections 2.2.1.3.1.2.2 and
    2.2.1.3.1.2.3, the client SHOULD locate the local RAIL window
    that corresponds to the WindowId reported in the Hdr field
    and apply the icon updates to the RAIL window. If no such
    window can be found, the client SHOULD ignore the order.

Indeed, we silently ignore such orders now.

libfreerdp-core: fix reading TS_ICON_INFO

The spec says that CbColorTable field is present when Bpp is 1, 4, 8.
Actually, bpp == 2 is not supported by TS_ICON_INFO according to the
spec (though, DIB definitely supports 16-color images).

    MS-RDPERP 2.2.1.2.3 Icon Info (TS_ICON_INFO)

    CbColorTable (2 bytes):
        This field is ONLY present if the bits per pixel (Bpp)
        value is 1, 4, or 8.

Omitting 8-bit value breaks 256-color icons which are incorrectly
read with color and alpha data mixed up.