Remove double cursor workaround in client

We've fixed the issue more properly in the server now, so remove the
workaround we had in the client.

Fix bug where server cursor might disappear

We shouldn't unconditionally remove the server side cursor just
because the client supports client side cursors. We need to also
check that the client fulfills the other criteria for taking over
the cursor.

Avoid extra framebuffer update messages

In most cases we will send an update at the end of processing client
messages so these extra calls only slow things down. Server initiated
events still need an explicit call though.

Fix for initial client side cursor

Improve the tracking of what cursor we've sent to the client to make
sure the initial cursor is sent properly. We previously tried to infer
this information based on if we've rendered a server side cursor or
not. This logic broke down if things triggered before we've sent the
first update to the client.

Fix "int-in-bool-context" compile errors

Fix compile error on windows when not building with GnuTLS

Override default window manager key bindings better

Merge pull request #500 from bphinz/issue106-rebase

Hook up the "Load X.509 Cert/Key" dialogs in winvnc.  Fixes #106

Update Chinese (simplified) translation

avoid putting more includes in headers than necessary

Move gnutls functions into SSecurityTLS

Handle FLTK arguments at any position

E.g. vncviewer server.example.com:2 -display :2

Don't open X11 display too early

We have to wait until we've passed the command line arguments or
we might overlook a -display argument.

Fix build Java errors caused by UTF-8 characters in source comments

See issue #534

Raise minimum supported Java version to 7

Changes to keyboard handling require methods from the KeyEvent
class that were not introduced until Java 7.

Override implicit focus transfer when "F10" key is pressed

Java L&F has pre-defined hook that transfers the focus to the
menu of the RootPane.  Since the viewer doesn't have a menu
this just causes a loss of focus and subsequent unexpected
behavior of keyboard and mouse.

Refactoring and implementation changes to keyboard and mouse handling

See issue #505

Add formatted string function to LogWriter.

Need to be careful with escapes though.

Rename CMsgWriter functions to match C++ code

Move XDesktop specific parameters in to same file

Fully standardise on send()/recv()

We already assume sockets here since we use select().

Merge branch 'x0-xdesktop-initial-cursor' of https://github.com/x11clone/x11clone

x0vncserver does not support SetDesktopSize; remove -AcceptSetDesktopSize

Add a Configuration::removeParam to support such cases.

Configuration::listParams wants width/nameWidth; not ConfigurationObject

Removed IRIX/Solaris overlay support

Legacy systems.

Remove support for Lynx/OS

Added my/Cendio copyright where relevant

Set the correct cursor when starting

XFixes only reports cursor changes. We must manually call
XFixesGetCursorImage() when starting.

Use send() with MSG_DONTWAIT when possible

Define cork() as pure virtual in Socket class

This makes it possible to create a derived class from Socket which is
not TCP based, without having VNCSConnectionST.cxx trying to call
setsockopt() on a non-socket.

Move XDesktop to separate file

Define ledNames in XDesktop.cxx, rename N_LEDS to XDESKTOP_N_LEDS

Using namespace rfb in header files is a bad idea

Build with support for MIT-SHM

No need for ifdefs; we can assume MITSHM. Already done in vncviewer.

Avoid corner case of applications rendering at frame rate

There is a problematic corner case where an application is updating
at exactly the same rate that we're sending update. In that case we
may miss updates regularly, getting a very uneven final frame rate.
Avoid this by introducing a slight offset when we start updating.

Fix broken logic for sending mouse cursor

The logic got inverted resulting in the cursor being sent needlessly
whenever the cursor moved.

Update distro package requirements

Patch the el7 xorg source appropriately

Add debug trace when the clipboard changes

Don't validate timestamp on clipboard request

The normal Xorg server doesn't check this field, so neither can we
as there are applications relying on this leniancy.

Make exception classes have clearer messages

Include the type of exception in the string generated by each
subclass. Also simplify the constructs to what is needed.

Add Travis CI configuration

Improve test if library is a CMake target

Newer CMake complains when you try to get the property of something
that isn't a target so use the specific if operator instead.

Avoid getting LOCATION property

It's not supported in newer CMake, so try to deduce the library
filename from the target name.

Push query connect timeout back in to Xvnc

It was moved to the common code in f8e3b34c6, but it was unreliable
because the state could sometimes get out of sync. Push it back in
to Xvnc since it isn't necessarily something all servers will have.

Don't queue a query if it is directly rejected

Otherwise we'll wedge the desktop object which expects an approval
or rejection to eventually show up.

Keep alpha in cursor alpha conversion

Don't touch the existing value or we might mess it up when we avoid
division by zero.

Use current server time for XUngrabKeyboard

We might be calling this outside a normal FLTK event, so we cannot
trust fl_event_time to have a sane value.

Remove accidental debug output

Make sure to start the frame timer after a buffer change

The buffer is considered changed right away, so we might not get any
add_changed() calls to trigger a start of the timer.

Get rid of getFbSize()/getDesktopSize() in common

It was only used by WinVNC, so push it there instead.

Switch back to low resolution mode on macOS

This is a revert of 6c0181c. Testing showed that we got a big
performance hit by enabling this, and we also saw some graphical
artifacts. More work is needed before this can be enabled by default.

Update keyboard test notes for raw keyboard mode

Add keyboard test notes for LED state handling

Be explicit which Exception class we want

Release pointer grab when cursor leaves window

