Handle clipboard peek with missing data

The peer expects a response, so we should also be able to respond that
there is no clipboard data currently available.

Improve clipboard debug logging in viewer

Handle unsolicited clipboard transfers

The extended clipboard protocol has the ability for the peer to request
things to be sent automatically, without a request message. Make sure we
honor such settings.

Log client/server clipboard capabilities

Add packages for CentOS 8

Initial support for ubuntu focal

Map macOS Eisu key to XK_Eisu_toggle

It's a better match for what the key symbolises, even though it is not
as common on Unix systems. But we want to avoid getting in to tailoring
things for any one specific input method implementation.

Merge branches 'japanese_layout' and 'win_japanese_keyboard' of https://github.com/gsittyz/tigervnc

Fix bad library references for some static deps

Add comment why unistring is linked dynamically

Remove bogus strncpy() calls

We don't know the size of the output buffer here and giving the input
string size is just confusing and upsets the compiler in some cases.

Merge branch 'master' of https://github.com/mriphysicist/tigervnc

Change to user's home directory before starting session

This is what display managers do, so it is expected by many
environments.

Fix incorrect daemonization by vncsession

We terminated the child instead of the parent after fork().

Reported by Jan Grulich from Red Hat.

Update more references to the old vncserver script

Fake key release events for some Japanese keys

Windows either completely omits WM_KEYUP for these, or only sends it
just before the next WM_KEYDOWN for the key.

Handle Korean keys on Windows

Add some more Japanese keys

This covers some Alt+key presses as well.

Only handle Japanese keys when Japanese layout

Some of the virtual key codes for Japanese keys are used for other
things on other layouts, so we need to make sure we only use this look
up table when the layout is indeed set to Japanese.

Don't show error dialog if exiting without message

This was already handled for most exit conditions, but not the early
ones.

Properly report connection errors to exit_vncviewer()

Update keycode maps to latest version

Mainly fixes for Japanese and Korean keyboard layouts.

Windows JIS Keyboard Support

Added vkey mappings for Japanese keyboards because the special keys for Japanese input do not work on TigerVNC currently.

Regenerate icons

Be explicit about SVG size when building icons

Not sure how this has changed, but ImageMagick is now using the first
-size to also determine how to scale the SVGs, which is not what we
want.

Generate icons with a depth of 8 bits

Anything else is excessive.

High-resolution app icon for MacOS

Merge branch 'master' of https://github.com/johnmartin-oracle/tigervnc

OS X Japanese Keyboard Support

Added OS X JIS Keyboard keys (kVK_JIS_Eisu, kvK_JIS_Kana). The Kana and Eisu keys are used to alter IME behavior, but currently the Kana key produces an unwanted space character and the Eisu key does not work.

Document that vncserver is now a service

Merge branch 'unicode' of https://github.com/CendioOssman/tigervnc

Correctly clear out output UTF-16 buffer

Each character is more than one byte, so adjust the clearing of the
buffer to reflect that.

Correctly handle invalid UTF-16 code points

Some code points are reserved for the UTF-16 coding itself and must not
appear as input data to the algorithm.

Fix conversion of latin-1 to UTF-8

Signed bug prevented anything not ASCII from being coded correctly.

Fix UTF-16 encoding/decoding of high code points

Everything outside of BMP was handled incorrectly and was coded as
completely different code points.

Return the correct number of consumed UTF-8 bytes

This would mess up most conversions from UTF-8 as the caller wouldn't
know how far to step to get to the next valid character, resulting in
markers for invalid data to be injected here and there.

Also add some unit tests to avoid this reoccurring.

Fix installation of vncsession.man

It is now dynamically generated, so it will be in the build directory
and not in the source directory.

Merge branch 'systemd' of https://github.com/grulja/tigervnc

Merge branch 'sysconfdir' of https://github.com/metalefty/tigervnc

Use /run instead of /var/run which is just a symlink

Replace some more hard-coded /etc with sysconfdir

Tolerate specifying -BoolParam 0 and similar

This is needed by vncserver which doesn't know which parameters are
boolean, and it cannot use the -Param=Value form as that isn't tolerated
by the Xorg code.

Do not assume sysconfdir is always /etc

Refer @CMAKE_INSTALL_FULL_SYSCONF@ instead.

Merge branch 'tlstofu' of https://github.com/CendioOssman/tigervnc

Properly store certificate exceptions in Java viewer

Like the native viewer, the Java viewer didn't store certificate
exceptions properly. Whilst not as bad as the native viewer, it still
failed to check that a stored certificate wouldn't be maliciously used
for another server. In practice this can in most cases be used to
impersonate another server.

Handle this like the native viewer by storing exceptions for a specific
hostname/certificate combination.

Remove incorrect umask change

Not sure why this got in there, but it is incorrect as it results in
creating files with too liberal permissions.

