winpr/thread: fix test compilation on windows

Use DWORD instead of QWORD.

Fix compiler warnings:

gcc 4.7.2

client/X11/xf_keyboard.c: In function ‘xf_keyboard_action_script_init’:
client/X11/xf_keyboard.c:48:6: warning: unused variable ‘exitCode’
[-Wunused-variable]

gcc 4.9

client/X11/xf_client.c: In function ‘xf_client_thread’:
client/X11/xf_client.c:1537:3: warning: ‘inputThread’ may be used
uninitialized in this function [-Wmaybe-uninitialized]

client/ios: fix unchecked strdups

Fix unchecked strdups

* add missing checks
* adapt function return values where necessary
* add initial test for settings

Merge pull request #2686 from bmiklautz/ios/openssl

ios: update openssl build script

Merge pull request #2705 from akallabeth/sspi_fix_v2

Sspi fix v2 - Fixes static build

Merge pull request #2697 from realjiangms/shadow_audio

Shadow server: fix channel disposal and add audio support. fix message mechanism in client

Merge pull request #2698 from realjiangms/shadow_stand_alone

server/shadow: Decouple shadow_subsystem and subsystem implementation detail.

Merge pull request #2699 from realjiangms/shadow_connect_disconnect

server/shadow: add connect/disconnect anchor in subsystem callback

Merge pull request #2701 from akallabeth/upn_syntax_support

Allowing UPN syntax for user argument.

Merge pull request #2684 from Skool/master

Add bépo layout

Merge pull request #2636 from xhaakon/master

Fix crashes in shadow server

Merge pull request #2713 from bmiklautz/divbyzero

core/metrics: prevent division by zero

Merge pull request #2702 from akallabeth/nla_null_identity_support

NLA Use current users credentials if none provided.

Merge pull request #2711 from nfedera/nf-fix-rdprecvmessagechannelpdu

core: message channel pdu broken with rdp security

core/metrics: prevent division by zero

core: message channel pdu broken with rdp security

rdp_recv_message_channel_pdu always read the rdp security header
even if it was already previously read (which is the case if rdp
security is active)

This caused malfunctions and disconnects when heartbeat or bandwidth
autodetect packets were sent/received in rdp security mode.

Credit goes to @MartinHaimberger for identifying the broken code
part.

Setting hostname as default domain now.

Fixed typo and invalid domain name.

Fixed domain setting for windows client.

UPN syntax fix: domain must be empty, not NULL.

Added user setting if no domain provided.

Fixed variable parsed.

Using last @ for domain split.
Command line parser splits username after
parsing all options.

Merge pull request #2706 from nfedera/nf-fix-reset-event

winpr/synch: fix ResetEvent & improve CTest

Merge pull request #2675 from akallabeth/path_make_path

Added PathMakePath function.

winpr/synch: fix ResetEvent & improve CTest

Unless there is an invalid parameter or internal error, ResetEvent
must return TRUE, even if the object is in nonsignaled state.

Also fixed and improved the CTest TestSynchEvent to test for the
expected/correct SetEvent and ResetEvent return values.

Removed duplicate function tables, respecting WITH_NATIVE_SSPI

Using InitSecurityInterfaceEx(0) now.

Fixed cmake warning.

Merge pull request #2703 from akallabeth/unified_sspi

Let SSPI module choose which implementation to use.

server/shadow: integrate comment from @hardening
Return directly at subsystem->ClientConnect
Also change to argument to accept shadow_client instead of freerdp_peer

server/shadow: Integrate comment from @hardening: use switch to handle different message id.

Let SSPI module choose which implementation to use.

NLA identity now a pointer, allowing NULL.

When NLA identity is NULL, the current user context
is used on windows.

Allowing UPN syntax for user.

server/shadow: Decouple shadow_subsystem and subsystem implementation detail.
Make shadow_subsystem standalone so that others can implement customized shadow server with shadow library.

server/shadow: add connect/disconnect anchor in subsystem callback
Other implementation of subsystem may want to track connect/disconnect of each client

