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

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

transport/mfreerdp: fix async transport

- handle WAIT_TIMEOUT result as error in async transport thread
  if an INFINITE timeout was specified in WaitForMultipleObjects
- fix mfreerdp's async transport handling to not use
  freerdp_get_event_handles/freerdp_check_event_handles if async
  transport is activated

Merge pull request #2620 from nfedera/fix-gdi-cache

gdi: fix nullification of instance->context->cache

gdi: fix nullification of instance->context->cache

Merge pull request #2611 from nfedera/fix-async-transport-thread

transport: fixed multiple errors in async thread

Merge pull request #2613 from oshogbo/gethostname

At least at FreeBSD - the gethostname(3) function can fail.

Merge pull request #2612 from oshogbo/style

Function return bool, so lets return FALSE if 0 or TRUE if 1.

At least at FreeBSD - the gethostname(3) function can fail.
Check returned value.

Function return bool, so lets return FALSE if 0 or TRUE if 1.

transport: fixed multiple errors in async thread

Merge pull request #2605 from nfedera/fix-2015-05-08-01

fixed multiple missing gdi return value checks

Merge pull request #2607 from nfedera/remove_free_null_checks

free can handle NULL perfectly fine

free can handle NULL perfectly fine

shadow fix multi-client hang

Merge from commit d4df3a952b
Conflicts:

server/shadow/shadow_subsystem.c

fixed multiple missing gdi return value checks

mainly gdi_Create* functions

Merge pull request #2594 from baruchsiach/fix-math-2

Don't disable C99 long double math on non Unix platforms

Merge pull request #2577 from hardening/listen_from_socket

Add the ability to have listener from an existing socket

Return FALSE under Win32

Test the number of listening handles

Add the ability to have listener from an existing socket

Merge pull request #2596 from akallabeth/wlog_default_out_fix

Wlog console output to stderr