[java] don't show password field for secTypes that don't use it

Merge pull request #1 from johnmartin-oracle/johnmartin-oracle-patch-1

Update Surface_X11.cxx

Update Surface_X11.cxx

Runtime sellection of ARGB  XImage byte order

Fix size of 16x16 icon

For some reason it got generated at an incorrect 20x20 size last time.

Hide base SConnection methods from VNCServerST

It should be using the safe wrappers for everything so make sure it
cannot bypass those and call the SConnection methods directly.

Call correct wrappers for clipboard handling

We incorrectly called the underlying functions instead of the safe
wrappers for the new clipboard functions. This had the effect of a)
crashing the entire server if one of these functions failed, and b) not
respecting the settings disabling the clipboard.

Change development version to 1.11.80

Update translation template file

Merge branch 'master-improve-ComparingUpdateTracker' of https://github.com/MarkMielke/tigervnc

Update build system so it will compile on Windows 10.

With this commit, TigerVNC will compile for Windows using MSYS2, MinGW-w64 and Inno Setup 6.  The resulting binaries have some dll dependencies (even with BUILD_STATIC).  The required dll dependencies are not included in the installer.

Unfortunately, the latest version of MSYS2 and MinGW-w64 do not produce a working executable for Windows 7.

Comment on SELinux rule affect other commands

This line affects every command run by the user, unlike everything else
in our policy which is just for vncserver/vncsession. It's easy to miss
this so add a comment pointing it out.

Merge branch 'vnc_home' of https://github.com/wrabcak/tigervnc

Merge branch 'master-fix-vncserver-display-selector' of https://github.com/MarkMielke/tigervnc

Free memory from getaddrinfo()

We handled this in the failure scenario, but not in the vastly more
common successful case.

Merge branch 'winvnc' of https://github.com/CendioOssman/tigervnc

Fix capitalisation of LICENCE.TXT

If InnoSetup is run on a case sensitive system then the configuration
files must match what the file is actually called.

Fixes for Java 11. SocketDescriptor change was a workaround for a previous java problem that no longer seems necessary

[SELinux] Allow vnc session create ~/.vnc

Allow vnc session labeled with SELinux vnc_session_t domain to create
directory in user homedir ~/.vnc labeled with SELinux label xdm_home_t.

Because also process vncpasswd creates ~/.vnc directory, file transition that
userdomain attribute SELinux domain (label for the process) can create the
directory with the same label needs to be added.

userdomain attribute contains following SELinux types:
    auditadm_t
    dbadm_t
    guest_t
    logadm_t
    secadm_t
    staff_t
    staff_wine_t
    sysadm_t
    unconfined_t
    user_t
    user_wine_t
    webadm_t
    xguest_t

Signed-off-by: Lukas Vrabec <lvrabec@redhat.com>

Merge pull request #1069 from MarkMielke/master-tigervnc-rpm-spec-updates

EL 7 tigervnc.spec updates

vncserver: Display check should confirm UNIX domain socket is still valid

If the X server is not shut down cleanly, it can leave UNIX domain
sockets around that "vncserver" has previously mis-identified as
evidence that the display number is still in use.

Instead of checking for existence of /tmp/.X11-unix/X<n>, the code
will now attempt to connect to the socket to confirm that there is a
server process listening on this UNIX domain socket. This will
eliminate false positives in the case the UNIX domain socket still
exists but is not associated with a listening Xorg server. The Xorg
server does not have a problem with fixing this file when it next
starts up. It is only important to avoid using the port if it is still
in use.

vncserver: Display check should be re-factored for Perl 5

The display check and related setup code was complex due to
compatibility with Perl 4. This included different ways of obtaining
system constants and building system data structures. Perl 5 provides
direct constants and utility methods to abstract this out of the
vncserver code.

vncserver: Display check should confirm lock file is still valid

If the X server is not shut down cleanly, it may leave lock files
around that "vncserver" has previously mis-identified as evidence that
the display number is still in use.

Instead of checking for existence of /tmp/.X<n>-lock, the code will
now extract the PID from the lock file and confirm that a process
exists with the same PID. This will eliminate false positives in the
case that this file references a PID that no longer exists. The Xorg
server does not have a problem with fixing this file when it next
starts up. It is only important to avoid using the port if it is still
in use.

vncserver: Display check should avoid duplicate code

The display check had duplicate code to first check if the X11 port is
not in use, and then check that the RFB port is not in use. Eliminate
the duplicate code by using a for-loop.

vncserver: Display check should use named variables

Several calculated values such as file paths and port numbers were
hard coded within strings or expressions, sometimes multiple
times. These values should be extracted into named variables to
improve self-documentation and avoid accidental divergence.

vncserver: Display check should check for lock file first

Checking for the existence of `/tmp/.X<n>-lock` is a fast read-only
operation, while calls to bind() or connect() are more expensive and
may have side effects. Perform the fast operation first.