This is originally to add audio support. Several fixes included:
1. Introduce message queue in shadow client. No longer use subsytem->MsgPipe->out to deliver message to clients.
We used to use subsytem->MsgPipe->out for messages which need to be sent to client. But it's not correct. Only one client would get the message if multiple client exists
This problem make the fix in PR #2643 incomplete.
Introduced reference count based solution to release resource taken by the message.
Also added APIs for client message delivery.
Also fixed msg pipe in subsystem to clean resource when destroyed.
2. Discard unused StopEvent in client. We actually use quit message instead.
3. Enhance disposal of channels.
Free context for remdesk and encomsp channels. The original fix only stop the threads, but doesn't release resource.
Dispose channels earlier. The channels are built on client->vcm. Disposing channels after client->vcm is closed cause unknown behavior.
Original fix is #2644
4. Start to add audio support.

Merge pull request #2687 from akallabeth/known_hosts_v2

Store SSL fingerprints with host and port

Merge pull request #2692 from nfedera/add-16bit-cursor-conversion

codec/color: support for 8bpp and 16bpp color pointer

Merge pull request #2688 from hardening/fix_packaging

Correct the included files to have a working freerdp-nightly-dev package

Renamed file to known_hosts2.

xfreerdp: xfc->palette wasn't updated with /gdi:sw

When using software gdi (/gdi:sw) the rdp update PDU callback
gdi_palette_update() is used which writes the new colors to
the rdpContext's gdi->palette buffer.
The X11 functions however access xfc->palette which gets only
updated by xf_gdi_palette_update() which is the callback if
/gdi:hw is used.

This commit changes xfc->palette to a pointer which points to
xfc's private buffer with /gdi:hw or to the gdi->palette if
software gdi is used.

codec/color: support for 8bpp color pointer

codec/color: support for 16bpp color pointer

Merge pull request #2682 from nfedera/fix-hresult-winpr-path

winpr/path: more PathCch* fixes

Merge pull request #2690 from llyzs/autodetect_fix

Fix timer precision for network autodetection on Windows

Updated tests.

Added NULL pointer checks, fixed warnings.

Added subject and issuer to saved data.

When a certificate has changed, display not only the
fingerprint but also subject and issuer of old certificate.

Fixed fingerprint comparison, return value was inverted.

Fixed formatting.

Added test for fingerprint read.

Fixed data replace.

Added certificate_get_fingerprint function to read out old one.

autodetect: use high-precision timer.

winpr: add high-precision GetTickCountPrecise for Windows.

Correct the included files to have a working freerdp-nightly-dev package

Added port to certificate warnings.

Using binary mode for fopen.

Opening correct file.

Opening file on use now.

Working unit test for known_hosts functions.

Fixed bugs found due to test.

Added unit test.

Added legacy known_hosts support.

If no entry for the <host> <port> combination
was found in the v2 file and there is a legacy file
check if a matching <host> entry can be found.
In case there is a matching entry and the <fingerprint>
also matches, create a new entry in the v2 file using the
current port.

Using '<host> <port> <fp>' format for known hosts.

Added StrSep function.

Removed CreateDirectoryA, handled by PathMakePath

Added RemoveDirectoryA, clean up test directories.

Added PathMakePath function.

ios: update openssl build script

* remove patch - not required anymore
* add support for arm64 and x86_64 openssl builds
* update documentation

Merge pull request #2670 from akallabeth/windows_config_paths

Adjusted config paths

Merge pull request #2683 from nfedera/fix-shadow-server

server/shadow: resuscitate shadow server

fix French bépo layout id to match the bépo.fr installer

server/shadow: prevent memcopy of NULL area

server/shadow: resuscitate shadow server

shadow_client_refresh_rect returned FALSE if the areas parameter
was NULL. It may only do so if the count parameter is > 0.

winpr/path: Denounce PathAllocCombine and more

Note: This commit does NOT really fix PathAllocCombine!

- print a warning message that the function is buggy and added a code
  comment describing the issues
- fix misuse of the S_FALSE HRESULT in error conditions
- prevent some segfaults
- check result of HeapAlloc

Fortunately PathAllocCombine is unused in FreeRDP

winpr/path: PathCchConvertStyle used S_FALSE

Use E_FAIL instead of S_FALSE which was used to indicate errors
although it is a HRESULT success code.

winpr/path: umimplemented tests print a warning

