Merge pull request #2398 from bjcollins/master

Fix the command line argument array setup for decorations flag

Merge pull request #2401 from bmiklautz/extend_wtsapi

Extend wtsapi with WTSStartRemoteControlSessionEx

wtsapi: add WTSStartRemoteControlSessionEx

WTSStartRemoteControlSession doesn't allow to specify additional flags
therefore add a new extended version WTSStartRemoteControlSessionEx
with an additional "flags" parameter.

The following flags are defined:

REMOTECONTROL_FLAG_DISABLE_KEYBOARD - disable keyboard input
REMOTECONTROL_FLAG_DISABLE_MOUSE    - disable mouse input
REMOTECONTROL_FLAG_DISABLE_INPUT    - disable input (keyboard and mouse)

gitignore: ignore clion files

Fix the command line argument array setup for decorations argument. The initial value was set for the wrong field in the structure. The decorations is correctly initialized to on
anyways, but this is the correct formatting for the array element.

Merge pull request #2392 from nfedera/fix-2015-02-12-01

winpr: stream ctest and missing read/peek types

Merge pull request #2394 from awakecoding/master

TS Gateway Connection and ADM Refactoring

libfreerdp-core: handle TCP/TLS connection at TSG RPC channel level

libfreerdp-core: more tsg in/out channel refactoring

libfreerdp-core: integrate new RTS pdu functions from denis

winpr: stream ctest and missing read/peek types

libfreerdp-core: improve TSG virtual connection and channel management

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

Merge pull request #2386 from bjcollins/master

Fix remote app in xfreerdp.

Merge pull request #2385 from hardening/nego_fix

Fix server-side protocol negociation

Merge pull request #2377 from bmiklautz/wtsapi_tests

wtsapi: add TestWtsApiExtra

Merge pull request #2388 from MartinHaimberger/mh-wtsapi-handler

wtsapi: added handler for LogonUser and LogoffUser

Merge pull request #2379 from streetdragon/master

Replaced ternary operator with condition only

wtsapi: added handler for LogonUser and LogoffUser

Fix remote app in xfreerdp.

Remove use of the visibleOffset, this completely breaks the display of all windows except for the main application window. Instead,
just maintain a local offset correction of the windowOffset.

Apply workaround to determining the workArea for remote app mode.

Fix server-side protocol negociation

Before this patch, RDP security was (wrongly) the fallback when negociating a
security protocol between the client and the server. For example when a client
was claiming TLS-only when connecting to a FreeRDP based-server with RDP security only,
the result of the negociation was that the server started to do RDP security.
The expected behaviour is to send a nego failure packet with error code
SSL_NOT_ALLOWED_BY_SERVER. This patch fixes this.

We also try to handle all cases of failed negociation and return the corresponding
error code.

libfreerdp-core: refactor tsg virtual connection, in/out channels

libfreerdp-core: gateway connection refactoring

libfreerdp-core: move stuff down from transport to tsg layer

Merge pull request #2384 from awakecoding/master

