core: Move image conversion for icon data into the library

Merge pull request #5384 from m4ntis/fix-readme

Add .md file extention to README

Merge pull request #5339 from akallabeth/update_paint_lock

Unified update->BeginPaint and update->EndPaint

Add .md file extention to README

Merge pull request #5381 from akallabeth/swscale

Added image scaling api for software drawing.

Added image scaling api for software drawing.

For future GFX channel functions an image scaling function is required.
This moves the implementation from wayland client to core library
and adds support for the much faster SWScale library.

Merge pull request #5376 from kubistika/bugfix/rdpgfx-server-v10.6-PDUs

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

Merge pull request #5377 from m4ntis/gfx/fix-caps-confirm-recv

client/rdgpfx_main.c: Fix rdpgfx_recv_caps_confirm_pdu caps length read

Merge pull request #5378 from m4ntis/gfx/manual-frame-ack-caps-sync

rdpgfx/client: Add callbacks for manual caps sending and frame acking

client/rdpgfx_main.h: Formatting

Fixed const correctness of gfx function pointer

Signed-off-by: Mati Shabtay <matishabtay@gmail.com>

rdpgfx/client: Add callbacks for manual caps sending and frame acking

client/rdgpfx_main.c: Fix rdpgfx_recv_caps_confirm_pdu caps length read

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>

Unified update->BeginPaint and update->EndPaint

Since these functions were called from 2 different threads
(main thread or dynamic channel) depending on fastpath or
gfx channel use lock between these.
If not locked the invalid region may be accessed from both
threads and lead to crashes as experienced with nightly df280a7ff

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