ListDictionary handle NULL list argument.

Merge pull request #3296 from akallabeth/clipboard_file_fix

Fix CB_FILECONTENTS_REQUEST message decoding.

Fix CB_FILECONTENTS_REQUEST message decoding.

Do not require optional fields to be present as required by
[MS-RDPECLIP] 2.2.5.3 File Contents Request PDU (CLIPRDR_FILECONTENTS_REQUEST)

Merge pull request #3294 from bmiklautz/misc_build

Misc build and test fixes

Merge pull request #3287 from bmiklautz/linking_fixes2

build: add missing dependencies

Merge pull request #3293 from nfedera/mitigate-cbclientaddress-verification

core: allow TS Gateway protocol violation

build: disable -Wl,--no-undefined per default

As building with -Wl,--no-undefined might cause issues when
combined with other linker flags (e.g. with -fsanitize=address see #3289)
it is now disabled per default but can be re-enabled with
-DWITH_NO_UNDEFINED=ON.

It is enabled for ci builds on linux to find undefined symbols.

core: allow TS Gateway protocol violation

According to [MS-RDPBCGR 2.2.1.11.1.1.1] the TS_EXTENDED_INFO_PACKET
structure's cbClientAddress field must include the _mandatory_ NULL
terminator of the clientAddress field in its byte count.
However, connections proxied via Microsoft's TS Gateway set the
cbClientDir value to 0.

Tests: Disable TESTS_WTSAPI_EXTRA per default

TESTS_WTSAPI_EXTRA was mistakenly always enabled if BUILD_TESTING
was enabled.

Merge pull request #3285 from rjcorrig/kb3148851

winpr: KB3148851: Updates to Russian Time Zones for 2016

build: add missing dependencies

Add all missing dependencies found when building with
STATIC_CHANNELS=OFF.

Fixes #3281

KB3148851: Updates to Russian Time Zones for 2016

Merge pull request #3274 from bmiklautz/linking_fixes

Fix build regressions

Merge pull request #3276 from nfedera/nf-fix-ms-android-protocol-violation

core: allow ms android client protocol violation

core: allow ms android client protocol violation

According to [MS-RDPBCGR 2.2.1.11.1.1.1] the TS_EXTENDED_INFO_PACKET
structure's cbClientDir field must include the _mandatory_ NULL
terminator of the ClientDir field.
However, since version 8.1.31.44, the Microsoft Remote Desktop Client
for Android sets cbClientDir to 0.

Merge pull request #3275 from nfedera/fix-server-establish-keys-typo

core: fix broken rdp security (server side)

core: fix broken rdp security (server side)

- fixed typo in rdp_server_establish_keys

build: add missing dependencies

Add all missing dependencies found with --no-undefined. Since
dependencies aren't exported anymore (if not required) it is no necessary
to explicitly list all required libraries.

Merge pull request #3273 from realjiangms/fix_android_crash_rotate_3258

Fix android rotation crash

build: use --no-undefined if available

Fix android rotation crash

Merge pull request #3254 from akallabeth/avc444_cmd_arg

Added advanced gfx command line options.

Cleaned up empty if.

Merge pull request #3259 from bmiklautz/overlinking

Fix overlinking

usbdrc: remove unused dependencies

First shot on fixing over linking

If a target is linked against libraries with cmake
(target_link_libraries) and the libraries are not marked as PRIVATE
they are "exported" and in case a other target is linked against this
target it is also linked against *all* (not private) libraries.

Without declaring private libraries PRIVATE a lot of over linking
(linking against unneeded libraries) was done.

winpr/TestFileGetStdHandle: rename variable stdout

stdout shouldn't be used as variable name (can't even be used on
windows).

Added advanced gfx command line options.

Merge pull request #3243 from hardening/typofix

Fixed a typo in debug messages

Merge pull request #3240 from akallabeth/drive_fixes

Small Drive fixes

Fixed a typo in debug messages

Replace SetFilePointerEx with SetFilePointer.

Print device name when loading channel.

Merge pull request #3164 from akallabeth/windows_UnixChangeFileMode

Implemented UnixChangeFileMode for windows.

Merge pull request #3215 from akallabeth/avc_444

Avc 444 support

Merge pull request #3204 from akallabeth/rdp_file_preference_fix

Rdp file preference fix

Assume the update rectangle is inclusive.

Fixed comments.

Renamed GFX defines to conform to spec.

Implemented GFX AVC444 support.

Implemented YUV444 related primitives.

Unified RDPGFX_RECT16 and RECTANGLE_16

Merge pull request #3226 from rjcorrig/#3198

winpr: Updates time zones and fixes bias values

Merge pull request #3234 from bmiklautz/rdkt_version

Increase rdtk version to 2.0.0

winpr_detect_windows_time_zone should have void argument

Parse RDP and assistance files in parser.

The command line parser is now populating the settings with
the RDP or assistance file content before parsing the rest
of the arguments.

Removed windows extra handling for RDP files.

Increase rdtk version to 2.0.0

rdtk was initially 1.0.0 then changed to 0.0.1.
To have a clean cut and prevent problems on systems which already have
FreeRDP in a version where rdtk was 1.0.0 increase it to 2.0.0

Updated ConvertToUnicode return check.

Merge pull request #3207 from akallabeth/mic_crash_fix

Fixed error handling for channel load failures.

Fixed double free.

Merge pull request #3214 from hardening/fix_systemd_appender

Make systemd appender honor layout and logs off

Merge pull request #3225 from mfleisz/fix_set_err_info

freerdp: Fix possible crash when setting error info in server-mode

