H264Libav: Use AVFrame for pixel format conversion buffer

This ensures that the buffer is allocated with the correct alignment and
padding for use with sws_scale.

This fixes out-of-bounds writes which would in some cases cause
segmentation faults and/or heap corruption.

(cherry picked from commit fff17cae7738d6b3d65426a0ce7df76e8034ad1c)

Handle existing config directory in vncpasswd

This fixes commit a79c33d.

(cherry picked from commit 445e0230cf4e939dcc59caf5d5f001c2f7b04da6)

Use proper gnutls_free() on Windows

The underlying issue requiring this hack was fixed ages ago.
Unfortunately, the fixed GnuTLS doesn't consider static linking. So we
need to add a new hack that permits that.

(cherry picked from commit cfa4cc7dd223641176488dc3b41a53e6a81dade4)

Include key code for fake key events

Let's try to mimic a real event as much as possible to avoid bugs.

(cherry picked from commit 6111c0587aec6ea4700a91a460c7027572ab5011)

Don't parse empty fence messages

We need to return here or we'll do an out-of-bounds access on the data
array.

(cherry picked from commit 7af69eb4a7a52d6baf4d09f79ccbec76bd1fcee1)

Avoid XKeysymToString() conflict with libX11

We implement our own version of this, and because of this we carry our
lookup table generated by the same tool that libX11 uses. Unfortunately,
that table is a global symbol and as such there might be a mismatch of
which table should be used.

Make sure we get our own copy by changing the name of it. We use a
define rather than modifying ks_tables.h as that file is automatically
generated.

(cherry picked from commit 01cf61b4e0f4cd5031794f0a82fdf603d20a7852)

Add missing comma in default security type list

Otherwise it merges with the next entry, removing both of them from the
default list.

(cherry picked from commit 4f6a3521874da5a67fd746389cfa9b6199eb3582)

vncsession: use /bin/sh if the user shell is not set

An empty shell field in the password file is valid, although not common.
Use /bin/sh in this case, as documented in the passwd(5) man page, since
the vncserver script requires a non-empty SHELL environment variable.

Fixes issue #1786.

Signed-off-by: Carlos Santos <casantos@redhat.com>
(cherry picked from commit 4db34f73d461b973867ddaf18bf690219229cd7a)

Use std::min replace min

(cherry picked from commit 9aaea93b2d1012b05ea24a15f4269cb9f356c3d9)

Generate better (fake) modelines

This is what Xwayland does, so let's try to do the same to avoid any
incompatibilites with applications.

(cherry picked from commit 876cbf0800634b382b0a99e8cd086f588997f9d8)

Improve unknown keysym handling

Instead of giving up after all free keycodes have been used, Keycodes from previously added keysyms will be reused.

Re: #93
(cherry picked from commit f65433a113ed2cce39161139982112aaa3ddf6e0)

Update Friulian translation

(cherry picked from commit 87029b7f38455ad4ffa0573107707ce221aa3773)

Update Chinese (simplified) translation

(cherry picked from commit 2e3347768d3d19852e2c575d1b9425e23d6efa87)

Explicity install missing deps GitHub Actions

We seem to have gotten this implicitly before, but now that GitHub has
switched to Ubuntu 24.04 we have to be explicit about it.

(cherry picked from commit a4c86a7f07d81450c5e0147c27ed737b7af4547c)

TigerVNC 1.14.0

Fix overflow; "Timer: gettimeofday is broken"

The rounding misbehaved when the tv_usec calculation overflows.

Fixes issue #1782.

(cherry picked from commit 422444acf3a519b75709fb14c02118b75ecbc898)

Be more careful with ComparingUpdateTracker pointer

As of 28c3f12, we might now be running the frame clock even without a
framebuffer present. This means we need to be more careful accessing the
ComparingUpdateTracker, as it might be NULL.

(cherry picked from commit c4075860579ad33011fafee32f34c5d3da71a910)

