Removed STANDARD_RIGHTS_WRITE from writeable check.

Fixed integer size warnings.

Additional error checks for drive channel.

Merge pull request #4055 from akallabeth/talos_v3

Security fixes and code cleanups

clipboard: fix possible invalid memory access

Fix an possible issue found by Sébastien Duquette.

Fixed missing include.

Fixed resource cleanup.

Fixed uninitialized values.

Fixed double free.

Fixed wrong type for AuthenticationLevel.

Replaced type BOOL with INT32 as according to MSDN they are typedefed.
This keeps the ABI compatible and fixes the wrong data type.

Fixed use of reserved keywords for include guards.

Fixed clang malloc integer overflow warnings.

Fix TALOS issues

Fix the following issues identified by the CISCO TALOS project:
 * TALOS-2017-0336 CVE-2017-2834
 * TALOS-2017-0337 CVE-2017-2834
 * TALOS-2017-0338 CVE-2017-2836
 * TALOS-2017-0339 CVE-2017-2837
 * TALOS-2017-0340 CVE-2017-2838
 * TALOS-2017-0341 CVE-2017-2839

Merge pull request #3956 from dodo040/smartcard

redirect specific smartcard readers

fix format code

fix smartcard argument parsing

fix wrong declaration

redirect specific smartcard readers

Merge pull request #4044 from akallabeth/disable_h264_if_not_compiled

Disable h264 if no backend compiled in

Fixed variable assignment.

Merge pull request #4047 from akallabeth/win_gfx_fix

Always use software GFX for windows client.

Added warning on GFX channel load that it does not support acceleration.

Don't use an option for WITH_GFX_H264

CMAKE_DEPENDENT_OPTION the option is initialized on the first run and
then saved. Later configuration changes are not considered anymore.
Therefor use an variable instead of an option here.

Always use software GFX for windows client.

Reordered backends with regard to feature completeness.

Merge pull request #4041 from wayk/PathMakePathA

