Use new API for getVncAuthPasswd()

Merge branch 'readonlypassword' of https://github.com/michalsrb/tigervnc into viewonly

Update comment of VncAuthPasswdGetter::getVncAuthPasswd.

Add AccessSetDesktopSize right.

So clients with limited access rights can not affect it.

Merge pull request #80 from bphinz/buildfixes

More RPM spec file cleanup

More RPM spec file cleanup

* Removed obsolete patches from openSUSE packages

Merge pull request #79 from bphinz/buildfixes

SPEC file cleanup

SPEC file cleanup

* Removed the hard coded snap tag, if needed it should be passed in
  via "rpmbuild --define 'snap ...'".
* Changed the hard coded version to "@VERSION@" to make it clear
  that the copy of the spec file in the repository needs to be
  updated manually.

Merge pull request #76 from bphinz/buildfixes

Remove libvnc server module from statically built versions

Remove libvnc server module from statically built versions

* The static libvnc server module can't be loaded anyway so just
  remove it from the el5 and portable binaries.

Merge pull request #75 from bphinz/java

More Java viewer clipboard handler fixes

More Java viewer clipboard handler fixes

* Override TransferHandler.exportToClipboard method to ensure that
  serverCutText updates get sent to the system clipboard.  This
  wasn't always happening when relying on the super class' paste()
  method alone.
* Removed some unnecessary setText() statements and one check for
  whether sending client cut text is enabled.

Change development version to 1.4.80

The 1.4 branch is now live, so change the version number of the
development builds.

Fix setsockopt() call on Windows

They use char* rather than void*, requiring a cast here.

Assume inet_pton() is missing on Windows

It's much more difficult to test for this on Windows since the
headers have version guards. Just play it safe and assume it is
missing. We can remove this check when we raise the base requirements
to Vista (or newer).

Update translation template file

Remove "Error:" prefix from error message

We had already fixed the others, but this one was overlooked.

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

Protect IPv6 address strings with []

This is the format we support when converting a string to an
address, so it makes sense to use them the other way around.

Don't assume that it's IPv4 just because it's not IPv6

There might be more protocols in the future, and we might also
misdetect things depending on varying IPv6 functions being missing.

inet_ntoa() want's the address directly, not a pointer to it

Merge pull request #73 from bphinz/java

Major rework of Java viewer clipboard handling

Major rework of Java viewer clipboard handling

* Clipboard dialog UI significantly improved.
  - Fixes issue where scrollpane size did not track dialog size
  - Removed unnecessary JPanel
  - Adjusted default size of dialog to something more appropriate.
* Clipboard dialog is now a pure clipboard viewer
  - Window title now reflects thati fact.
  - Eliminates problems with concurrency and updating contents.
  - Clipboard dialog now updates contents if a clipbard transfer
    occurs and the dialog is visible.
  - Prevents possible loop condition when text in the scrollpane
    is selected (ie: selection buffer triggers clipboard updates).
* Custom TransferHandler implemented.
  - Limits the size of outbound clipboard transfers.
  - On Windows and Linux this implementation does not appear to
    provoke huge memory spike issue when the transferable is
    accessed.  Unfortunately, the OSX implemenation still
    suffers from this problem, but that is a JRE issue. For the
    time being, this at least minimizes the problem.  Additionaly,
    if an OutOfMemoryError is thrown, it's now caught and an error
    is logged rather than the viewer bailing out.
  - Vastly simpler implementation - the copy/paste methods inherent
    in the JTextArea are utilized for interfacing with the system
    clipboard.  This eliminates the need for checking permissions,
    as well as streamlining the code quite a bit.

Merge branch 'ipv6' of https://github.com/twaugh/tigervnc into ipv6

Merge pull request #71 from bphinz/buildfixes

Update BUILDING.txt to document GnuTLS requirements

Removed references to libgcrypt and libgpg-error in BUILDING.txt

