an argument check in xf_UpdateWindowArea

Remove redundant condition

[channels/urbdrc/client/data_transfer.c:2310] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.
[channels/urbdrc/client/data_transfer.c:2311] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.
[channels/urbdrc/client/data_transfer.c:2312] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.
[channels/urbdrc/client/data_transfer.c:2313] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.
[channels/urbdrc/client/data_transfer.c:2314] -> [channels/urbdrc/client/data_transfer.c:2322]: (warning) Either the condition 'if(transfer_data)' is redundant or there is possible null pointer dereference: transfer_data.

Resolve possible null pointer dereference

[channels/urbdrc/client/isoch_queue.c:124] -> [channels/urbdrc/client/isoch_queue.c:126]: (warning) Either the condition 'if(isoch)' is redundant or there is possible null pointer dereference: isoch.

Remove redundant condition

[client/X11/xf_rail.c:205] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:206] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:207] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:208] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:215] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:216] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:217] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.
[client/X11/xf_rail.c:218] -> [client/X11/xf_rail.c:220]: (warning) Either the condition 'if(appWindow)' is redundant or there is possible null pointer dereference: appWindow.

Merge pull request #3729 from chipitsine/master

remove redundant assignment

remove redundant assignment

[client/Windows/wf_cliprdr.c:575]: (warning) Redundant assignment of 'clipboard' to itself.

Merge pull request #3728 from chipitsine/master

removed redundant condition

removed redundant condition:

[winpr/libwinpr/synch/semaphore.c:131] -> [winpr/libwinpr/synch/semaphore.c:136]: (warning) Either the condition 'if(semaphore)' is redundant or there is possible null pointer dereference: semaphore.
[winpr/libwinpr/synch/semaphore.c:132] -> [winpr/libwinpr/synch/semaphore.c:136]: (warning) Either the condition 'if(semaphore)' is redundant or there is possible null pointer dereference: semaphore.
[winpr/libwinpr/synch/semaphore.c:133] -> [winpr/libwinpr/synch/semaphore.c:136]: (warning) Either the condition 'if(semaphore)' is redundant or there is possible null pointer dereference: semaphore.
[winpr/libwinpr/synch/semaphore.c:134] -> [winpr/libwinpr/synch/semaphore.c:136]: (warning) Either the condition 'if(semaphore)' is redundant or there is possible null pointer dereference: semaphore.

Merge pull request #3725 from chipitsine/master

make cppcheck even more happier

Merge pull request #3726 from hardening/add_const

Added missing const modifiers for source pointers in codecs

Added missing const modifiers for source pointers in codecs

Merge pull request #3715 from akallabeth/reproducible_build

Reproducible build

Merge pull request #3722 from akallabeth/ccache_support

Added CCACHE detection and option to disable.

make cppcheck even more happier:

[channels/tsmf/client/gstreamer/tsmf_X11.c:317] -> [channels/tsmf/client/gstreamer/tsmf_X11.c:322]: (warning) Either the condition '!decoder' is redundant or there is possible null pointer dereference: decoder.
[channels/tsmf/client/gstreamer/tsmf_X11.c:470] -> [channels/tsmf/client/gstreamer/tsmf_X11.c:475]: (warning) Either the condition '!decoder' is redundant or there is possible null pointer dereference: decoder.
[channels/tsmf/client/gstreamer/tsmf_X11.c:472] -> [channels/tsmf/client/gstreamer/tsmf_X11.c:475]: (warning) Either the condition '!decoder' is redundant or there is possible null pointer dereference: decoder.
[channels/tsmf/client/tsmf_media.c:179] -> [channels/tsmf/client/tsmf_media.c:181]: (warning) Either the condition '!stream' is redundant or there is possible null pointer dereference: stream.
[client/Windows/wf_cliprdr.c:2219] -> [client/Windows/wf_cliprdr.c:2222]: (warning) Either the condition '!formatDataResponse' is redundant or there is possible null pointer dereference: formatDataResponse
[client/Windows/wf_cliprdr.c:2445] -> [client/Windows/wf_cliprdr.c:2448]: (warning) Either the condition '!fileContentsResponse' is redundant or there is possible null pointer dereference: fileContentsResponse.
[client/X11/xf_cliprdr.c:911] -> [client/X11/xf_cliprdr.c:913]: (warning) Either the condition '!clipboard' is redundant or there is possible null pointer dereference: clipboard.
[client/X11/xf_graphics.c:504] -> [client/X11/xf_graphics.c:506]: (warning) Either the condition '!xfc' is redundant or there is possible null pointer dereference: xfc.
[libfreerdp/core/transport.c:861] -> [libfreerdp/core/transport.c:863]: (warning) Either the condition '!transport' is redundant or there is possible null pointer dereference: transport.
[server/shadow/shadow_server.c:777] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:778] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:779] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:781] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:782] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:783] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:784] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:785] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:787] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.
[server/shadow/shadow_server.c:789] -> [server/shadow/shadow_server.c:791]: (warning) Either the condition '!server' is redundant or there is possible null pointer dereference: server.