vncserver: Remove legacy HP-UX support

The check for existence of `/usr/spool/sockets/X11/<n>` has been
removed. This file is only relevant on HP-UX, and TigerVNC dropped
support for HP-UX in commit 31e5aa3.

EL 7 tigervnc.spec updates

Add missing build requirements: libpng-devel, zlib-devel

Remove obsolete build requirements: libXfont-devel

Fix unexpected rpm macro expansion `%{VERSION}` which is supposed
to be passed through as literal to `rpm -q --qf`. This hasn't
caused problems before, but it was found to cause problems when
built from EL 8 when it expanded to "1.10.1".

Enhanced ComparingUpdateTracker to crop changed blocks

64x64 changed block can be large for fine changes such as cursor
movement and typing in terminal windows, or an update to a clock.

If the block can be efficiently cropped, this will reduce latency
and bandwidth. Every pixel cropped is a pixel less to analyze, encode,
transmit, and decode.

The previous code already detected the top of the change in order
to determine if the block had changed. However, it did not use
this information to reduce the size of the change rectangle, nor
did it calculate any of the other edges.

The new code introduces detection of the other edges, and uses
the information to build a reduced area change rectangle. This
has the additional effect of reducing the number of discrete pixel
values in the change block which may allow a more efficient
encoding algorithm to be selected.

As this section of code is performance sensitive, the method
of detecting the edges has been optimized to quickly fall back
to pessimistic values as soon as a single comparison fails on
each edge. In the case that full 64x64 block are changing,
there will be three extra comparisons per block.

In cases where the change rectangle can be reduced from 64x64,
the reduced size of the change rectangle represents reduced
effort to encode, transfer, and decode the contained pixels.

In the case of images with high frequency changes, which
specifically includes text, the lossy JPEG encoding can be
highly distorted, especially with JPEG level 6 or less. The
quick flash from a distorted JPEG to a lossless JPEG can
appear as a flickering to some people. This effect was more
obvious when the surrounding area is not expected to change,
but is being distorted anyways due to being part of the 64x64
blocking algorithm.

In the case of a user typing in a terminal window, this change
may commonly reduce the number of pixels updated with every
character typed from 4096 pixels (64x64) to 640 pixels (32x20)
or less.

Warn that WinVNC is unmaintained

Split WinVNC to a separate installer

It doesn't work very well so it shouldn't be included by default.

Run maketarball through bash

It uses bash-isms so a POSIX shell will not work correctly.

Move release cmake stuff to release directory

It's just confusing when this is split over two directories.

Test packaging on Travis

Don't just build things, also test the various packaging.

Merge branch 'master' of https://github.com/ayoder770/tigervnc

Merge branch 'master-Fix-division-by-zero-exception-in-SSecurityPlain' of https://github.com/MarkMielke/tigervnc

Correction to socket error connection message

Merge branch 'user-trailing-spaces' of https://github.com/grulja/tigervnc

Fix division by zero exception in SSecurityPlain.

If using SSecurityPlain and the user specifies an empty username
and password, it will invoke InStream::checkNoWait(0) which will
cause a division by zero when calculating the number of available
items.

Enhance InStream::check() to behave properly when asked for
zero items, or zero sized items.

Add comments to InStream::check(), InStream::checkNoWait(),
and InStream::readBytes() to document expected behaviour
when requested to check or read zero items, or an item with
zero size.

Remove trailing spaces in user name

It's quite easy to make a mistake and add an additional space when configuring
users in the vncserver.users config file. You will then get an error that the
user doesn't exist and it's hard to spot the mistake. Same applies for a space
before the display number.

Fix some inappropriate format specifiers

A "0x" prefix suggest hexadecimal values, not decimal.

Fix non-functional MaxDisconnectionTime

Since 8e09912 this wasn't triggered properly as we checked if all
clients were gone before we actually removed the last client from our
list.

Update gesture unit test to cover angle cornercase

If the difference in angle is larger than 180 degrees we hit a corner
case. This commit modifies the coordinates of an existing test to cover
this.

Use hypot(x,y) instead of sqrt(x*x + y*y)

Merge branch 'regions' of https://github.com/CendioOssman/tigervnc

Improve names of many gesturehandler unit tests

Makes it easier to understand what the tests do.

Fix typos in comments

Remove unnecessary touchEnd from unit tests

Add proper spacing to if statements

Add constant for TWOTOUCH_TIMEOUT

Update Info.plist copyright, added recommended key.

Automatically pick the correct X.org patch

Update RPM package build process to automatically detect and apply
the correct X.org patch.

Merge branch 'travis' of https://github.com/CendioOssman/tigervnc

Set up Windows and macOS travis builds

Use mingw's stdio routines where needed

It allows us to use the more standard format specifiers.