rdpgfx: Fix GFX v10.6 PDUs parsing and naming according to the spec

Merge pull request #5308 from akallabeth/fix_win32_overlinking

Fixed windows overlinking.

Fix smartcard segfault when no devices are present (#5364)

* Don't assume smartcard reader list is not empty.

* Check status of retrieval of smartcard device list before trying to filter it

* style fixes

android_freerdp.c: fixed wrong registered signature (#5371)

Merge pull request #5260 from akallabeth/rail_spec_update

Rail spec update

Merge pull request #5306 from akallabeth/gfx_server_caps_input_checks

Fixed input stream length checks in rdpgfx_recv_caps_advertise_pdu

Fixed typo in RAIL_ZORDER_SYNC

Fixed input stream length checks in rdpgfx_recv_caps_advertise_pdu

5360 segfault using smartcard without any readers (#5361)

* Fix: corrected test for astyle.

* Fix: added test for NULL readers returned by ConvertFromUnicode.

rail_orders.c: Fix wrong callback call on server handshake ex recv (#5345)

xf_cliprdr: detect null terminators more reliably. (#5353)

Clipboard formats containing plain text are specified to be terminated
by a \0 character in MS's documentation on standard clipboard formats:
https://docs.microsoft.com/en-us/windows/desktop/dataxchg/standard-clipboard-formats

xf_cliprdr_server_format_data_response receives pasted data from the
server to transfer to the client, in a sufficiently raw form that the
\0 terminator is still present, so it has to remove it. It does so by
checking only at the very end of the data. But I've observed that when
pasting out of at least one Windows program (namely Outlook 1903 on
Windows 10), the intended paste data arrives in this function followed
by \0 and then a spurious \n. In that situation the null-terminator
removal will fail to notice the \0, and will leave both bogus
characters on the end of the paste.

Fixed by using memchr to find the _first_ \0 in the paste data, which
should not lose any actually intentional data because it's in
accordance with the spec above.

Merge pull request #5354 from ffontaine/master

xf_floatbar.c: fix build without Xfixes

xf_floatbar.c: fix build without Xfixes

Remove unneeded include on Xfixes.h as it is not always available and
not used in xf_floatbar.c

Fixes:
 - http://autobuild.buildroot.org/results/69245e574787bada718c52c805ec137041dc233d

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

FreeRDP#5329 if using OldLicenseBehaviour, don't try to save the Cal since we're not going to try to load it (#5330)

Fix segfault when IniFile_Load is called with invalid input (#5331)

* Fix segfault when IniFile_Load is called with invalid input

* Fix leak in TestIni.c third testcase

* TestIni.c: Refactor in order to avoid some old compilers errors

Merge pull request #5240 from akallabeth/sign_compare_fixes_v2

Sign compare fixes v2

Added optional PreFrameAck callback to RdpgfxClientContext  (#5303)

* RdpgfxClientContext: Add PreFrameAck callback

* client/rdpgfx_main.c: Call PreFrameAck callback before acking frame

* client/rdpgfx_main.c: Fix windows_msbuild_vs2010 CI build

* client/rdpgfx_main.c: Fix error logs log level

Remove unnecessary clipboard data copy.

clipboard: correctly handle flags during renegociatiion (#5323)

Fixed ntlm_av_pair_get_len return, no signed value required

Fixed argument pointer type cast

Fixed sign issues

Fixed unnecessary casts in freerdp_image_copy.

Disabled -Wimplicit-fallthrough for instances where its use is intended.

Fixed broken implicit fallthrough

Removed duplicate argument checks.

Fixed GetModuleFileNameW.

Fixed review remarks.

Fixed SleepEx return value and added log messages for unimplemented features.

Fixed unused argument warnings for tests.

Fixed NULL argument warning.

Added event name to handle.

Fixed NULL argument warning.

Fixed unused argument warnings.

Fixed unused argument warnings.

Silenced unused parameter warnings, added log messages.

Silenced unused parameter warnings, added log messages.

Implemented GetModuleFileNameW

Improved timer implementation and error messages.

Improved error logging for wait functions.

Improved implementation of LoadLibrary functions.

Improved mutex and event functions

* Added name for debugging to handle.
* Implemented *Ex functions

Fixed unicode key down and up messages.

Fixed dead store warnings

Fixed dead store warnings

Fixed dead store warnings

Fixed dead store warnings

Fixed sign-compare warnings.

Fixed unused store warnings.

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed unchecked return warning.

Fixed unused variable warnings.

Fixed sign-compare warnings, removed internal functions from API

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed _config_size return

Fixed sign-compare warnings

Fixed const free warning.

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Cast SEC_* return values to HRESULT (Same as WIN32 API)

Case SCARD_* return values to HRESULT (Same as WIN32 API)

Cast E_* errors to HRESULT (same as defined in WIN32 API)

Addin function pointers now expect const strings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed unused variable warnings.

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed missing return value check.

Fixed sign-compare warnings

Fixed sign-compare warnings

Fixed free checks to eliminate bad free warnings.

Fixed sign-compare warnings

Fixed NSC SSE argument checks.

Merge pull request #5285 from akallabeth/urbdrc_preparations

Channel, log debug and semaphore related fixes

Merge pull request #5309 from smaeul/patch/ntlm

Fix NTLM AvPair lists

Merge pull request #5318 from akallabeth/filetime_fix

Fix #5317: Change file attributes before modifying file time.

Merge pull request #5251 from akallabeth/wayland_cursor_handling_fix

Moved wayland cursor handling to seat.

Merge pull request #5273 from akallabeth/mac_compile_warning_fix

Fixed mac compilation warnings.

Merge pull request #5319 from akallabeth/clip_file_fix

Fixed \0 terimination of clipboard file list