Fix TS Gateway Disconnection (broken by #2383)

libfreerdp-core: fix tsg crash on disconnect

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

Merge pull request #2383 from hardening/transport_cleanup

Cleanups

Fix socket leak when transport is disconnected

Fix compilation with valgrind helpers

Replaced ternary operator with condition only

xfreerdp: fix egfx multimonitor support

Merge pull request #2378 from awakecoding/master

xfreerdp fullscreen fix

xfreerdp: fix fullscreen mode

wtsapi: add TestWtsApiExtra

Move wtsapi tests that only make sense when run within a session
("interactive") to TestWtsApiExtra since they either brake automated
builds or don't make sense if the return value isn't checked.

TestWtsApiExtra is not build by default but can be enabled by specifying
-DTESTS_WTSAPI_EXTRA=ON (this option is only available if BUILD_TESTING
is enabled).

The tests set the ctest label WTSAPI_EXTRA and can therefore also
be run explicitly with ctest -L WTSAPI_EXTRA.

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

Merge pull request #2372 from MartinHaimberger/wtsapiextension

wtsapi: extended wtsapi

Merge pull request #2374 from bjcollins/master

Apply correct exit code to xfreerdp application for certain log offs

Merge pull request #2373 from akallabeth/reconnect_resource_fix

Reconnect resource cleanup fix

Merge pull request #2371 from bmiklautz/terminate_process

winpr/process: handle PIDs <= 0 in TerminateProcess

Merge pull request #2375 from hardening/minimize_diff

Check return value for fcntl()

Merge pull request #2376 from hardening/minimize_diff2

Handle OOM in Stream_New()

Handle OOM in Stream_New()

Check return value for fcntl()

Apply correct exit code to xfreerdp application for log off from certain flavors of Windows.

Fixed codecs_free

Fixed resource cleanup on disconnect.

Fixed resource cleanup on disconnect.

wtsapi: extended wtsapi

- added missing definitions for session change notification
- extended wtsapi to allow remote logon and logoff against the wtsapi (this allows remote wtsapi usage)

winpr/process: handle pids <= 0

TerminateProcess shouldn't call kill if the PID is <=0 because this has
unwanted effects (and is not what TerminateProcess should do):

* with PID == 0 any process in the same process group gets the signal
  sent
* with PID == -1 *every* processes that the running users has
  permissions to gets the signal sent
* with PID < -1 the process within the same process group and -PID gets
  the signal send

For more details see kill(2).

libwinpr-winsock: add WSAEventSelect, ioctlsocket

Merge pull request #2368 from awakecoding/master

AutoReconnect Fixes

Merge pull request #2367 from eledoux/ports

fixed: serial redirect not work #2333

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

Merge pull request #2366 from xsumbe00/master

Fixes Solaris bus error on sparc.

xfreerdp: fix egfx multimon on reconnect

libfreerdp-core: add channel reconnect

libfreerdp-core: refactor client info pdu functions

libfreerdp-core: fix auto reconnect cookie security verifier computation

libfreerdp-core: improve client core info debug output

libfreerdp-core: improve reconnection

serial: changed the default underlying ServerSerialDriverId from SerialDriverSerCx2Sys to SerialDriverSerialSys

xfreerdp: partial reconnect fixes

libfreerdp-codec: allow region_uninit to be called multiple times

Fixes Solaris bus error on sparc.

Merge pull request #2364 from awakecoding/gateway

Gateway Improvements and Fixes

wlfreerdp: fix unused variable warning

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

Merge pull request #2355 from xsumbe00/master

Fixes some build issues on Solaris 11.

Merge pull request #2343 from kilobyte/master

Fix build failure on x32.

Merge pull request #2340 from akallabeth/openssl_detection_fix

Openssl detection fix

Merge pull request #2349 from RolKau/2349_dvp

Forward layout ID of Programmer Dvorak to server

Merge pull request #2350 from giox069/master

Fix freerdp_channels_free for clients with multiple active connections

libfreerdp-core: make tsg rpc connection event-driven, fix race condition on connection

Merge pull request #2361 from bmiklautz/set_pointer_fix

xfreerdp: fix problem with SetPosition

libfreerdp-core: add event-driven RTS connection code

libfreerdp-core: start event-driven gateway http code

libfreerdp-core: add more IN/OUT gateway channel states

Better to use HAVE_SYS_FILIO_H when we already have it.

"ar" and zero or several 'm' matches also to "sparc". Adding '^' at the begiinning seems like the safest option.

xfreerdp: fix problem with SetPosition

Since XWarpPointer generates an pointer motion event, as if the mouse
was moved by hand, xfreerdp sends back the received mouse position to
the server. This behavior is wrong since pointer positions set via
server pointer update pdu shouldn't get sent back to the server
(it's not a "real" mouse move).

To fix this problem change the x windows event mask to not handle
pointer motion events before setting the new pointer position and change
it back again afterwards.

Possible downside of this is that some pointer motion events might get
lost but this shouldn't be noticeable.

Thanks to nfedera for the neat idea ;).

winpr-comm: fixed timeouts to return immediately with zeroed timeval structures instead of a NULL pointer.

libfreerdp-core: improve error handling in ts gateway

Merge pull request #2358 from nfedera/fix-2015-02-03-01

core/fastpath: removed flawed slow-path conversion

Merge pull request #2356 from nfedera/fix-2015-02-02-01

core/capabilities: remove misplaced/useless code

pkg/rpm: fix changelog order

pkg/rpm: fix typo in changelog

Merge pull request #2360 from bmiklautz/version_update

Update version to 1.2.1

Merge pull request #2359 from bmiklautz/ringbuffer_debug

ringbuffer: add option to enable/disable debugging

pkgs: update version to 1.2.1

Update version to 1.2.1

This allows code using FreeRDP to detect the presence of the skipCompression field in
surface commands.

ringbuffer: add option to enable/disable debugging

Add option WITH_DEBUG_RINGBUFFER to enable/disable ringbuffer debugging
at compile time.

Even if it is possible to filter specific wlog tags it's not yet
possible to exclude one or more and ringbuffer adds massive debugging
output if enabled and WLOG_LEVEL is set to DEBUG.

core/fastpath: removed flawed slow-path conversion

Commit 0357a38e3129b67093c7c055c7eaa242ce293673 modified the function
fastpath_send_update_pdu() to check if the desired update is possible
by checking the payload size against the computed maxLength and the
clients's advertised max request size.
If the check failed that commit added a workaround which simply
copied the payload to a slow path updade.
This workaround is totally flawed and causes protocol errors:
- the fast path update code is not checked and required data format
  conversions are missing
- depending on the fast path update code rdp_send_data_pdu() would
  have to be called with differend data pdu type values but the
  workaround always uses DATA_PDU_TYPE_UPDATE
- the workaround does not check if the total size would exceed
  the maximum possible size for a slow path update

The check if a fast path output is actually possible with the
passed parameters is basically a good idea.
However, if that check fails it would only indicate an error in
the server implementation who must not generate updates that
exceed the client's max request size.
Even though a slow-path conversion would be possible there is
much more involved than simply copying the payload stream.
In addition it is highly doubtful if there is a benefit at all.
Even the oldest rdesktop and windows ce clients do support fast
path and although some lack the multi-fragment update capability
we cannot really send larger updates using slow-path outputs.

For the reasons elucidated above, I have removed the workaround
but kept a modified version of the check if a fast-path output
is possible at all.

libfreerdp-core: add better state machine transitions to ts gateway

libfreerdp-core: improve http parsing

Added ifdef to previous commit.

core/capabilities: remove misplaced/useless code

Commit 0357a38e3129b67093c7c055c7eaa242ce293673 has added some code
without any effect.
That commit added code to rdp_read_capability_sets() to check if
CAPSET_TYPE_MULTI_FRAGMENT_UPDATE was not received which caused
settings->MultifragMaxRequestSize to be set to 0.

- this was done in the wrong place because we do these kind
  of checks in rdp_recv_confirm_active() by consulting the
  variable settings->ReceivedCapabilities[]
- the code had no effect at all because MultifragMaxRequestSize gets
  set to FASTPATH_FRAGMENT_SAFE_SIZE in rdp_recv_confirm_active()
  if the CAPSET_TYPE_MULTI_FRAGMENT_UPDATE was not received.

Merge pull request #2354 from bmiklautz/ts_pointer_pdu

Handle Server Pointer Update PDU client side

libfreerdp-core: make tsg threadless

Fixes some build issues on Solaris 11.