winpr/path: unimpl. functions should not succeed

Unimplemented PathCch* functions now return E_NOTIMPL instead of the
HRESULT S_OK and print an error message.

winpr: fix PathCchFindExtension

Fix multiple inplementation errors including the incorrect usage of S_FALSE.
Make this function behave like the Windows 8 implementation and
the according MSDN specification.

- return S_OK if the extension was NOT found
- if no extension was found, ppszExt must point to the string's terminating null
- return E_INVALIDARG if pszPath is not null-terminated within the cchPath range
- return E_INVALIDARG if pszPath is NULL
- return E_INVALIDARG if ppszExt is NULL
- return E_INVALIDARG if cchPath is Zero
- return E_NOTIMPL instead of S_OK in PathPathCchFindExtensionW()

Also extended/fixed the TestPathCchFindExtension ctest

Merge pull request #2676 from nfedera/fix-hresult-xxx

winpr: fixed some misuses of S_FALSE

Add Bépo Layout

winpr: fix PathCchAppend

Fix incorrect usage of S_FALSE which was used to indicate errors
although it is a HRESULT success code.
Make this function behave like the Windows 8 implementation and
the according MSDN specification.

- return E_INVALIDARG instead of S_FALSE if pszPath is NULL
- return E_INVALIDARG instead of S_FALSE if pszMore is NULL
- return E_INVALIDARG if cchPath is zero
- return E_INVALIDARG if cchPath is greater than PATHCCH_MAX_CCH
- return HRESULT_FROM_WIN32(ERROR_FILENAME_EXCED_RANGE) if the combined
  path size exceeds cchPath (this is the same HRESULT which the Windows
  implementation returns in this case and which is referred to as
  PATHCCH_E_FILENAME_TOO_LONG on msdn)

Also extended/fixed the TestPathCchAppend ctest

winpr: fix PathCchStripPrefix

And again S_FALSE was incorrectly used to indicate an error.
Fix this and be more compatible to the Windows 8 implementation of
this function:

- return E_INVALIDARG instead of S_FALSE if pszPath is NULL
- return E_INVALIDARG instead of S_FALSE if cchPath < 4
- return E_INVALIDARG if cchPath > PATHCCH_MAX_CCH
- allow stripping of "\\?\c:" and don't require "\\?\c:\"
- verify that the character at the drive letter position is
  actually an alphabetic character
- since the passed pszPath must not necessarily be null terminated
  and we always have enough space after the memmove we can always
  ensure the null termination of the stripped result

Also extended/fixed the TestPathCchStripPrefixctest

winpr: fix PathCchAddExtension

The HRESULT S_FALSE does not indicate an error:
- return E_INVALIDARG instead of S_FALSE
- return HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER) instead of S_FALSE

Also extended/fixed the corresponding ctest

winpr: fix PathCchAddBackslashEx

The HRESULT S_FALSE does not indicate an error:
- return E_INVALIDARG instead of S_FALSE
- return HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER) instead of S_FALSE

Also extended/fixed the corresponding ctest

winpr: fix PathCchAddBackslash

The HRESULT S_FALSE does not indicate an error:
- return E_INVALIDARG instead of S_FALSE
- return HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER) instead of S_FALSE

Also extended/fixed the corresponding ctest

client/windows: invalid usage of S_FALSE HRESULT

S_FALSE is a success code and not an error (note the S_)

Merge pull request #2673 from nfedera/fix-hresult

winpr: fixed HRESULT & SCODE types, added a ctest

winpr: fixed HRESULT & SCODE types, added a ctest

- HRESULT was unsigned which means that until now all usages of the
  SUCCEDED(hr) and FAILED(hr) macros never detected any errors
- Also fixed the (unused) SCODE typedef
- Added new ctest TestTypes

Merge pull request #2672 from akallabeth/img_test_fix

Fixed for TestImage unit test

Writing failure to stderr.

Fixed image test.

Fixed image test.

Merge pull request #2649 from xhaakon/cross-monitor-subrect

shadow_screen: allow sub-rectangle to span multiple monitors

Fixed size of string buffer.

Merge pull request #2669 from llyzs/h264

h264: add encoder constant QP rate control.