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

Merge pull request #2669 from llyzs/h264

h264: add encoder constant QP rate control.

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

Merge pull request #2644 from xhaakon/shadow-server-disposal

shadow_client: dispose encomsp and remdesk

Merge pull request #2643 from xhaakon/shadow-server-client-disconnect

Don't dispose shadow server before all clients have disconnected

freerdp: patch multiple leaks and OOM errors

winpr-thread: better error message upon a failure of TestThreadExitThread

winpr-thread:
  - added the unit test: TestThreadExitThread
  - fix: ensure thread_list to be up to date before to call ExitThread()
  - possibly resolved: Problems with serial redirection #2389

libwinpr-shell: use getpwnam_r

shadow_client: dispose encomsp and remdesk

Their threads were kept running after the client's exit.

Don't dispose shadow server before all clients have disconnected

Ensure all client threads are disconnected before we stop the server
thread and thus make it available for deletion. We do this because
shadow_client_context_free() accesses rdpShadowServer structure, which
we have to keep alive as long as it is needed.

Calling shadow_client_context_free() after shadow_server_stop() and
shadow_server_free() were called may have caused an application crash.

Merge pull request #2638 from kwitaszczyk/master

Assign peer to settings when a new RDP module is being initialized.

Specify to skip compression with remoteFx frames

Merge pull request #2640 from xsumbe00/master

winsock.c Solaris build fixes

channels/rdpdr: add more server-side error checking

winsock.c Solaris build fixes

channels/rdpdr: treat OOM and status failures in server-side code

channels/rdpdr: cleanup server-side rdpdr code

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

libwinpr-utils: update backtrace utils

Assign peer to settings when a new RDP module is being initialized.

Merge pull request #2637 from nfedera/fix-shadow_client_context_cleanup

shadow/client context cleanup and unused vars fix

Merge pull request #2583 from bceverly/OpenBSD-cleanup

Two minor source code changes to remove warnings

shadow/client context cleanup and unused vars fix

Merge pull request #2630 from hardening/new_2616

Add checks for some XXX_New and XXX_Add functions

Merge pull request #2557 from realjiangms/shadow_fix_hang2395

shadow fix multi-client hang

Merge pull request #2632 from akallabeth/client_bandwidth_measurement_callback

Added client bandwidth measurement result callback.

Fixed declaration of xf_event_action_script_init()

Take in account nfedera's review

Fixed mistake with bitwise operator in previous two commits

Merge branch 'awakecoding' of https://github.com/vworkspace/FreeRDP

Conflicts:
channels/rdpdr/server/rdpdr_main.c

cmake: add PULSE_FOUND variable to FindPulse.cmake

Moved thread code to OpenBSD only

Added client bandwidth measurement result callback.

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

Conflicts:
client/Windows/wf_client.c
libfreerdp/common/assistance.c

Merge pull request #2633 from clivest/progressive_codec_new_surface_fix

Fix rounding error in progressive codec surface new

Merge pull request #2635 from clivest/gdi_gfx_leaked_regions_fix

Fix two memory leaks in gdi/gfx

Fix two cases of leaked regions in gdi/gfx

Fix rounding error in progressive codec

The grid is composed of 64x64 blocks and should not be smaller
than the surface. If width or height were not a multiple of 64
the previous rounding resulted in a grid smaller than the surface.

Fixed last VirtualChannelEntry

Fixed functions that return BOOLs

Merge pull request #2627 from nfedera/fix-x11-polyregion

client/x11: fix + simplify invalidate_poly_region

client/x11: fix + simplify invalidate_poly_region

Merge pull request #2624 from kwitaszczyk/master

QueryContextAttributes shouldn't return an error when a user or a domain is empty.

smartcard: fix compiler warnings

gcc 4.7.2:
warning: passing argument 1 of ‘free’ discards ‘const’ qualifier from
pointer target type

Add checks for some XXX_New and XXX_Add functions

Based on PR #2616

QueryContextAttributes shouldn't return an error when a user or a domain
is empty.

Merge pull request #2623 from nfedera/fix-xfclient-if-semicolons

client/x11: fixed if statements with empty body

client/x11: fixed if statements with empty body

Merge pull request #2622 from kwitaszczyk/master

Pass valid context to CompleteAuthToken function in Negotiate Security Package.

Pass valid context to CompleteAuthToken function in Negotiate Security Package.

Merge pull request #2621 from nfedera/fix-mac-async-transport

transport/mfreerdp: fix async transport