TcpSocket: IPv6 handling throughout (#68).

Added more #ifdefs for IPv6.

Update BUILDING.txt to document GnuTLS requirements

See Issue #44 for further details.

Merge pull request #70 from bphinz/java

Fixes for java viewer listen mode, TcpSocket methods, and code cleanup

Fixed issue where java viewer in listen mode did not exit properly.

The viewer count was being incorrectly incremented, preventing the
listening viewer from exiting when the window was closed.

Fix getSockPort methods in java TcpSocket

Although not actually used anywhere, the getSockPort method was
actually returning the the remote socket's port number.

Also simplified some of the other socket methods.

Make sure sockets are shutdown in java viewer listen mode

The finalizer method may not be strictly necessary but for the sake
of consistency shutdown the socket listener the same way we do for
a regular TcpSocket.

Java code cleanup

Deleted some useless comments in java viewer

IPv6 loopback support too.

Add IPv6 support (#68).

Revert "Use VncAuth as the default security type, ..."

This reverts commit 39d7dc043afcf6f94b9c16942765fdb9a6a3202e.

We should be secure by default, even if it might cause a performance
hit. Those who wish to make that trade off are more likely to notice
the lack of performance than others noticing the lack of security.

Make sure we do an initial resize

The local window manager might give us a window of a different size
than the one we request. Make sure that the remote resize setting
gets respected even in this case.

Always log chosen security type

It's often useful to see if encryption was used, and how the user
authenticated (if at all).

VncAuth: Read and use readonly password.

Try to read second password (for read only access) from rfbauth file. If client
sent second password instead of first one, allow him read only access.

vncpasswd: Ask for read only password.

Ask for optional second password that is used for read only access. Save it
after the main password. This is same format as tightvnc uses.

Allow SSecurity to specify AccessRights for SConnection.

SConnection has AccessRights property that specifies what connected client can
do. Set this property to value given by SSecurity after successful
authentication. This way individual SSecurity subclasses can accept clients but
restrict their access.

Merge pull request #62 from bphinz/buildfixes

Build enhancements for RPM-based distros

Parameterize jar file signing for rpm-based distros

* Allow build time selection of self-signed or keystore signing

Set default file encoding to prevent java compiler warnings

* The keymap files have UTF-8 characters embedded in comments.
  javac complains about this unless the default encoding is set.

Merge pull request #61 from bphinz/buildfixes

Buildfixes for el5 & el6

Remove non-applicable changes from cmake patch

* This line should not have been changed, it's inside an "ifdef(APPLE"
  block anyway, but removing it all the same.

more el5 build fixes

* Link dynamically against libm to avoid relocation errors
  when building mesa on x86_64.

Merge pull request #60 from bphinz/java

More enhancements to java SSLEngineManager

More enhancements to java SSLEngineManager

* Blocking behavior in the read() method was leading to high CPU
  usage.
* GC wasn't purging temporary byte arrays used to transfer data
  between I/O streams and NIO byte buffers, causing excessive
  memory usage.
* Some optimization of read() behavior to reduce the frequency
  of blocking operations.

el6 package enhancement

* Bumped the xserver patch version to keep up with the installed
  version of xorg source.

more el5 build fixes

* Need to pass the path to the static X11 libraries to cmake

more el5 build enhancements

* Fixes for unresolved dependency issues when linking against static fltk
* Major overhaul of static build.  Previously, fltk & tigervnc were built
  against the dynamic system libraries (and thus not entirely portable).
  All of the X11 libraries required for Xvnc are now built first and
  everything is linked against them.

more el5 static build enhancements

Various fixes for linker issues when building fltk statically.

* The native FindX11.cmake module can't be overridden so that
  fltk links against the static libraries that we've just built.
* Need to link fltk against libdl.a or else an unresolved
  dependency error occurs while linking tigervnc.
* Additional fixes to fix dependency resolution issues when
  linking fltk against X11 statically.

el5 build enhancements

The el5 build was derived from fc12.  Adding all patches from fc12's
version of fontconfig.

Merge pull request #59 from bphinz/buildfixes

Static build fix - only attempt to link against libtasn1 if it's present

Static build fix - only attempt to link against libtasn1 if it's present

GnuTLS may be compiled without external libtasn1, so we shouldn't try
to link against it unless it's actually present.

Back out previous change

Needed to merge upstream changes in before creating pull request.

Merge remote-tracking branch 'upstream/master' into buildfixes

Static build fix - only attempt to link against libtasn1 if it's present

GnuTLS may be compiled without external libtasn1, so we shouldn't try
to link against it unless it's actually present.

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

Handle mingw circular dependencies in base libraries

Our link order should now match the built-in library list.

Handle static linking flags in .la files

Handle arguments that have the -l prefix

Split up any entries that contain multiple link arguments

This happens sometimes in order to enforce a strict ordering.

The output variable from find_library() needs to be cleared

Otherwise you'll just get the initial response back every time
you call it.

Remove obsolete reference to staticlib

We don't use that directory anymore.

Merge pull request #53 from bphinz/buildfixes

Major overhaul of SSLEngineManager

Major overhaul of SSLEngineManager

The SSLEngineManager was basically an abomination. The work is now
done as it should be, with the buffers being fed and the engine
deciding when data is consumed or produced. The engine should be
much more robust now as well.

Additionally, although JRE 7 supports the TLSv1.1 and TLSv1.2
protocols, they are not actually enabled by default. The JSSE
reference cites compatibility reasons for this but this doesn't
appear to be the case with the TigerVNC server and they will be
enabled by default in JRE 8.

The regular expression for enabling anonymous DH cipher suites
was too narrow and excluded the elliptic curve ciphers, which
are now ordered ahead of the ephemeral ciphers by the default
security provider.

Lastly, increase the size of the buffer in FdOutStream from 8Kb
to 16Kb.  I'm not sure why FdInStream and FdOutStream were
asymmetric to begin with, but 16Kb is the default size for TLS
packets and there seems to be now negative effects on plain
text connections.

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

Use dynamic builds by default

Static builds are the special case, so it should be left off
by default.

Remove unused variable

Remove verbose TLS logging

We don't even log this much for RFB protocol stuff, and it makes
it very annoying to run with full debugging on.

Include gettext and zlib in static GnuTLS link

This is needed to make sure they are always included, and in the
proper order.

Only link with crypt32 on Windows

crypt32 is a Windows thing, and it is used no matter the crypto
backend.

Put quotes around entire static link variables

CMake is very aggressive about minimising the list of libraries,
so we need to be similarily aggressive about quotes to make sure
we get everything we want and in the correct order.

Merge pull request #50 from bphinz/buildfixes

Fixes for package builds after commit f95272d.

Add in missing dependency for windows builds.

Commit f95272d created some undefined references in GnuTLS.

Set -DBUILD_STATIC=off for package builds.

Commit f95272d set the BUILD_STATIC flag on by default, which is unnecessary for package builds.

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

Merge branch 'remote-shm' of https://github.com/twaugh/tigervnc

Extend "static" build option

Try to link as much as possible statically in an effort to produce
binaries that can be run everywhere.

Add Carbon and Cocoa dependencies to vncviewer

We use both frameworks directly from vncviewer so we should
make sure they get pulled in.

Move ws2_32 dependency to the correct place

It is rdr and network that needs it, not vncviewer.

Link basic X11 libraries last

The extension libraries might depend on libXext or libX11, so
make sure they come last on the link line.

Check the return value from XShmAttach.

Remove libjpeg-turbo build instructions

There is nothing special about this library that warrants special
treatment.

Clean up GnuTLS linking and instructions

GnuTLS can have different crypto backends, and it is rarely gcrypt
these days. So we should not be including that unconditionally,
and should not be pointing people at it either. Also remove the
section about Win32 binaries as those are out of date and probably
insecure. Lastly remove the section about static builds as it is
a general issue and in no way complete with just the GnuTLS portions.

Don't use MIT-SHM on remote displays.

When run from within 'ssh -X' or 'ssh -Y', MIT-SHM will appear to be
available even though it won't work: the shared memory will be on the
machine running vncviewer, not on the machine running the X server it
displays to. Depending on what shm segments are available on the
machine running the X server, the failure may not be apparent when
checking that MIT-SHM works.

Avoid this by not using MIT-SHM when XDisplayName() indicates we may
not be running locally.

Original bug report:
  https://bugzilla.redhat.com/show_bug.cgi?id=1072733

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

Use exceptions rather than asserts for for "normal" errors

Although these are rare, they are still not indicative of bugs so
an exception (which ends up in the log and might also be shown to
the user) is more appropriate than an assert. Asserts should only
be used to catch blatant API misuse and similar.

Use the same style for exception messages

Make sure we check failed memory allocations

Remove unused assert.h include

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

Wrong padding size calculated (copy-paste error)

Make sure stride handling is also tested

Use constants for the test buffer dimensions

Merge pull request #42 from bphinz/buildfixes

Fix pointer initialization to prevent NULL events.

Handle PrintScreen and SysReq on Windows

This key requires a bit extra care on Windows. We get wildly
different messages depending on if Alt is pressed or not. The symbol
chosen now follows the standard behaviour on X11.