Split optimized YUV implementations to separate files.

Merge pull request #4389 from hardening/primitives

primitives: use INIT_ONCE instead of static booleans

primitives: use INIT_ONCE instead of static booleans

This may prevent some race conditions

Merge pull request #4381 from bmiklautz/suse_asan

fix freerdp-nightly on suse: disable asan

fix freerdp-nightly on suse: disable asan

Compilation fails on 42.3 (gcc 4.8) when address sanitizer is on.

Merge pull request #4379 from akallabeth/memleak_fix

Fixed memory leak.

Fixed memory leak.

Merge pull request #4328 from akallabeth/nightly_sanitize_address

Activated address sanitizer for nightly builds.

Merge pull request #4375 from akallabeth/memleak

Fixed memory leak.

Fixed memory leak.

Merge pull request #4373 from akallabeth/warning_fixes2

Fixes some warnings introduced with last pull

Fixes argument size type, function static where appropriate.

Merge pull request #4367 from akallabeth/android_version_next

Updated gradle build version and dependencies.

Merge pull request #4369 from akallabeth/openh264_update

[android] Openh264 update scripts

Updated OpenSSL version for android.

Updated to OpenH264 1.7 and relocatable binaries.

Updated gradle build version and dependencies.

Merge pull request #4365 from akallabeth/negociate_context_reset_fix

Negociate context reset fix

Fixed missing packageName setup in server NLA

Simplified package name comparisons.

Changed length arguments and return to size_t

Refactored NTLM, functions static where approprate

Refactored kerberos SSPI
* Functions static where approrpriate
* Variables static const where appropriate

Fixed ICU support for CP_ACP

Fix #4306:
* Do not reset context when changing package.
* All functions not exported static.

Merge pull request #4362 from akallabeth/ifdef_fix

Fixed broken #ifdef

Fixed broken #ifdef

Merge pull request #4359 from MartinHaimberger/yuvfix

YUV conversation: fixed luma frame creation

Merge pull request #4358 from akallabeth/ntlm_debug_fix

Fixed #4357: NTLM debug message.

Fixed bitmap capability debug.

Fixed #4357: NTLM debug message.

YUV conversation: fixed luma frame creation

in the conversation function general_RGBToAVC444YUVv2_ANY

Merge pull request #4355 from jukeks/topic/smartcard_status_cbAtrLen

Fix smartcard login by ignoring Status_Call.cbAtrLen as is required by [MS-RDPESC]

[MS-RDPESC] 2.2.2.18 Status_Call.cbAtrLen: Unused. MUST be ignored upon receipt.

The [MS-RDPESC] clearly states in section 2.2.2.18 that Status_Call's
cbAtrLen must not be used when generating Status_Return response. This
is also how FreeRDP 1.1 behaves.

Merge pull request #4349 from akallabeth/primitives_update

Added new and optimized YUV primitives.

Added new and optimized YUV primitives.

* RGBToAVC444YUVv2: Split a RGB frame in luma and/or chroma v2 YUV420 frames
* RGBX specific versions of RGBToAVC444YUV and RGBToYUV420_8u_P3AC4R

Merge pull request #4339 from akallabeth/gdi_sw_crash_fix

Fixed #4336: Initialize rectangle.

Reduced log priority in gdi_CRgnToRect

Merge pull request #4323 from akallabeth/scanbuild_fixes

Scanbuild and other warnings fixed

Merge pull request #4338 from 2asoft/2asoft/fix-html-clipboard

Ensure HTML Format clipboard synthesizer always converts number strings as if they are in base 10

Merge pull request #4333 from akallabeth/drive_param_check_fix

Fixed drive redirection argument check

Merge pull request #4334 from h3xx/fix-ffmpeg-support

Fix building against current ffmpeg

Merge pull request #4332 from hardening/xrandr_and_fixes

Xrandr and fixes

Fixed #4336: Initialize rectangle.

StartHTML and EndHTML values can be left-padded with 0 characters. strtol and friends treat this as base-8 if base is specified as 0.
Because these values are always sent in base-10, fix is to always use base-10

Merge pull request #4335 from floppym/ssse3

Rework SSE2/SSE3/SSSE3 build code

codec/nsc_sse2: add runtime CPU feature check

cmake: do not add -msse2/-msse3 globally

