Fix compilation error under Visual Studio 2010

Visual Studio 2010 use a compiler that supports only C89, which
only supports declaring variable at top of a local scope. Moving
scope variable to the top of function should solve this problem.

Merge pull request #2983 from nitomartinez/master

Fixes #2982. The idea is to be able to create the socket externally a…

Fix formatting: else if in its own line and no brackets for single statements

Disable setting socket options for preexisting socket like: keepalive and tcp_nodelay, these should be set extenally if needed. Do not close the socket if the clientaddress could not be set

Merge pull request #2981 from mfleisz/server_cliprdr_fix

cliprdr/server: Fix parsing of file contents request PDU

Fixes #2982. The idea is to be able to create the socket externally and pass that socket FD to FreeRDP so that it can be used there.

The idea suggested is to use the following interface:

settings->ServerHostname = "|"
settings->ServerPort = SocketFD

cliprdr/server: Fix incorrect message flags

cliprdr/server: Fix incorrect message header

cliprdr/server: Fix parsing of file contents request PDU

Merge pull request #2970 from bmiklautz/prep_2.0

2.0 preparations

Merge pull request #2971 from woutershep/cmake-manual-path

cmake: tweak manpage install location

wlog: cleanup API includes

Only include necessary header files.

wlog/journald: don't set a default identifier

Don't set the identifier to "winpr" as default value because journald
will use the programs name as default if no identifier is set.
This way a program using WLog doesn't need to set an identifier (except
it want something different then it's name).

wlog: change variable naming and fix documentation

* change State to active and make it BOOL since it's only got two
  used values
* fix some typos in the documentation

Merge pull request #2917 from binglinoioij/master

add values-zh/string.xml,that was profiles for chinese

Merge pull request #2930 from dbungert/glyph

Clamp ultra-wide glyph opRight vals to desktop width

Merge pull request #2962 from akallabeth/client_build_fixes

Allow client build to be disabled

cmake: tweak manpage install location

This patch is needed when wanting to install binaries/libraries to other
locations than data. The linux distro Exherbo installs binaries and
libraries to /usr/<chost>/bin, /usr/<chost>/lib respectively but
manpages should still go in /usr/share/man/ because they are
architecture independent, without this patch they go in
/usr/<chost>/share/man unconditionally.

cmake documentation states:
DATAROOTDIR - read-only architecture-independent data root (share)

So this patch makes it use that so that its configurable.

wlog: add some rudimentary documentation

wlog: fix build on windows

wlog: fix android build

Fix compiler warnings

gcc (Debian 4.9.2-10) 4.9.2

winpr/libwinpr/thread/argv.c: In function ‘CommandLineToArgvA’:
winpr/libwinpr/thread/argv.c:94:6: warning: unused variable ‘index’ [-Wunused-variable]
  int index;

winpr/libwinpr/file/test/TestFileGetStdHandle.c: In function ‘TestFileGetStdHandle’:
winpr/libwinpr/file/test/TestFileGetStdHandle.c:44:2: warning: implicit
declaration of function ‘CloseHandle’ [-Wimplicit-function-declaration]
  CloseHandle(stdout);
    ^
libfreerdp/codec/test/TestFreeRDPRegion.c: In function ‘test_norbert2_case’:
libfreerdp/codec/test/TestFreeRDPRegion.c:697:6: warning: unused variable ‘i’ [-Wunused-variable]
  int i;

channels/cliprdr/server/cliprdr_main.c: In function ‘cliprdr_server_receive_format_list’:
channels/cliprdr/server/cliprdr_main.c:636:24: warning: unused variable ‘cliprdr’ [-Wunused-variable]
  CliprdrServerPrivate* cliprdr = (CliprdrServerPrivate*) context->handle;
                        ^
channels/cliprdr/server/cliprdr_main.c: In function ‘cliprdr_server_init’:
channels/cliprdr/server/cliprdr_main.c:1097:24: warning: unused variable ‘cliprdr’ [-Wunused-variable]
  CliprdrServerPrivate* cliprdr = (CliprdrServerPrivate*) context->handle;

Adapt error level for debug messages

When using xfreerdp a regular invocation should really only report grave
problems as error. "Regular" messages should be either info or even
debug.

wlog: rework, cleanup and stabilize API