Added CCACHE support for android OpenH264 build.

Merge pull request #3724 from chipitsine/master

make cppcheck a bit happier

Added CCACHE support for OpenSSL builds.

make cppcheck a bit happier:

[channels/printer/client/printer_cups.c:103]: (error) Resource leak: fp
[server/Mac/mf_event.c:195]: (error) Memory leak: event_queue
[server/shadow/shadow_capture.c:233]: (error) Memory leak: capture
[winpr/libwinpr/sspi/test/TestSchannel.c:440]: (error) Memory leak: lpTokenIn
[winpr/libwinpr/thread/argv.c:198]: (error) Memory leak: lpEscapedChars
[winpr/libwinpr/utils/sam.c:312]: (error) Memory leak: entry

Merge pull request #3723 from chipitsine/master

trivial issue found by cppcheck:

trivial issue found by cppcheck:

[uwac/libuwac/uwac-display.c:73]: (error) va_list 'args' was opened but not closed by va_end().

Merge pull request #3721 from akallabeth/x11_color_fix

X11 color conversion fix

Added CCACHE detection and option to disable.

Fixed conversion to XColor.

Fixed stride for XCreateImage.

Use CMake TIMESTAMP, remove newline from TODAY

Merge pull request #3718 from bmiklautz/fix/nightly-pkgs

Fix nightly packages

Fix nightly packages

Man pages weren't installed or wrong path was used.

Merge pull request #3708 from akallabeth/termination_fixes

Client channel shutdown fixes

Fixed channel connected status.

Merge pull request #3664 from realjiangms/fix_gfx_ack

Server/shadow: Fix handling for gfx acknowledge according to spec

Merge pull request #3716 from akallabeth/planar_warning_fix

Fixed uninitialized variable warning.

Fixed uninitialized variable warning.

Merge pull request #3710 from akallabeth/local_16bpp_fixes

Local 16bpp fixes

Fixed alpha channel for color formats without.

Fixed staging surface scanline.

Fixed color format selection for 16bpp

Fixed planar with if local framebuffer is 16bpp

Fixed scanline for RFX.

Merge pull request #3703 from akallabeth/prim_optimize

[PERFORMANCE] Inlined heavily used functions

Merge pull request #3711 from hardening/monitor_layout

Misc fixes and improvements

make build reproducible

by allowing build systems to use a constant date instead of the current one
see https://reproducible-builds.org/ for why this is good
and https://reproducible-builds.org/specs/source-date-epoch/
for the definition of this variable

This call is designed to work with GNU date. If BSD support is a concern, it can be added.

Add a callback that allows to adjust monitors layout

This callback is useful in server mode when you want to adjust the monitor
layout just before the server monitor layout PDU is built and sent.

Treat the result of the Capabilities callback

The result was ignored. Plus return the correct value in case of licence error (-1
instead of FALSE).

Merge pull request #3705 from akallabeth/android_openssl_1_1

Android openssl 1 1

Disabled OpenH264 in default android build.

Progressive structs to private header for tests.

Progressive struct now opaque.

Fixed missing return.

Progressive conversion and speed fixes

* Using destination color format and precomputed tile stride
* Using preallocated logger

Removed obsolete primitive usage.

Optimized functions, removed duplicates.

Inlined shift operations.

Inlined freerdp_image_copy helpers.

Inlined interleaved helper functions.

Optimized freerdp_image_fill

Optimized general_YUV444ToRGB_8u_P3AC4R for BGRX

Inlined heavy used functions.

Optimized general_RGBToRGB_16s8u_P3AC4R for BGRX