Round up expiration time for timers

Otherwise we won't properly guarantee that we'll wait *at least* as long
as the specified time.

(cherry picked from commit c831c9084afffdf10fa5dda865a6df5d643d83dd)

Fix x0vncserver handling of timers

Bug fix for bf28683.

(cherry picked from commit 80ce950887824a120c8de07dac9191e7fff72de4)

Update Slovak translation

(cherry picked from commit 60962a99dde101099bc775dd98354a43b9b36c9a)

Update Swedish translation

(cherry picked from commit 0e9a00e87cf12087b253e0be214c7a7a045c94de)

Update Finnish translation

(cherry picked from commit b0c87ad5d437eb9462d5230afd1b75b7bf31db53)

Update Czech translation

(cherry picked from commit c4972537884a42e2171d8a1981d3c2d772f64a14)

Update Indonesian translation

(cherry picked from commit f7e4771c836e07e32680fa525c4fabab46b611f6)

Update Bulgarian translation

(cherry picked from commit e8312b5fad4da00a51151c4492143cf64ea4bf34)

Update Swedish translation

(cherry picked from commit 2095bc16fae7463f513eb747ae99ebf212741842)

Update Hebrew translation

(cherry picked from commit e725e51b11cb00ef9178b68ad14d6191f349a066)

Update Romanian translation

(cherry picked from commit 81dd3035bf5a3eb899fb8433dc466385dd7bc616)

Update Spanish translation

(cherry picked from commit 1ab83ee6a7deedb4fe5e3ca4cef100aed9d435c0)

Update Ukrainian translation

(cherry picked from commit f3026de283d85db01e06c3086dc004068fcb38a0)

TigerVNC 1.13.90 (1.14.0 beta)

Update translation template file

Merge branch 'dri3' of github.com:CendioOssman/tigervnc

Automatically pick a suitable DRI3 render node

Lay the groundwork for a more intelligent selection of GPU in systems
that have multiple cards. This initial implementation simply picks the
first GPU that we have permission to open.

Add option to specify used DRI3 render node

Allows users to use a secondary GPU, or to disable DRI3 in case of
issues.

Hide from GNOME to enable animations

One GNOME developer had a bad experience with VNC one time, and
therefore all VNC must forever be punished.

Avoid detection by GNOME so that animations aren't forcefully disabled
without any option for the user to re-enable them.

Add basic DRI3 support

This provides buffer sync with the primary render node of the system (if
available). By doing so, OpenGL and Vulkan can use the render node for
acceleration.

This first implementation is extremely pessimistic and synchronises
buffers whenever there's even a slight chance they will be used.

Fix includes for Xorg dri3 module

It was incorrectly designed to only build properly if built as part of
the "Xorg" server and could fail to build when just "Xvnc" was built.

Backport of upstream commits 0ce93e5 and acc581c.

Avoid VNC updates for offscreen windows

Windows aren't always directly shown, e.g. when the desktop is
composited. We should not be sending screen updates for these as the
framebuffer didn't actually change.

In the case of CopyWindow, we're even sending the wrong screen data,
which is how this bug was discovered.

Merge pull request #1767 from bphinz/ubuntu-noble

Updates for package builds

Remove support for EoL operating systems

Merge branch 'xserver-cleanup' of github.com:CendioOssman/tigervnc

Remove old Xorg configure arguments

These are no longer used in the versions of Xorg currently in use.

Raise Xorg requirement to 1.20+

No current distribution uses anything older, so let's make things easier
for us to maintain.

Remove Ubuntu 18.04 packaging

Ubuntu 18.04 has been EOL for almost a year, so remove those packages so
we can ease our maintenance burden.

Avoid patching miinitext.c for Xvnc

Things do not need to be this complicated. We can make sure the VNC
extension is always included in a much cleaner way, at least since Xorg
1.16.

Merge branch 'present' of github.com:CendioOssman/tigervnc

Stop explicitly including fbcmap_mi.c

