libfreerdp-core: fix code style

libfreerdp-core: Gateway RDP8 BIO correction. Fixes connection losses.

Merge pull request #2716 from realjiangms/shadow_xor_pointer

server/shadow: Fix alpha pointer message so that it allows pixel with both 'xor' and 'and'

Merge pull request #2739 from realjiangms/shadow_resize

server/shadow: Enhancement regarding the screen and resolution - addresize support and fix subRect feature

Merge pull request #2798 from psumbera/master

This array is so big that it eats whole stack for the thread and makes

Merge pull request #2827 from HenryJacques/winpr_cleanup

Remove unused variable in RegQueryValueExA()

Merge pull request #2838 from akallabeth/smartcard_no_readers

Added return value checks for smartcard functions.

Merge pull request #2844 from akallabeth/getcomputername_fix

Getcomputername fix

Merge pull request #2829 from akallabeth/mf_compile_option

media foundation support now a compile time option.

Merge pull request #2785 from MartinHaimberger/mh-channel-fix

Channel hardening and fixing

Remove WIN32ERROR type

All return values are UINT now.

Merge pull request #2846 from llyzs/monitor

Read and write desktop and monitor data.

Merge pull request #2847 from akallabeth/base64_decode_ensure_null_termination

Ensure output of crypto_base64_decode is NULL terminated.

gcc: read and write desktop scale settings in core data.

Ensure output of crypto_base64_decode is NULL terminated.

gcc: read and write monitor extended data.

x509_get_default_name tries to get FQDN before hostname now.

Fixed GetComputerNameExA and GetComputerNameA.

Merge remote-tracking branch 'upstream/master' into mh-channel-fix

Conflicts:
channels/drdynvc/client/drdynvc_main.c
channels/rdpgfx/client/rdpgfx_main.c
client/X11/xf_gfx.c

Added return value checks.

media foundation support now a compile time option.

Remove unused variable

Merge pull request #2823 from awakecoding/master

egfx channel open/close fix, H.264 Media Foundation decoder support

Merge branch 'master' of github.com:FreeRDP/FreeRDP

libfreerdp-codec: fix Windows XP build for H.264 decoder

libfreerdp-codec: fix leaks in H.264 Media Foundation decoder

libfreerdp-codec: initial Media Foundation H.264 decoder support

Merge pull request #2811 from clivest/client_channel_interface_leak_fix

Fix race condition in freerdp_channels_client_load that could cause memory leak

Merge pull request #2817 from FreeRDP/mh-fix-collection-free

fixed misc *_free functions to accept NULL

libfreerdp-codec: add more Media Foundation H.264 code

fixed wrong returntype under windows

Merge remote-tracking branch 'upstream/master' into mh-channel-fix

Conflicts:
channels/smartcard/client/smartcard_main.c
channels/smartcard/client/smartcard_operations.c

fixed misc *_free functions to accept NULL

Following types of collections support now
NULL in the free call:

* ArrayList
* BitStream
* ContdownEvent
* Dictionary
* KeyValuePair
* MessageQueue

misc fixes:

fixed all bugs from the review
checking all WaitFor*Object/s calls

Merge pull request #2815 from FreeRDP/mh-wait-SetLastError

winpr: WaitFor*Ojbect/s set lastError now

winpr: WaitFor*Ojbect/s set lastError now

Merge pull request #3 from bmiklautz/pull/2785

Misc fixes

libfreerdp-codec: start H.264 MediaFoundation support

remdesk,encomps: misc fixes

* only free stream when an error occurs
* ignore unhanded messages instead of throwing an error
* fix memory leak in settings

Fix race condition in freerdp_channels_client_load

If g_pInterface is set to NULL outside the critical section in one thread
while another thread is in the critical section, the channel client context
allocated in the VirtualChannelEntry won't end up getting freed.

The channel client context is normally loaded from g_pInterface in
FreeRDP_VirtualChannelInit and stored in the pChannelInitData, then copied
from there onto the pChannelOpenData structure in FreeRDP_VirtualChannelOpen
and finally freed in freerdp_channels_free.

libfreerdp-codec: add x264 stubs

Update version to 1.4.5

Merge branch 'master' of github.com:FreeRDP/FreeRDP

Merge pull request #2799 from kaliway/scardlistreaders-fix

Smartcard Reader Connection Handling

Merge pull request #2801 from bmiklautz/fix/thread

winpr/thread: small cleanups

Merge pull request #2804 from bmiklautz/fix/misc

Misc fixes

Merge pull request #2793 from rkondratenko/pcsclite-transactions

Fix for transactions support for Smart Cards

rdpdr/printer: ignore unhanded packetID

Ignore packetIDs PRN_CACHE_DATA and PRN_USING_XPS instead of returning
an error.

rdpsnd: include win32error.h

If using rdpsnd externally (for example in other projects) WIN32ERROR
needs to be defined otherwise some compilers might create a warning or
error.

core/listner: fix possible problem in _free

winpr/thread: don't close invalid file descriptors

In case the file descriptor is -1 don't call close.

winrp/thread: don't call pthread_exit on return

As the pthread_exit man page states pthread_exit is called implicitly
when the start function returns.

winpr/event: remove critical section