Optimized general_yCbCrToRGB_16s8u_P3AC4R for BGRX

Fixed broken variable declaration.

Merge pull request #3702 from akallabeth/clear_reset_fix

ClearCodec fix #3642

Merge pull request #3707 from akallabeth/AMV007-patch-2

Rebased #2541

Fixed warnings due to API changes.

Merge pull request #3704 from bmiklautz/versioning

Library naming and man pages

Return OK in case of channel not connected.

Checks, return value fixes

* Added missing argument checks
* Use opened drdynvc->log where possible
* Fix drdynvc_send return in case channel not connected.

Do not terminate channel disconnect if one fails.

Decreased log verbosity.

Fixed audio termination code.

Fix typos in man pages.

Fixed SONAME

Merge branch 'patch-2' of https://github.com/AMV007/FreeRDP into AMV007-patch-2

Conflicts:
channels/tsmf/client/tsmf_ifman.c
channels/tsmf/client/tsmf_media.c

Merge pull request #3669 from xhaakon/master

Limit MultifragMaxRequestSize in server mode

Updated library names after change in #3704

Updated gradle settings.

Renamed OpenSSL library

To avoid collisions with system provided OpenSSL use
a different (unique) library name for android builds.

Added android release build configurations.

Updated android build scripts for OpenSSL 1.1

Install man pages

* man pages are only build/installed if WITH_MANPAGES is enabled
* create a new cmake function install_freerdp_man to unified install man
  pages
* install all man pages using the new function
* update the nightly packages accordingly

wlfreerdp: add initial man page

shadow-cli: add initial man page

shadow server: add more error messages

cmdline: fix wlog man page reference

It's wlog(7) now.

winpr-makecert: add initial man page

makecert: fix a regression with > 2048 bit

When certificates with more than 2048 bits were generated and written to
file the read function used a erroneous calculated length causing the
used buffer to overflow.

winpr-makecert: use sha256 and update command line

* use sha256 instead of sha1 as default hash algorithm
* fix command line parser
* mark not implemented command line switches as unsupported

winpr-hash: cleanup cmd line and add man page

* add a initial man page
* clean up the command line a little bit (basically error messages)

utils: remove unnecessary debug message.

Change the level from info to debug as it's nothing a regular users
would have interest in.

Include major version number in library names

Currently it is not possible to cleanly install multiple major version
of FreeRDP concurrently as some of the development libraries (.so files)
files can conflict.

This change renames all libraries to include the major version number in
the library name to fix this limitation.

The list of changed libraries:

libwinpr-tools.so -> libwinpr-tools2.so
libwinpr.so -> libwinpr2.so
libfreerdp.so -> libfreerdp2.so
libfreerdp-client.so -> libfreerdp-client2.so
libfreerdp-shadow.so -> libfreerdp-shadow2.so
libfreerdp-server.so ->  libfreerdp-server2.so
libfreerdp-shadow-subsystem.so -> libfreerdp-shadow-subsystem2.so
libuwac.so -> libuwac0.so

As the library names have changed, projects that use FreeRDP will need to
update their dependencies. -
If pkg-config or cmake find modules are used, reconfiguration might be
sufficient.

Fixes #3460

Merge pull request #3701 from akallabeth/progressive_inlined

[performance] Progressive inlined

Do not reset V-Bar storage.

Merge pull request #3665 from realjiangms/fix_ssl_add_all_digests

Winpr/openssl: Fix digests initialization in multi-thread

Merge pull request #3694 from akallabeth/avc420_black_artifact_fix

Fixed solid fill command.

Merge pull request #3688 from akallabeth/clear_codec_buffer_resize_fix

Clear codec buffer resize fix

Merge pull request #3696 from akallabeth/gdi_stride_init

Initializing gdi.stride on resize.

Inlined progressive helper functions.

Unified clipping of byte values.

Merge pull request #3698 from rjcorrig/kb3203884

winpr: Updates time zone data to December 2016

Update timezone data to Dec 2016

Allow custom MultifragMaxRequestSize in server mode

FreeRDP calculates default request size in server mode based upon
the desktop resolution. In practice, however, I've found clients (namely
MS Lync 2013) which would show only a black screen when the value is
greater than 0x3EFFFF.

This change allows to override the default in such special cases by
assigning a different value to server->settings->MultifragMaxRequestSize
after the shadow server structure is initialized in shadow_server_new().