This hasn't been needed since Xorg 1.16.

Slow down fake clock when no clients

Run the frame clock at a slow 1 Hz if there are no clients connected.
This is similar to what a normal X server does when the screen is
blanked, and should keep applications waiting for the frame tick happy.

Note that we still only keep the frame clock running if there is any
application that are interested in it. Otherwise we still stop it
completely.

Stop linking to libX11

Unclear why this was ever needed. libX11 is for X11 clients, and we're
building a server.

Keep frame clock running if waiting for frame tick

If there is something interested in synchronizing to a frame tick, then
keep the frame clock running, even if there are no updates.

This is need mainly when something starts rendering, but also when
something renders much slower than the frame clock (so it is essentially
constantly "starting"). Such an application will not draw anything until
it gets a new frame tick, which it won't get as the frame clock is
waiting for something to start drawing.

Clean up CPPFLAGS handling in xserver

Make things a bit easier to maintain. This style is also more in line
with what is being done for other DDX.

Restart the frame timer immediately

This marks the timer as started again, before we call out to various
external places that might be confused by the frame timer reporting that
it is stopped.

Remove unused NO_MODULE_EXTS define

This has never actually been used, as far as I can tell.

Add support for X Present extension

This makes it possible for applications to synchronize their updates to
the updates sent out to clients. This avoids tearing, and could in the
future also help with rate limiting applications to what the client can
actually show.

Actually use config.h in Xvnc/libvnc.so

Rather than disabling this, make sure the build can find the relevant
file.

Add missing ErrorF() newlines

Simplify Xorg version handling

VENDOR_RELEASE has been available as a #define since Xorg 1.7, so let's
use that and remove all of our custom magic.

Make SDesktop::start()/stop() optional

Let's avoid requring these as a desktop implementation can now set up
everything in the init() method.

Maintain a constant VNCServer/SDesktop connection

The desktop isn't completely paused just because there are no clients,
so it might still need some support from the server object.

This is primarily an issue for headless servers, where they need to
continue emulating things even without clients. A scraping server can
generally go completely passive if there are no clients.

Make it easier to change timer interval

Explicitly request timer repetition

One-shot timers are more common, so let's change the API a bit to make
that use case simpler. This API also makes it more clear what is
happening.

Fix up Timer comments

They were badly formatted, way longer than the normal 72 columns.

Add safety check to getNextTimeout()

It currently won't ever be called with an empty list of timers, but it
is a public function so that might happen in the future. Make sure this
case is handled without crashes.

Stop treating "0" as "no timeouts"

It is much more sane to treat "0" as "a timer is ready NOW", so let's
change to using -1 as the invalid timeout value.

Remove unneeded iteration

This should have been done in a4308c9.

Detect conflicting timer rescheduling

Repeating a timer can be done in two ways:

 * Returning true from the handler
 * Calling start() again in the handler

The latter is useful if you want to change the timer interval.

If both are used, then it becomes ambiguous when the timer should fire
again.

Detect this case and warn about it. Current implementation will respect
the new interval given to start(), rather than the interval set before
running the handler.

Ubuntu packaging updates

Add support for PKCS11 jar signing

Merge branch 'pam_env' of github.com:CendioOssman/tigervnc

Create common recursive mkdir()

Avoid duplicating this complexity in too many places.

At the same time make the interface more identical to regular mkdir(),
for familiarity.

Use $XDG_STATE_HOME for known hosts storage

This is an often changed database, so it is better suited for
$XDG_STATE_HOME rather than the more static $XDG_DATA_HOME.

Only do restorecon() for legacy directory

This is only needed because of an historical type change of the legacy
directory, so avoid doing it anywhere else.

Fix late vncsession logging

The syslog file descriptor will be closed when we are cleaning up in
preparation for running the vncserver script, so we need to explicitly
reopen things in case we need to log errors.

At the same time, try to be polite and explicitly close the log when
appropriate.