* only expose necessary functions and types in header
* don't expose appender internals
* add generic function WLog_ConfigureAppender to have the possibility
  to configure appender specific settings
* detect appender availability if WLog_SetLogAppenderType or
  WLog_Appender_New return FALSE or NULL respectively the appender isn't
  available or the initialization failed. This is very useful for the
  use with optional appenders.
* add Free to the appender interface. At the time of the Free the
  appender is known and available so it can be called directly (instead
  of calling the right function according to the type)
* make all appender internal function static
* all appenders return the generic wLogAppender type now. Typecasts
  are internally done where necessary this abstracts the appenders more
  cleanly

Add build-config.h

build-config.h should contain configure/compile time settings that are
relevant for projects that use FreeRDP.

For example the compiled in plugin search paths.

Pump version to 2.0.0

Merge pull request #2964 from hardening/udp_appender

Add an UDP appender to wLog

Take in account @nfedera's remarks

Merge pull request #2954 from mfleisz/cliprdr_fix

cliprdr/server: Server-side file content receiving used wrong callback

Merge pull request #2926 from bjcollins/rail_fixes

Rail fixes

Merge pull request #2963 from nfedera/fix-fullscreen-multimon

client/X11: multimon/fullscreen fixes

Merge pull request #2968 from nfedera/fix-commandlinetoargv

winpr/argv: fix CommandLineToArgvA memory access

winpr/argv: fix CommandLineToArgvA memory access

- fixed access of unitialized memory beyond terminating null
- minor simplifications

Add an UDP appender to wLog

This appender allows to receive the logs over a network connection using UDP packets.
You can see the logs using a listening netcat, for example: nc -ul 127.0.0.1 20000.

client/X11: fix post fullscreen repositioning

client/X11: multimon/fullscreen fixes

- fixed wrong calculation of xfc->fullscreenMonitors.[right|bottom]
- only use _NET_WM_FULLSCREEN_MONITORS if at least 2 monitors are involved
- call XMoveWindow before setting the _NET_WM_STATE_FULLSCREEN property

Fixed WITH_CLIENT_CHANNELS dependencies.

Added WITH_CLIENT_COMMON option.

Disabled client builds if WITH_CLIENT is not set.

Fixed return values.

Merge pull request #2961 from nfedera/nf-fix-image-copy-segfault

codec/color: fix segfault in freerdp_image32_copy

codec/color: fix segfault in freerdp_image32_copy

Code path for dstBytesPerPixel == 3 moved src and dst pointers
beyond their respective buffers.

Merge pull request #2951 from bmiklautz/friday

Misc fixes

Merge pull request #2957 from hardening/journald

Adds a systemd journal appender

Take in account @bmiklautz remarks

* I have added a function to set the journal identifier
* the appender name has been changed from SYSTEMD to JOURNALD

Fix the env var leak

Use unbuffered output

Adds a systemd journal appender

Merge pull request #2940 from hardening/syslog_appender

Add a wLog syslog appender

Fixed a warning

Support syslog type for the WLOG_APPENDER env var

cliprdr/server: Server-side file content receiving used wrong callback

Don't include config.h in public headers

In fact we don't need the ifdefery in public headers, so don't include unneeded
config.h file.

Merge pull request #2952 from mfleisz/server_cliprdr_config

cliprdr/server: Provide the server with a possibility to configure capabilities

cliprdr/server: Add comment

Include config.h file when needed

cliprdr/server: Provide the server with a possibility to configure capabilities

winpr API: add EnvironmentBlockToEnvpA

* expose EnvironmentBlockToEnvpA
* cleanup includes in process.c
* removed unused "flag" variable in _CreateProcessExA
* make ProcessHandleCloseHandle static

winpr/stream: don't allow 0-size streams

winpr: fix compiler warning

winpr/libwinpr/utils/wlog/wlog.c: In function ‘WLog_PrintMessageVA’:
winpr/libwinpr/utils/wlog/wlog.c:234:7: warning: ‘status’ may be
used uninitialized in this function [-Wmaybe-uninitialized]
BOOL status;
      ^

gcc 4.9.2

Merge pull request #2945 from mfleisz/wlog_compile_fix

wlog: Fix variable definitions

wlog: Fix variable definitions

