resolve trivial issues found by cppcheck

[server/shadow/X11/x11_shadow.c:511]: (error) Uninitialized variable: x
[server/shadow/X11/x11_shadow.c:512]: (error) Uninitialized variable: y

revolve possible null pointer dereference found by cppcheck

[channels/tsmf/client/gstreamer/tsmf_X11.c:472] -> [channels/tsmf/client/gstreamer/tsmf_X11.c:474]: (warning) Either the condition '!decoder' is redundant or there is possible null pointer dereference: decoder.

Merge pull request #3779 from chipitsine/master

resolve a typo found by cppcheck

resolve a typo found by cppcheck

[server/Windows/wf_mirage.c:319]: (error) Invalid number of character '(' when these macros are defined: ''.
[server/Windows/wf_mirage.c:319]: (error) Invalid number of character '(' when these macros are defined: 'DFMIRAGE_LEAN'.
[server/Windows/wf_mirage.c:319]: (error) Invalid number of character '(' when these macros are defined: '_WIN64'.

Merge pull request #3733 from chipitsine/master

Resolve issues detected by cppcheck

Merge pull request #3775 from nfedera/rgb_to_avc444yuv

RGB to AVC444YUV converter with SSSE3 support

Merge pull request #3756 from bigpjo/master

.RDP Password Attribute

an argument check in xf_UpdateWindowArea

Change request from akallabeth

Change to allow username command line parameter to overwrite the .RDP
username

primitives: SSSE3 RGB to AVC444YUV converter

------------------------------+---------+----------+-----------+-------
RGB TO AVC444YUV 1080p 32bit  |   COUNT |    TOTAL |       AVG |    IPS
CPU: 3.1 GHz Core i5-2400     |         |          |           |
------------------------------+---------+----------+-----------+-------
general_RGBToAVC444YUV_ANY    |     500 | 13.0164s | 0.026033s |     38
general_RGBToAVC444YUV_BGRX   |     500 |  3.9584s | 0.007917s |    126
ssse3_RGBToAVC444YUV_BGRX     |     500 |  0.7694s | 0.001539s |    650

primitives: added RGB to AVC444YUV converter

Merge pull request #3772 from akallabeth/sse_test_fixes

Sse test fixes

Merge pull request #3773 from hardening/gfx_and_sample_fixes

Gfx and sample fixes

Fix plugin loading for the sample client

Some missing initialisation bits were preventing the sample client from loading
the dynamic channel.

Add some useful debug info for the egfx channel

Fixed warnings.

Added value comparison for all YCbCr functions.

Reduced test runtime.

Fixed compiler warnings.

Improved YCoCgToRGB tests.

Fixed YUV test result print.

Fixed SSE yCbCrToRGB_16s8u_P3AC4R, improved tests.

Fixed SSE checks and RGBToRGB_16s8u_P3AC4R

Merge pull request #3767 from akallabeth/ssse3_optimize

Ssse3 optimize and sanitize address

Merge pull request #3762 from akallabeth/sspi_init

Use INIT_ONCE for SSPI initialisation. #3471

Merge pull request #3770 from Abhineet-Ayan-Verma/patch-1

Fixed undefined behavior in wf_peer_main_loop function

Merge pull request #3771 from mfleisz/code_cleanup

core: Get rid of useless settings copy

Merge pull request #3769 from akallabeth/queue_fix

libwinpr-utils: fix 3 logic errors

Added GFX surface command profiler.

Closing dynamic channels on shutdown.

Fixed possible memory leak.

core: Get rid of useless settings copy

Merge pull request #3472 from ilammy/x11-cliprdr/raw-data-cache

Fix cached clipboard responses during raw transfers

Fixed alignment checks for SSE yCbCrToRGB

Using aligned RFX buffers.

Refined alignment checks for SSSE3 YUV444ToRGB

Align scanline to multiple of 16, required for ASM

Fixed undefined behavior in wf_peer_main_loop function

Undefined behavior on "wfPeerContext *context", as it is accessed before being initialized or assigned. Fixed.

libwinpr-utils: fix 3 logic errors

Refactored AVC444 decoding to single YUV420 buffer

Since not all H264 decoders support multiple YUV420 output
buffers process H264 decoding and YUV to RGB conversion
sequentially to avoid overriding the input data.

Fixed libavcodec issues.

Reordered SSSE3 YUV444ToRGB

Use bitmap color format in gdi_BitBlt

Fixed freerdp_client_parse_rdp_file_buffer

The parser function needs a modifyable copy of the input buffer.

Print profiler header and footer.

Added FPS in profiler print.

Use aligned malloc/free for GFX surfaces/buffers

SSSE3 YUV444ToRGB process 16 pixel per loop.

Merge pull request #3765 from mfleisz/gdi_resize

gdi: Allow changing buffer even if size remains the same

Merge pull request #3755 from pentagra/master

ifdef's for Cygwin compilation

Merge pull request #3764 from akallabeth/libavcodec

Updated LIBAVCODEC support for H264 decoder.

Updated LIBAVCODEC support for H264 decoder.

gdi: Allow changing buffer even if size remains the same

Merge pull request #3754 from akallabeth/asm3

NEON and SSSE3 decoder optimisations

Use INIT_ONCE for SSPI initialisation. #3471

Use POSIX cp syntax.

Optimized clear codec line copy.

Merge pull request #3758 from akallabeth/rfx_tile_crash

Limit RFX update region to screen.

Fixed YUV444ToRGB, do not discard chroma frame.

Fixed primitives sign test buffer size.

NEON optimized YCoCg conversion.

