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.

Allowing UPN syntax for user.

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.

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

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

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.

Added config compatibility for default builds.

Added missing NULL check.

Fixed directory creation checks.

Fixed settings path creation.

Fixed static function arguments.

Using vendor/product scheme for settings now.

Fixed windows known path locations.

h264: add encoder constant QP rate control.

Merge pull request #2658 from hardening/fix_2657

Fix for #2657

Merge pull request #2647 from hardening/fix_message_queue

Fixed MessageQueue and callers

Take in account nfedera's remarks

Check some return values

Added misc checks in rdg.c

Fixed nfedera's remarks

Fixed MessageQueue and callers

Merge pull request #2661 from MartinHaimberger/mh-stream-fix

stream: check stream_new in winpr and libfreerdp

stream: check stream_new in winpr and libfreerdp

also fixed a few things

Merge pull request #2608 from oshogbo/master

Add check to protect memcpy(3) from using NULL pointer.

Merge pull request #2642 from hardening/shadow_fix

Specify to skip compression with remoteFx frames

Merge pull request #2659 from oshogbo/unused

Remove unused variable.

Merge pull request #2660 from nfedera/fix-tsmf-channel

tsmf: fix invalid check and a potential segfault

Merge pull request #2646 from eledoux/thread_list

fixed a synchronization issue on thread_list

tsmf: fix invalid check and a potential segfault

This resuscitates tsmf which was accidently broken in commit f8120919

winpr-thread: added HP's copyright

winpr-thread: thread_list's synchronization based on a pthread_cond_t

Remove unused variable.

Fix for #2657

Merge pull request #2653 from bmiklautz/lzcnt

winpr: add intrin.h

winpr/test: incorporate comments from pull request

* Fix TestIntrinsics - always returned 0
* rename TestWinpr to TestWinPR for consistency
* update gitignores accordingly

winpr-thread: a better error message and comments about thread_list

winpr-thread: added an assertion to enforce that thread_list is correctly set

winpr-thread: thread_list's synchronization based on an Event

winpr: add intrin.h

Exposing lzcnt in crt.h might causes compiler errors (redefinition) with
recent versions of gcc (>=4.9) when winpr is included in other projects.
As lzcnt isn't part of crt according to MSDN and also shouldn't be
exported by default it was moved to intrin.h.

The related test was also moved to the top level directory of winpr.

In c file the name is channelId.

Merge pull request #2654 from oshogbo/typos

Typo

Merge pull request #2652 from nfedera/fix-ios-context-new

client/iOS: fix ios_context_new

client/iOS: fix ios_context_new

- fixed wrong signature since ContextNew is BOOL now
- added error checking

shadow_screen: allow sub-rectangle to span multiple monitors

When we share a sub-rectangle of our desktop, allow it to span over
multiple monitors. Previously it was possible to export an area larger
than one display, however the contents were always cropped to
the boundaries of the selected monitor and the rest was visible for
the client only as a black area .

Add check to protect memcpy(3) from using NULL pointer.

The ntlm_construct_challenge_target_info function can potentially pass NULL as
argument to the ntlm_av_pair_add function (for example DnsDomainName.Buffer).
This NULL finally lands in the CopyMemory (which is macro to the memcpy(3)
function) which can't handle NULL.

Merge pull request #2639 from awakecoding/master

WLog, server-side rdpdr, remote assistance, byteswap, etc