We don't need the grab any more if the pointer cannot click on our
window. This makes it possible to shift focus to another application
when we aren't covering all monitors.

Don't throw exceptions in destructors

Handle unknown keysyms properly in x0vncserver

We would accidentally store they keycode 0 for unknown keysyms, wedging
the lookup table.

Handle XKB groups properly in x0vncserver

Update mouse cursor position in x0vncserver

This is necessary to make sure the server side rendered cursor works
properly.

Only select XFixes event once

We never un-select it, and it's not a problem if we get extra ones so
register it once right away.

Merge branch 'cursor' of https://github.com/alanc/tigervnc

Merge branch 'fix-warnings'

rfb: Avoid unused argument warnings in headers

Don't force users of these headers to squash compiler
warnings about unused arguments, annotate with __unused_attr.

Fix shift state test in lock key heuristics

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

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

Merge branch 'qemukbd-merge' of https://github.com/CendioOssman/tigervnc

Clear client cursor when switching to server side

Otherwise the client can end up with two visible cursors.

Remove unused FLTKPixelBuffer files

These are unused since 403ac27d, but the files were never removed.

Add support for raw keyboard in vncviewer

Make sure it can map between the key codes of the local system
in to the key codes used by the protocol.

Fix indentation bug

Add support for raw key codes to Xvnc/libvnc.so

rfb_win32: Use scan codes if available

If scan codes are available using QEMU Extended Keyboard Messages
from clients, use that to inject scancodes directly into the
system using the SendInput API.

No conversion is needed as Windows uses the same scancode encoding.

Signed-off-by: Rahul Kale <Rahul.Kale@barco.com>
Signed-off-by: Peter Korsgaard <peter.korsgaard@barco.com>

x0vncserver: keyEvent(): use scancodes if available

Use the client provided (using QEMU extendend key event) scancodes if
available instead of reverse mapping the keysym.

X11 unfortunately uses keyboard driver specific keycodes and provides
no direct way to query this, so guess based on the keyboard mapping.

Handle the two most likely keyboard drivers, the old xorgkbd and evdev.

Signed-off-by: Peter Korsgaard <peter.korsgaard@barco.com>

Include server name in TLS handshake for SNI

In case the server is a front-end to multiple systems and needs to
know which system we're after.

Use better security method description when using VeNCrypt

The sub-modules generally provide a better description than just the
short security method name.

Add better error message for insecure certificate algorithms

Display security state when asking for password

Indicate to the user how secure the transport channel is so they
can avoid entering their password for untrusted sites.

Merge authentication dialogs

Avoid having two separate code paths for the user/password and
only password dialogs. Makes it easier to extend things in the future.

Track keys based on client supplied key codes

This makes it easier to provide more sane events to the backend code
even with crazy clients.

Remove unused needsLastRect state variable

Fix wrong argument for CMsgWriter::clientCutText()

As a result we weren't overloading properly.

Basic support for QEMU Extended Key Events

This adds the basic infrastructure and handshake for the QEMU
Extended Key Events extension. No viewer or server makes use of
the extra functionality yet though.

rfb_win32: Add support for LED state notifications

LED support added using Windows GetKeyState() API call.

The state is polled for change in CapsLock/NumLock/ScrollLock
status in the same code block where chages to Cursor shape is polled.

Signed-off-by: Rahul Kale <Rahul.Kale@barco.com>
Signed-off-by: Peter Korsgaard <peter.korsgaard@barco.com>

x0vncserver: add support for led state notifications

Listen for XKb XkbIndicatorStateNotify events for scroll/num/caps lock and
map them to the RFB protocol.

Signed-off-by: Peter Korsgaard <peter.korsgaard@barco.com>

Send lock LED state from server to client

Add support for lock LED state to Xvnc/libvnc.so

Add server side lock key sync heuristic

Based on QEMU's behaviour.

Sync LED state when gaining focus

The state might have changed when we didn't have focus. Get
everything back in sync once we're back in control.

Add client support for LED state sync

Add libpng as a dependency for freetype for static builds

Newer versions of Freetype requires libpng as well.

Hook up the "Load X.509 Cert/Key" dialogs in winvnc

Respect modifiers in x0vncserver

Using XKeysymToKeycode() often gives the incorrect keycode as it
doesn't respect the current modifier state. Use XKB to find the
proper key instead.

This however also means that we need to track the mapping for all
pressed keys to make sure we know the correct keycode when it is
time to release the key.

Support clients that only support alpha cursors

Send cursor pseudo-encodings in order of preference

Some servers respect the ordering of pseudo-encodings as well, so
make sure we list the best one first.

Fully implement support for alpha cursor pseudo encoding.
Alpha cursors are not supported in java on Windows so disable
it in that case.  Ideally it would be nice to be able to test
whether or not the client OS supports it, but at the moment
MS Windows is the only one that I'm aware of that doesn't.

Update min version of gettext required to build with .desktop files

.desktop files support was not added to msgfmt until v0.19

Fixed issue where CLI paramters specified as "-param value" caused
a java.nio.BufferOverflowException while the same parameter specified
as "-param=value" worked fine.

Fixed issue where -dotWhenNoCUrsor cli arg was not being honored

x0vncserver: Use Xfixes to display cursors if available

https://github.com/TigerVNC/tigervnc/issues/361

This is a simple implementation that refetches and transforms the cursor
image every time it changes, and doesn't use the cursor naming functions
of the XFixes extension to save & cache cursor images.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>