The critical section isn't required and not used anymore since
commit 8a2220a3d9c5ad70b425a33772f14f3dbe956841.

winpr/thread: small cleanups

Also adds error checking of winpr_StartThread in winpr_StartThread.

Merge pull request #2789 from akallabeth/start_thread_fix

[Thread] Fixed mutex unlocking and return checks.

Removes unused (big big) array. Adds ifdef for debug stuff.

channels/smartcard: fix possible mszReaders leak in smartcard_ListReaders(A/W)_Call

channels/smartcard: make smartcard_ListReaders(A/W)_Call return PCSC error statuses to the server

fixed audioserver

if no data could be read, the ERROR_NO_DATA is returend to
indicate a further read.

fixed some remakes and compiler errors

This array is so big that it eats whole stack for the thread and makes
the process dump core on some systems (e.g. Solaris).

Merge pull request #2751 from realjiangms/shadow_fix_compile_error_win

server/shadow: Fix compile error on win32.

Merge pull request #2 from bmiklautz/pull/2785

rdpsnd: restore old behavior and fixes

rdpsnd: restore old behavior and fixes

* oss: fix function signature
* autodetection - if only /sound is given all enabled
  plugins are tried in order and the first successful loaded is used.
- this restores the previous behavior
* alsa/pulse fix command line parsing - no parameters shouldn't be
  treated as error

Removing private context on hCard so that transaction will properly unlock when context is released.

Fixing condition on SCardIsValidContext to be true when context is valid

Re-enabled PCSC-Lite transactions and fixed smartcard deinitialization so it will release all contexts to unblock calls waiting for transactions

Merge pull request #2791 from bmiklautz/100percent

transport: fix regression introduced with PR #2770

transport: fix regression introduced with PR #2770

When the event is reset in transport_check_fds xfreerdp doesn't work and
consumes 100% CPU (see #2790). On windows this is require otherwise the
CPU consumption is 100% there.
This quick fix only resets the event on windows. It's a working approach
but definitely not the final solution.

Fixed mutex unlocking and return checks.

Merge pull request #2770 from akallabeth/bio_event_reset

Resetting frontBio after read now.

Merge pull request #2723 from akallabeth/pwd_from_stdin

New /from-stdin

Merge pull request #2786 from mfleisz/fix_makecert

winpr/tools: Fixed x509_get_default_name failing with long computer names

fixed compiler errors

winpr/tools: Fixed x509_get_default_name failing with long computer names

Merge remote-tracking branch 'upstream/master' into mh-channel

Conflicts:
channels/audin/client/oss/audin_oss.c
channels/drive/client/drive_main.c
channels/printer/client/printer_cups.c
channels/printer/client/printer_main.c
channels/rail/client/rail_main.c
channels/rdpgfx/client/rdpgfx_main.c
channels/rdpsnd/client/oss/rdpsnd_oss.c
channels/remdesk/client/remdesk_main.c
channels/remdesk/server/remdesk_main.c
channels/tsmf/client/tsmf_media.c

introduced channel error reporting system

The rdpContext gets an event which will
get set if an error occoured in a channel.

If a thread or a void callback has to report an
error it will get signaled by this system.

Merge pull request #2766 from akallabeth/snprintf_to_sprintf_s

Replaced snprintf with sprintf_s

Merge pull request #2782 from bmiklautz/order_does_matter

x11: reorder xf_post_disconnect

x11: reorder xf_post_disconnect

Channels like EGFX need resources like the main window in order to
work correctly. Before the window, GDI,.. is freed it needs to be
ensured that all channels are stopped properly to prevent them to access
already freed resources. Disconnecting the channels first fixed a
possible race condition/SEGFAULT that could occur with remote initiated
disconnects.

Merge pull request #2781 from bmiklautz/fdread

winpr/synch: remove the dependency on winsock.h

winpr/handle: define WINPR_FD_ on all platforms

winpr/synch: remove the dependency on winsock.h

winsock.h pulls in a lot of defines and dependencies that are not
required and partially unwanted in winpr's core (for parts that are not
related to network). In order to get rid of this dependency and have an
independent defines for extended winpr functions the WINPR_FD_* defines
are used internally (and for exposed functions). Where required, like in
WSAEventSelect, the FD_* is mapped to WINPR_FD_*.

Merge pull request #2778 from akallabeth/cert_store_compare_fix

Fixed comparison bug, cleanup and error handling.

Using lowercase hostname for comparison now.
fixed argument mixup.

Fixed comparison bug, cleanup and error handling.

Error checks for BIO_get_event
ResetEvent before reading data from transport BIO.

channel smartcard hardend

channels/rdpgfx: fix TotalDecodedFrames counter

channels/rdpgfx: reset state on channel close

channels/drdynvc: use same status code as mstsc for DVC opening failure

Resetting frontBio after read now.

channels/rdpgfx: improve frame ack handling

Merge pull request #1 from hardening/mhchannel

Mhchannel

Hardened urbdrc channel

Fixed some more checks in rdpsnd channel

Replaced snprintf with sprintf_s

Merge pull request #2764 from akallabeth/pull_fixes

Pull fixes

Added missing include breaking windows build.

Fixed inverted bAttached setting.

If bAttached is TRUE, the resources of the event are not
cleaned up on CloseHandle.