cmake: rework logic to apply -msse2, -msse3, -mssse3 to specific files

If we apply -mssse3 to files that don't need it, gcc may automagically
generate instructions that will not run on certain CPUs. This breaks the
runtime feature detection code (IsProcessorFeaturePresent).

Fixes: https://github.com/FreeRDP/FreeRDP/issues/4308

Provide support for older versions of libavcodec

My method was to find the LIBAVUTIL_VERSION_* at the commit where the
updated AV_ prefixed constants became available, add one "micro" version
to it, then if it's not at least that version, provide the unprefixed
constant. It just so happens that the same commit introduced all
the AV_* constants.

Merge pull request #4327 from akallabeth/disconnect_fix

Exit client thread if freerdp_connect fails

Merge pull request #4329 from akallabeth/sw_gdi_fix

Sw gdi fix drawing problems

Fix future ffmpeg support

The AV_ prefixed constants have been available since
ffmpeg@def97856de6021965db86c25a732d78689bd6bb0 (2015-07-07), or version
2.8.

The non-prefixed constants are set to be removed in the next ffmpeg
release, and were removed in
ffmpeg@302554835e39b79b977ed60c9afe81b44590dfef (2016-12-05).

Fixed drive redirection argument check

Allow special cases '*' and '%' to pass parameter checks.

Fix for #4330