winpr: Updates time zones and fixes bias values

Merge pull request #3223 from comicfans/wayland_fix

only compile wayland client when WAYLAND_FOUND

freerdp: Fix possible crash when setting error info in server-mode

Merge pull request #3224 from mfleisz/win32_bld_fix

winpr: Fix definition of PathFileExists on Win32

winpr: Fix definition of PathFileExists on Win32

only compile wayland client when WAYLAND_FOUND

Reverted WTS API changes.

Fixed return value for failed malloc.

Fixed error handling for channel load failures.

Merge pull request #3209 from akallabeth/android_translation_fix

Reverted translation of internal VALUE arrays.

Merge pull request #3221 from bmiklautz/fix_nightlies

pkg/deb: adapt channel paths

Merge pull request #3212 from realjiangms/fix_default_audin

audin: Fixed default sys argument for audin

Removed untranslatable strings from translated files.

pkg/deb: adapt channel paths

Merge pull request #3213 from akallabeth/android_build_fixes

Android build and settings fixes

Merge pull request #3177 from akallabeth/codec_reset_fix

Fixed codec reset, now resetting resolution too.

Make systemd appender honor layout and log off

Replaced version patch with make argument.

Updated OpenSSL version in script.

Fixed settings load/store.

Fixed paralell building issue.

audin: Fixed default sys argument for audin
PR #3173 fixed argument parsing. However it breaks the usage without specifying subsystem: "/microphone"
In particular, aFreerdp microphone redirect is broken as it exactly uses "/microphone"
This PR fixes the regression.
Fix: We should not call CommandLineParseArgumentsA if argc is only 1 (same case as rdpsnd_main.c)

Merge pull request #3210 from hardening/env_fix

Don't require HOME env var to be set for server-side code

Don't require HOME env var to be set for server-side code

Marked non translatable strings as such.

Reverted translation of internal VALUE arrays.

Merge pull request #3172 from realjiangms/android_launch_by_scheme_2720

android: Allow freerdp mobile version to be launched from URI (freerdp://)

android: Allow freerdp mobile version to be launched from URI (freerdp://)
It would be good if we have a easy way to call aFreeRDP in another Android APP (Requirement also mentioned in #2720)
We can define a scheme (freerdp://) as unified way to launch FreeRDP from another APP or browser and connect to compatible RDP server
1. Define scheme freerdp://
2. General form could be freerdp://user@hostname:port/connect?key1=value&key2=-&key3=%2b&key4=
3. [user] part would be translated to /u:
4. [hostname:port] would be translated to /v:
5. The [user@hostname:port] part would be used as app title, currently it's just the progress dialog title
6. query parameters would be translated to command line arguments. Later same arguments will overwrite the formers:
    a. key1=value:  => /key1:value
    b. key2=-:  => -key2
    c. key3=%2b  => +key3 (%2b is url encoded +)
d. key4=  => /key4
e. Especially, drive=sdcard will be properly handled with local sdcard path. On my device it will be translated to /drive:sdcard,/storage/emulated/0

Owing to the refactor work in PR #3097, we now pass same command line argument to JNI for freerdp settings.
We just need to make the SessionActivity accept freerdp scheme and translate argument from URI form to command line form.

Merge pull request #3181 from akallabeth/wlog_filter_fix

Respecting filter in WLog_GetLogLevel

Merge pull request #3206 from akallabeth/openssl_unification

Moved assistance to winpr crypto.

Set variable to NULL to prevent double free.

Moved assistance to winpr crypto.

Merge pull request #3086 from DavBfr/fix-rdpdr

Fixes for rdpdr

Merge pull request #3200 from dwmw2/master

winpr: Bump API version to 2.0

Use common command line parser.

Reparse command line after RDP or assistance file.

Fix memory leaks if parsed multiple times.

Merge pull request #3202 from akallabeth/file_bsd_fix

Fixed FreeBSD support for filetime settings.

Fixed FreeBSD support for filetime settings.

winpr: Bump API version to 2.0

In commit 12bd0ec8 ("winpr: BUMP the API version to 1.2") it was stated
that the intention was to allow multiple winpr versions to be installed
in parallel.

However, it's not sufficient to change just the *minor* version of the
library. The soname is still 'libwinpr.so.1', and thus we still can't
coexist with earlier versions — in particular, we can't install FreeRDP2
in parallel with existing distribution packages of earlier versions.

Bump the soname so it does actually work

Merge pull request #3194 from nfedera/fix-convertfromunicode-hardening

ConvertFromUnicode fixes and misc hardening

Merge pull request #1726 from grpomega/patch-3

Need Week instead of Day for TZ transistion

fix logon info processing

since there have been reports that certain windows versions send
zero values in the cbDomain field of the Logon Info Version 2
(TS_LOGON_INFO_VERSION_2) struct we allow zero values for cbUserName
and cbDomain in rdp_recv_logon_info_v1 and rdp_recv_logon_info_v2.

Merge pull request #3173 from akallabeth/audin_arg_fix

Fixed argument parsing for audin.

Merge pull request #3196 from akallabeth/exported_symbols_fix

Fixed symbol export for sound plugins.

Fixed symbol export for sound plugins.

ConvertFromUnicode fixes and misc hardening

- Added missing ConvertFromUnicode checks
- If ConvertToUnicode allocates memory, guarantee the null termination
  similar to ConvertFromUnicode's implementation
- Fixed some TestUnicodeConversion.c CTest return values
- Added some CTests for ConvertFromUnicode and ConvertToUnicode
- Misc code and protocol hardening fixes in the surrounding code regions
  that have been touched