Fixed PathMakePathA (returned true even if it can't create the last f…

Readded H264 struct pointers, keep API compatible.

Add 10.3 capability check.

Fixed config option.

Merge pull request #4043 from rjcorrig/kb4020322

winpr: Updates time zone data to July 2017

Deactivated all H264 related code paths.

Disable WITH_GFX_H264 if no backend compiled in.

Removed H264 dummy, did not work (crash on use)

Disable GFX H264 if no backend compiled in.

Update time zone data to July 2017

Change documented source of windowsZones.xml file

Fixed PathMakePathA (returned true even if it can't create the last folder of the path)

Merge pull request #4039 from nfedera/fix-misc-compiler-warnings

Fixed a few compiler warnings

Fixed a few compiler warnings

Merge pull request #4038 from nfedera/fix-sample-server-pixmap-format

Fixed sample server's wrong rfx pixel format

Fixed sample server's wrong rfx pixel format

Merge pull request #4021 from scroogie/master

Fix compilation on RHEL 6.x by macro-guarding two functions

Merge pull request #4029 from realjiangms/fix_shadow_fragment_size

server/shadow: Fix incorrect bitmap fragment update.

Merge pull request #4033 from blino/wayland-fullscreen

Add fullscreen support in Wayland clients

Merge pull request #3992 from johnjmullen/fix-barcode-scanner

  Fix issue #3229 Barcode Scanner USB - missing double characters

Merge pull request #4032 from blino/clipboard-build

Fix clipboard POSIX build because of basename conflict

Fix clipboard POSIX build because of basename conflict

FreeRDP/winpr/libwinpr/clipboard/posix.c:397:20: error: conflicting types for ‘basename’
 static const char* basename(const char* name)
                    ^
In file included from FreeRDP/winpr/include/winpr/collections.h:25:0,
                 from FreeRDP/winpr/libwinpr/clipboard/posix.c:37:
/usr/include/string.h:599:14: note: previous declaration of ‘basename’ was here
 extern char *basename (const char *__filename) __THROW __nonnull ((1));

Support fullscreen setting in wayland client

Implement fullscreen for wayland shell surface

server/shadow: Fix incorrect bitmap fragment update.
Legacy bitmap update might fail with 'fast path update size (xxxxx) exceeds the client's maximum request size (xxxxx)'
Original code might update last fragment with exceeded fragment size incorrectly. Fix the logic to prevent it.

Merge pull request #4024 from bmiklautz/fix_4022

Fix a regression introduced with PR #4013

Fix a regression introduced with PR #4013

If numWindowRects/numVisibilityRects is zero a realloc might either
return NULL or a free able memory. In the first case the introduced
regression caused a double free.
As 0 is a possible value that can be received in both cases rail was
broken.

Fixes #4022

Merge pull request #4008 from hardening/h264_split

Split h264 subsystems in different files

Fix compilation on RHEL 6.x by macro-guarding two functions that were
added in gstreamer 0.10.31 (RHEL 6.x being on 0.10.29).

Merge pull request #4013 from weizhenwei/master

fix FreeRDP some memory leak problem

code format adjustment

remove redundant NULL pointer check

remove duplicate code when freeing drive

remove useless NULL pointer check before free

add NULL pointer check and set freed pointers to NULL afterward

refactor to remove duplicate code and replace free+malloc with realloc

refactor on redundant code copy

fix compile error on win64.vs2010 platform

fix memroy leak of window_icon->iconInfo at update_read_window_icon_order()

fix memory leak of pKeys at xf_rail_invalidate_region()

fix memroy leak of fd at FindFirstFileW()

fix memory leak at update->window->window_state.windowRects/visibilityRects at update_read_window_state_order()

fix memory leak at update->window->window_state.titleInfo.string at update_read_window_state_order()

fix memory leak on rail_virtual_channel_event_data_received() -> StreamNew()

fix memory leak at drive_register_drive_path() and drive_free()

fix memory leak at rail_client_execute()

fix memroy leak on rail_recv_exec_result_order() execResult

code clean on free(update->window->monitored_desktop.windowIds);

fix memory leak on update->window->monitored_desktop.windowIds which is realloced at update_read_desktop_actively_monitored_order()

fix a memroy leak on rdpBitmap free

Fix openh264 backend compilation

Fix variable declaration and win32 compilation

Split h264 subsystems in different files

Merge pull request #4003 from akallabeth/macos_mouse_wheel_shadow

Fixed shadow server mouse wheel mapping on MacOS

Fixed shadow server mouse wheel mapping on MacOS

Merge pull request #3989 from akallabeth/avc444_alignment_fix

Align width and height for AVC444 decoding to 16.

Merge pull request #3996 from akallabeth/drive_size_fix

Fix #3995: File size information

Fix #3995: File size information

Merge pull request #3905 from ilammy/x11-cliprdr/file-clipping

Local-to-remote file clipping for xfreerdp

Fix issue #3229 Barcode Scanner USB - missing double characters

With some usb barcode scanners, repeated characters do not appear in the freerdp session.
It looks like this is because the KeyRelease signal is not sent for the first character.
Removing this if check fixes the problem.

channels/cliprdr: avoid possible integer overflow

If the server sends us garbage (or the client provides it) then it is
possible for the multiplication to overflow (as it is performed on
unsigned 32-bit values) which will result in a false positive failure of
the sanity check. Avoid it by rearranging arithmetics a little.

Keep the multiplication in the error message because we are interested
in the number of bytes in the stream and how it compares to the number
we have expected based on the presumed file count.

Align width and height for AVC444 decoding to 16.

Merge pull request #3987 from chipitsine/cppcheck

resolve an issue found by cppcheck

resolve an issue found by cppcheck:

[libfreerdp/crypto/certificate.c:315] -> [libfreerdp/crypto/certificate.c:316]: (warning) Either the condition 'if(fingerprint&&fprint)' is redundant or there is possible null pointer dereference: fingerprint.

Merge pull request #3969 from chipitsine/cppcheck

resolve issue found by cppcheck

Merge pull request #3986 from akallabeth/debug_fix

Fixed missing external declaration (#3982)

Fixed missing external declaration (#3982)

Merge pull request #3985 from ilammy/x11-cliprdr/fix-html

Do not fixup HTML data length

resolve an issue found by cppcheck:

[channels/tsmf/client/tsmf_main.c:89] -> [channels/tsmf/client/tsmf_main.c:95]: (warning) Either the condition '!callback' is redundant or there is possible null pointer dereference: callback.

client/X11: do not fixup HTML data length

The data provided by local applications can be actually encoded in
UTF-16 (e.g., Firefox does this to HTML). UTF-16 allows embedded null
bytes so we should not use strlen() to fix up the data. The HTML format
synthesizer can handle trailing null bytes just fine and can detect
whether it deals with UTF-8 or UTF-16.

Merge pull request #3980 from uplusplus/patch-5

fixed issue:#3974

fixed issue:#3974

ConvertToUnicode failed

winpr/file: add missing NULL check

ValidFileNameComponent() has been missing a NULL check for its argument.
It's pretty obvious that NULL is not a valid file name component.

wClipboard: drop WITH_DEBUG_WCLIPBOARD option

This preprocessor definition has been initially intended to disable some
computationally expensive logging, however it turned out that there is
not much computation involved in the resulting implementation of new
wClipboard subsystems. Therefore we do not actually need the compilation
option, the logs can be filtered by "com.winpr.wclipboard.*" tag at
runtime if necessary. So drop the WITH_DEBUG_WCLIPBOARD CMake option and
convert all detailed logs to use WLOG_TRACE level via WLog_VRB macro.

x11/cliprdr: make callbacks static

Most of the functions is this file are internal-use callbacks so they do
not need to be exported from the compilation unit. Mark functions static
as appropriate.