Respect PAM modules for $XDG_STATE_HOME

E.g. pam_env.so might modify this variable, so we should see what we get
out of PAM when building a log file path.

Add comment that PAM configuration is an example

There is too much variation between distributions for us to have a good
PAM configuration that works everywhere. Try to make this more obvious
by having a comment at the top of the file.

Merge branch 'casantos-rhbz_2180903' of https://github.com/casantos/tigervnc

Don't check pointers for zero length buffers

It's perfectly fine to give a NULL pointer if the length has explicitly
been specified as zero.

Merge pull request #1737 from 62832/fix-1195

Allow for alternative user config locations, deprecate `~/.vnc` in favour of XDG Base Directory Specification paths

Implement XDG Base Directory paths, deprecate ~/.vnc

vncconfig: add option to force view-only remote client connections

Specifies that the server must ignore all keyboard or mouse events sent
by the client.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2180903
Signed-off-by: Carlos Santos <casantos@redhat.com>

Remove the network::SocketServer interface.

Move these RFB specific things to rfb::VNCServer, for clarity.

Signed-off-by: Pierre Ossman <ossman@cendio.se>
Signed-off-by: Carlos Santos <casantos@redhat.com>

Add .gitignore for SELinux files

Don't trigger build from install target

The norm is that the install target is read only from the point of view
of the source and build directory, so avoid accidentally triggering any
build.

Clean up temporary SELinux directory

Overwrite vncsession.pp.bz2 each time

Make it easier to work iteratively by not having to remove the output
file each time.

Move the AccessRights type and constants to the rfb namespace

They must belong to the rfb namespace, not to the SConnection class.

Also add an AccessNone constant, since it's better to use a mnemonic
symbol rather than zero to initialize the accessRights members.

Signed-off-by: Carlos Santos <casantos@redhat.com>

Use XQueryPointer() instead to detect zaphod

FLTK can send events with fl_xevent pointing to random memory, which
means that we cannot trust that structure. XQueryPointer() can give us
what we need, at the cost of an extra round trip.

Correctly set socket write notification

That extra add_fd() only set the read notification, overwriting any
write notification set up in the lines just above.

Update github actions to latest versions

Primarily to avoid the versions that are now deprecated.

Merge pull request #1742 from TigerVNC/issue843

Fix for issue 843

Fix for issue 843

Begin work on XDGBDS compliance and overrideable configs

Don't link gcrypt for static GnuTLS

GnuTLS seems to have dropped grypt support ages ago, so let's not
confuse things by assuming it might be needed when linking GnuTLS
statically.

Properly propagate CursorWarpedTo()

Although unlikely, there might be other parts of the X server that are
also interested in this call. Make sure we propagate things on properly.

Clarify AcceptPointerEvents setting

It doesn't just control button events, but all types of pointer events,
including movement.

Fix shared memory leak

The MIT-SHM documentation:
  https://www.x.org/releases/X11R7.7/doc/xextproto/shm.html
says to run XShmDetach() first, and then to destroy the segment.

(cherry picked from commit 585ee24d4cd81759ca7206d5d5181a491c2e8898)

Increase time required to override pointerClient

Mouse events in shared connections are locked to one viewer at a time.
If the a mouse drag is happening in the first viewer, the second viewer
can't interfere unless the first viewer has been inactive for a set
amount of time. This timeout was put in place in
71c83b479335337b25dc331d3fb4a6e29382ceb5 to handle the case where the
first viewer disconnects while the mouse drag is happening. Prior to the
timeout, this resulted in all input being blocked.

This commit increases this timeout from 3 seconds to 10 seconds. This is
to minimize the risk of this happening during "normal" use. With this
change, the first viewer has to be inactive for 10 seconds before the
second viewer can take control again.

In the case where the first viewer disconnects, a waiting time of 10
seconds for the second viewer seems very reasonable.

Break long lines in new pointerEvent logic

Fix bad variable reference in file logger