Merge pull request #2929 from mfleisz/cliprdr_fix

cliprdr/server: Don't call CloseHandle on ChannelEvent

cliprdr/server: Fix memory leak

Adapt to last wLog changes

Add a wLog syslog appender

Merge pull request #2927 from MartinHaimberger/mh-log-returnvalues

wlog: fixed return values

Merge pull request #2939 from nfedera/fix-x11-clip-id

xfreerdp/clipr: fix self owned test and hardening

wlog: fixed return values

wlog used to return an int but the only meaning
of the return value was:

 * negative ... error
 * 0 or positive ... success

but the positve returned value was 1 or some id of some
subsystem, nothing meaningful for the caller.

For a more meaningful returnvalue we now use BOOL.

If something goes wrong FALSE is returned.

Merge pull request #2938 from LookBehind/master

Fix rdp_recv_logon_error_info

xfreerdp/clipr: fix self owned test and hardening

- xf_cliprdr_is_self_owned() lied if multiple xfreerdp instances were
  running.
- fixed a few unchecked callocs
- added/modified and handled some return values in compliance with
  the new hardened channel api

Fix rdp_recv_logon_error_info

As https://msdn.microsoft.com/en-us/library/cc240641.aspx says - first is going errorNotificationType then errorNotificationData.

Merge pull request #2933 from realjiangms/fix_gdi_bitmap_update

GDI: Fix gdi_bitmap_update to check dest buffer size.

GDI: Fix gdi_bitmap_update to check dest buffer size.

cliprdr/server: Don't call CloseHandle on EventHandle

Merge pull request #2910 from dupondje/master

FindGStreamer_1_0: fix build failure for new gstreamer versions

update values-zh/strings.xml

Cleanup unnecessary/unintended changes from last commit

RemoteApp Support

1. Remove all uses of "localWindowOffsetCorr" variables, they added an extra layer of complexity and they are not actually needed to handle coordination of window position/size between
the local coordinate system and the remote one. This logic was causing issues in the case where the window was moved off the left side of the screen.

2. Update the xf_setWindowVisibilityRects function to offset the visibility rects as necessary when the window is hanging off the left side of the screen.

3. Stop sending mouse events when doing keyboard moves/sizes(as desired), and stop sending two mouse events for non-keyboard moves/sizes

4. Move location of new UTF8_STRING variable from previous commit

5. Refresh window and window shape for any window position/size updates, this helps keep the local and server windows in sync and works around some race conditions

Remote app support

All window state order offsets are signed according to the RDP spec, lets treat them as such.

Store Miscellaneous X11 Atoms into variables to match how we handle other X11 Atoms.

RemoteApp support

Remove use of WindowRects to affect window shape, the VisibilityRects are used for this purpose.

Ensure that app windows always get a window type assigned to them.

Continue processing all other window orders for new remote app windows instead
of existing after only looking at part of the information. For instance, window visibility
rects are part of the message with the new order and were being ignored.

Do not support fullscreen toggle keyboard sequence in remote_app mode. There is no support in interface for this
and the fullscreen code is not designed to handle remote app windows.

Fix xf_rail_paint fencepost error

The regions used to store and calculate the invalidRegion are exclusive
of the bottom and right edges, not inclusive.

Fixes "mouse droppings" in mspaint.exe when moving the mouse leftwards
across the canvas.

Clamp ultra-wide glyph opRight vals to desktop width

Merge pull request #2924 from awakecoding/master

egfx multimon fixes

libfreerdp-gdi: fix egfx multimon support

xfreerdp: fix multiple egfx context support in X11 GFX

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

Merge pull request #2919 from realjiangms/fix_allow_empty_password

Sec/NLA: Support passwordless (blank password) login with NLA.

Merge pull request #2920 from awakecoding/mbedtls

WinPR mbed TLS support

libwinpr-crypto: fix OpenSSL 1.0 HMAC API change

libwinpr-crypto: fix error checking

libwinpr-crypto: avoid linking to less common digests and ciphers

Merge pull request #2912 from realjiangms/shadow_fix_auth_nopasswd

Server/Shadow: Minor fix on authentication in shadow_client.c.

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

winpr: make clean non-OpenSSL build possible (without schannel, makecert)

libwinpr-crypto: add generic digest API and OpenSSL compatible key derivation