Since ec027bf dynamic resolution is broken when used with egfx. Before that commit
we were tracking a server sent resize by setting a DesktopResize callback. This callback
is called when the desktop is resized by the server. Anyway the problem was that when this
callback is called, the activation sequence is not always completed, which were leading to
some freeze with 2012r2 servers (sending packets before the sequence is finished).
So with the faulty commit, we are tracking server resizes by subscribing to the Actived
event, that is called at the end of a reactivation sequence, so we're sure to not send packets
when not fully activated.
Anyway the issue that shows on (#4330) is that when you use egfx, no reactivation sequence happens,
the server only sends a ResetGraphics message with the new size, and so we miss the resized event.
This fix introduces a new GraphicsReset event, makes the display channel subscribe to that event,
and react accordingly.

Use more xrandr attributes when available and preserve [desktop|device]ScaleFactor

Fix a too restrictive detection of XRandr support.
Set more monitor's attributes like physical size and orientation when XRandr is
available.
Set [desktop|device]ScaleFactor when it has been given on the command line, this
should fix #4324.

micro code style changes

zgfx: add a check

Fix #4299: gdi_CRgnToRect

The coordinates from RDP ROP commands do not (always) have
0,0 as the origin of the drawing operation.
Adjust the coordinates to our local coordinate system.

Activated address sanitizer for nightly builds.

Exit client thread if freerdp_connect fails

freerdp_disconnect must only be called, if the connection
was established. Otherwise all resources are cleaned up twice
leading to double free and other issues.

Added ROP to primary order debug message.

Added ROP to string functions for log messages.

Fixed dead store warnings.

Removed unused functions, fixed feature define guards

Removed unused variables.

Merge pull request #4319 from akallabeth/fullscreen_legacy_fix

Fullscreen without _NET_WM_FULLSCREEN_MONITORS

Fixed unused variable warnings.

Fixed missing initialization warnings.

Fixed smartcard related scanbuild warnings.

Merge pull request #4321 from akallabeth/rdp_parser_fix

Fixed RDP file parser for unicode.

Fixed various scanbuild warnings.

Merge pull request #4303 from krisztian-kovacs-balabit/use-redirection-password

Use redirection password

Fullscreen without _NET_WM_FULLSCREEN_MONITORS

Some window managers do not support _NET_WM_FULLSCREEN_MONITORS.
In that case multimonitor fullscreen does not properly work, so
add a path resizing the window over all screens instead.
Based on @erbth pull request, adding proper X11 atom checks.

Merge pull request #4320 from ondrejholy/coverity-fixes

Coverity Scan fixes

Fixed RDP file parser for unicode.

There was a memory corruption hidden due to ignoring the
buffer size argument.

Merge pull request #4318 from hardening/display_improvement

Dynamic resolution improvements

Merge pull request #4239 from akallabeth/test_memleak_fixes

Test memleak fixes

disp: improve window resizing

To workaround a bug with 2012r2 servers, don't send any resize during renegociation
and don't resize at a too high rate (every 200ms is good enough).

Add an activated event and a Timer event

Added an event that is triggered when the activation sequence is finished.
We also define a timer event that is neat to have for regular operation.

crypto: Improve PER OID calculations

"(oid[0] << 4) & (oid[1] & 0x0F)" statement is always 0. It is not
problem currently because the only OID which is written by this
function should have 0 there. The function to read/write are pretty
limited anyway and can't work properly with all kind of OIDs. Maybe
it would be better to hardcode the OID there without decoding
and encoding. But those functions are already there so let's improve
them a bit according the spec and warn about limited set of
supported OIDs.

See:
https://msdn.microsoft.com/en-us/library/windows/desktop/bb540809

Merge pull request #4313 from hardening/dyn_res_update

disp: implement dynamic resolution for X11

core: Remove redundant stream position changes

Stream_Seek() is used, but consequently Stream_SetPosition() is used
for position obtained by Stream_GetPosition() immediatelly before
Stream_Seek(). Let's remove this stream position changes due to its
redundancy.

tsmf: Prevent string overflow and unterminated strings

Device variable can overflow, or be unterminated. Replace strcpy
by strncpy and be sure that the string is terminated (sizeof() - 1).

core: Fix array overrunning during FIPS keys generation

p is 20 and r is 1 in the last iteration of fips_expand_key_bits,
which means that buf[21] is read (of BYTE buf[21];). However,
the value is not needed, because it is consequently discarded by
"c & 0xfe" statement. Let's do not read buf[p + 1] when r is 1
to avoid this.

Merge pull request #4315 from MrCsabaToth/patch-1

Docu copy-paste error fix when referring to the Android 64 bit config

Documentation copy-paste error fix when referring to the Android 64 bit config

Merge pull request #4312 from bmiklautz/smartcard-cleanup

Smartcard fixes and cleanup

fix client/smartcard: indentation and return value

Integrate feedback from pull request review.

fix smartcard: SCardStatus unicode handling

* fix StatusW_Call to rely and use SCardStatusW
* fix trace call in  StatusW_Call - needs to be called after the sizes
  are set
* unify SCardStatus functions for pcsc - let the internal function handle unicode directly

This fixes an issue with size calculations of SCardStatusW.

disp: implement dynamic resolution for X11

If the display channel is available we use it to allow the user to resize the
xfreerdp window. When the window is resized we announce a new monitor layout and
the server reacts by doing a reactivation sequence to the new size.
The minimum window size is limited to 300x300 as 2012 servers crash horribly
if we send them a smaller layout.

Merge pull request #4305 from hardening/xrandr_fix

Fix XRandr for old systems and MacOSX

fix channel/smartcard: leak in Connect[AW]

smartcard_Connect[AW] leaked the reader buffer in error case.

feat winpr/smartcard: add test for SCardStatus

Add an extensive test for SCardStatus. It's not enabled per default
as it requires a reader with card to be present.

fix channel/smartcard: SCardStatus

Add support for fmszReaderNamesIsNULL. Formating of the touched files
was also updated.

fix channel/smartcard: compiler warnings

* remove unused variables
* remove unused jump label

refactor winpr/smartcard/pcsc: reader/group naming

The PCSC SCard implementation in winpr tried to rename reader and group
names received from PCSC to something similar to what the windows smart
card service would return.

Because of the following reasons this mapping was removed:

* reader names are not standardized
* no mapping of reader name should be required at all
* the mapping added extra complexity
* the mapping didn't produce the same names as if the reader was
  directly connected on windows (or redirected from a windows host)

In case there are situations where this is nevertheless required this
feature can simple be (re-)implemented a part of the smart card channel.

Also the formatting was fixed.

feat winpr: add WINPR_UNUSED macro

WINPR_UNUSED can be used to mark intentionally unused function parameters.

fix channel/smartcard: remove SCardAddReaderName

SCardAddReaderName isn't part of the SCard API.

Note: removing this also removes the possibility to redirect single
smartcard readers with /smartcard:READERNAME. However this features
wasn't implemented in a general way and will be re-added as part of
the smart card channel directly.

fix channel/smartcard: simplify channel variables

Path was not really used and name was duplicated. Use the device->Name
directly.