Added NEON and SSSE3 YCbCr conversion functions.

Added NEON and SSSE3 YUV conversion optimisations.

Merge pull request #3760 from akallabeth/filetime_conversion

Fixed time conversion in FileSetFileTime. #3508

Limit RFX update region to screen.

Fixed time conversion in FileSetFileTime. #3508

client/X11: cache original clipboard data for raw transfers

FreeRDP uses clipboard->data to cache the result of the Windows->X11
clipboard format conversion, and xf_cliprdr_process_selection_request()
immediately provides this result to local applications if they request
the same clipboard format again. This saves us a possibly costly
conversion in case where the user pastes data repeatedly.

However, this caching mechanism did not support raw clipboard transfers
where the unmodified data is passed between two FreeRDP clients. We use
the same XClipboard protocol for this, so the clipboard->data is in play.
We clear the cached value when we receive new data from the server, so
initially raw transfers are fine. But if some local application (e.g.,
a clipboard manager) asks for some data format before the data is pasted
into the second FreeRDP session then clipboard->data will contain the
*converted* data. And this converted cached data will be provided to
the second FreeRDP session as a part of the raw data transfer. Instead
we should have provided the original data.

In order to achieve this we are now caching the original data in the
same way as the converted one, and the original data is now correctly
provided when the second FreeRDP session asks for a raw data transfer.

.RDP Password Attribute

Allow password to be stored in .RDP file and parsed and settings
updated, this will allow for dynamic .RDP files to be created with
complete login credentials, using this method the username, server and
password will no longer be visible within process lists.

Also fixed issue of username and domain being read from .RDP files and
set to null by command line processor.

ifdef's for Cygwin compilation

Merge pull request #3753 from akallabeth/regression_fixes

AVC444 and MIPS fixes

Fixed AVC444 luma/chroma checks.

Added fallback for CMSPAR. (See debian #854689)

Fixed missing chroma data for AVC444

Merge pull request #3752 from akallabeth/rfx_respect_height

Respect width and height in rfx_process_message

Respect width and height in rfx_process_message

Merge pull request #3750 from hardening/gcc_fix

Parses the SupportStatusInfoPdu early capability and send it to clients if supported

Parses the SupportStatusInfoPdu early capability and send it to clients if supported

Merge pull request #3748 from uplusplus/patch-1

Fix tsmf pulse plugin initialisation

bug fix

bug fix

Merge pull request #3747 from nfedera/improve-generic-rgb-to-yuv420

primitives: RGB to YUV420 (non SIMD) optimizations

primitives: RGB to YUV420 (non SIMD) optimizations

- added a dedicated non-simd encoder for BGR[AX] source formats
- simplified the generic RgbToYuv420 encoder and fixed the crash
  with odd dimensions

Merge pull request #3740 from akallabeth/gdi_color_conversion

Fix for GDI color decoding issues.

Added warning for unsupported color depth.

Merge pull request #3743 from nfedera/ssse3_rgbx_to_yuv420

primitives: added SSSE3 rgb to yuv420 encoder

primitives: added SSSE3 rgb to yuv420 encoder

Currently supported source pixel formats are:
- PIXEL_FORMAT_BGRA32
- PIXEL_FORMAT_BGRX32

Support for PIXEL_FORMAT_RGB[XA]32 can simply be added if
required (see the comment in prim_YUV_opt.c).

On my old 3.1 GHz Core i5-2400 the new SSSE3 function can convert
over 900 1080p BGRX frames per second.

The current non-optimized C version (which supports all pixel formats)
can't do more than 40 yuv conversions per second on this cpu.

---------------------------+---------+-------------+-----------+-------
RGB TO YUV420 1080p 32bit  |   COUNT |       TOTAL |       AVG |    FPS
---------------------------+---------+-------------+-----------+-------
general_RGBToYUV420        |     500 |    13.1776s | 0.026355s |     38
ssse3_RGBToYUV420          |     500 |     0.5320s | 0.001064s |    940

Also fixed an error in TestPrimitivesYUV which generated resolutions
with height or width set to zero

Remove redundant condition

[channels/urbdrc/client/data_transfer.c:2310] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.
[channels/urbdrc/client/data_transfer.c:2311] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.
[channels/urbdrc/client/data_transfer.c:2312] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.
[channels/urbdrc/client/data_transfer.c:2313] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.
[channels/urbdrc/client/data_transfer.c:2314] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.

Merge pull request #3717 from akallabeth/prim_fixes

Fixed primitives.

Merge pull request #3735 from mfleisz/cmake_install_pdb_fix

Build: Use correct pdb names when installing with symbols

Fixed GDI color decoding issues.

Merge pull request #3737 from mbaum2000/flush-stdout

Flush stdout when asking about certificate

Merge pull request #3736 from akallabeth/date_legacy_support

Support old CMake versions without TIMESTAMP #3727

Flush stdout when asking about certificate

Support old CMake versions without TIMESTAMP #3727

Removed unused profiler.

Build: Use correct pdb names when installing with symbols

Resolve possible null pointer dereference

[channels/urbdrc/client/isoch_queue.c:124] -> [channels/urbdrc/client/isoch_queue.c:126]: (warning) Either the condition 'if(isoch)' is redundant or there is possible null pointer dereference: isoch.

Fixed argument order of yCbCrToRGB_16s8u_P3AC4R

Added performance profiler to YUV test.

Remove redundant condition

[client/X11/xf_rail.c:205] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:206] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:207] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:208] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:215] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:216] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:217] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:218] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.

Fixed compiler warning.