summaryrefslogtreecommitdiffstats
path: root/common/rfb
Commit message (Collapse)AuthorAgeFilesLines
* Flush decoder errors on close()Pierre Ossman2022-11-071-0/+10
| | | | | | We don't want any stray exceptions as we are cleaning up, so handle any still pending decoder errors by just logging them. We are already shutting down so there is no need to abort the connection here.
* Don't enable RSA-AES by default in serversPierre Ossman2022-10-141-3/+0
| | | | | | These require a key to have been set up on the server beforehand, so they do not give a good default experience as clients will be unable to connect.
* Ignore whitespace around components of host-and-port specificationCatherine Tower2022-09-011-2/+22
| | | | This is to make the code more tolerant of typos when entering a hostname
* Fix incorrect nettle library referencesPierre Ossman2022-09-011-1/+4
|
* Add support for RSA-AES security typespdlan2022-09-0112-13/+1316
|
* Move include_directories() to proper placesPierre Ossman2022-08-251-1/+5
| | | | We should scope these as narrowely as possible to avoid side effects.
* Remove unused CSecurity::description()Pierre Ossman2022-08-2510-25/+3
|
* Safely discard large (extended) clipboard contentsPierre Ossman2022-06-282-8/+40
| | | | | | | | | Avoid having to buffer everything we want to discard, and instead do it piece by piece. This is more efficient, and avoids hitting any limits on the buffering. Note that this is safe here because we already know we have all the compressed data. It would not be safe for a general input stream.
* Merge branch 'mft-crop-fix' of https://github.com/mmozeiko/tigervncPierre Ossman2022-06-132-8/+220
|\
| * Workaround for MFT cropping in H264 decodingMartins Mozeiko2022-06-072-8/+220
| | | | | | | | | | | | | | | | | | It seems MFT h264 decoder does not support frame cropping. Which means that if frame width or height is not multiple of 16 then decoded output can potentially be used with wrong offset. This code adds explicit parsing of SPS to extract cropping information to use, and will apply cropping if reported size differs from expected.
* | Find FFMPEG using pkg-configPierre Ossman2022-06-071-0/+1
| | | | | | | | | | | | This is the standard method of finding it on Unix systems, so make sure we use it. Still keep a fallback, though, for other systems, e.g. Windows.
* | Fix up ffmpeg header and library handlingPierre Ossman2022-06-071-1/+2
| | | | | | | | | | FFMPEG is needed by the RFB library, so all the details about it should only be applied there and not for other components.
* | Use pkg-config for pixmanPierre Ossman2022-06-071-2/+3
| | | | | | | | | | | | This is the standard method of finding it on Unix systems, so make sure we use it. Still keep a fallback, though, for other systems, e.g. Windows.
* | Avoid temporary cmake variablesPierre Ossman2022-06-071-24/+13
|/ | | | | Modern cmake has better support for adding source files and libraries incrementally, so let's use that to clean things up.
* Assert that accessCheck isn't called too earlySamuel Mannehed2022-04-291-0/+3
| | | | | These checks depend on accessRights being set up, this has only happened if we have done the QUERYING steps.
* Initialize SConnection's accessRightsSamuel Mannehed2022-04-291-1/+1
| | | | | Properties should be initialized in order to avoid random values in case they are used earlier than expected.
* Print GnuTLS certificate status as hexPierre Ossman2022-03-241-1/+1
| | | | It's a bit field so it's easier to decode in hex.
* Allow exceptions for expired certificatesPierre Ossman2022-03-241-20/+55
| | | | | The browsers allow users to make an exception for expired certificates, so we should probably also.
* Make sure server name is always a valid stringPierre Ossman2022-03-242-2/+9
| | | | | Otherwise we can get crashes on NULL dereference. This should only happen on reverse connections where we don't have a server address.
* Fix compilation issue.Chris2022-02-281-1/+1
| | | | | | | | | | | | | | | | | Ubuntu 20.04 gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0 libx264-dev:amd64 2:0.155.2917+git0a84d98-2 [ 11%] Building CXX object common/rfb/CMakeFiles/rfb.dir/H264LibavDecoderContext.cxx.o tigervnc/common/rfb/H264LibavDecoderContext.cxx: In member function ‘virtual bool rfb::H264LibavDecoderContext::initCodec()’: tigervnc/common/rfb/H264LibavDecoderContext.cxx:50:40: error: invalid conversion from ‘const AVCodec*’ to ‘AVCodec*’ [-fpermissive] 50 | AVCodec *codec = avcodec_find_decoder(AV_CODEC_ID_H264); | ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~ | | | const AVCodec* make[2]: *** [common/rfb/CMakeFiles/rfb.dir/build.make:895: common/rfb/CMakeFiles/rfb.dir/H264LibavDecoderContext.cxx.o] Error 1 make[1]: *** [CMakeFiles/Makefile2:597: common/rfb/CMakeFiles/rfb.dir/all] Error 2 make: *** [Makefile:130: all] Error 2
* get rid of c++11 stuffVladimir Sukhonosov2022-01-252-3/+3
|
* Improve logging around screen resizingPierre Ossman2022-01-252-4/+17
| | | | | | Avoid duplication, and add logging for some more error conditions so it's possible to see in the logs why a resize has failed or produced unexpected results.
* support H264 decoding with MediaFoundation on WindowsMārtiņš Možeiko2022-01-213-0/+382
|
* H264 DecoderVladimir Sukhonosov2022-01-2110-0/+617
| | | | Linux implementation using ffmpeg
* Fix handling of VMware cursorsPierre Ossman2022-01-181-2/+2
| | | | | | This is a regression from ad0f061. If a VMware cursor rect was split up over multiple read()s then the stream would become corrupted as we set the restore point at the wrong place.
* Be consistent in including config.hPierre Ossman2021-12-3062-20/+250
| | | | | | The generally recommended way is to include it from source files, not headers. We had a mix of both. Let's try to be consistent and follow the recommended way.
* Log decoding stats on disconnectPierre Ossman2021-11-042-0/+66
| | | | | Can be helpful to see what encodings were actually used during a connection, and how they performed.
* Throw more descriptive error on rect read errorPierre Ossman2021-11-041-2/+6
| | | | Gives a bit more context where this error happened.
* Remove single cpu decoder shortcutPierre Ossman2021-11-041-27/+1
| | | | | Single CPU machines are extremely rare now, so let's avoid the hassle of multiple code paths.
* Merge branch 'master' of https://github.com/twhitehead/tigervncPierre Ossman2021-10-141-1/+1
|\
| * System certificate load returns number loaded not GNUTLS_E_SUCCESSTyson Whitehead2021-10-071-1/+1
| |
* | Don't fail on user CA/CRL problemsPierre Ossman2021-10-141-2/+2
|/ | | | | | | | The certificates might still be fine using the system trust store, or the user can make an exception. So let's just log and continue on. This got very common after 960c7d2 where we now always have a default value for these settings.
* Keep ownership of second selection when first is lostPierre Ossman2021-07-201-4/+3
| | | | | | | | | | | | | | This fixes regression introduced by the extended clipboard extension. Previously it was possible for the server to hold on to the CLIPBOARD selection even if another application took ownership of PRIMARY. This is important to handle the common use case of selecting something in order to paste over it. The new request based model doesn't readily support this as we assume the client has lost its data once we push the new PRIMARY selection to it. So to handle this we have the maintain a cache of the client's data, and make sure to fill that cache before we do anything that might cause the client to lose the data.
* Fix priority string when using newer GnuTLSJan Grulich2021-07-162-2/+6
| | | | | The call of gnutls_set_default_priority_append() expects a normal priority string, which means it must not start with ':'.
* Merge branch 'utilize-system-wide-crypto' of https://github.com/grulja/tigervncPierre Ossman2021-07-143-29/+105
|\
| * Utilize system-wide crypto policiesJan Grulich2021-07-143-29/+105
| |
* | Merge branch 'master' of https://github.com/jasonsikes/tigervncPierre Ossman2021-06-162-4/+41
|\ \
| * | Use RFC7919-2048 group in GnuTLS for FIPS compliance.JASON SIKES2021-06-122-4/+41
| | |
* | | Avoid some variable aliasingPierre Ossman2021-06-142-2/+2
| | |
* | | Remove unnecessary header includePierre Ossman2021-06-131-1/+0
| | |
* | | Clean up default value for X509 parametersPierre Ossman2021-06-136-42/+23
| | | | | | | | | | | | | | | Let's avoid making this too complex and force every user to know about magical functions.
* | | Keep own memory for duplicate config valuePierre Ossman2021-06-112-6/+11
| | | | | | | | | | | | | | | Allows the default value to be dynamically generated in a buffer that is reused.
* | | Remove unused LogParameter::setDefault()Pierre Ossman2021-06-112-14/+0
|/ /
* | Don't wait for TLS close responsePierre Ossman2021-06-112-2/+6
| | | | | | | | | | Our current architecture doesn't support waiting for a response here, so don't even try or we'll just get an error.
* | Handle GnuTLS shutdown on dead sessionPierre Ossman2021-06-113-11/+16
| | | | | | | | | | The session might have died, or failed to initialise properly, so be prepared for gnutls_bye() to be unable to do its job.
* | Properly clean up client VeNCrypt handlingPierre Ossman2021-06-111-0/+1
| | | | | | | | | | | | We would leak the security module below the top client VeNCrypt module, meaning that those modules would not get a chance to shut down gracefully.
* | Remove unneeded NULL checksPierre Ossman2021-06-114-15/+6
| | | | | | | | | | It's perfectly safe to delete NULL pointers, so simplify things by removing these checks.
* | CharArray: pre-fill empty array with zeroesJan Grulich2021-06-071-0/+1
|/ | | | | | | | CharArray should always be null-terminated. There is a potential scenario where this all might lead to crash. In Password we call memset(), passing length of the array we get with strlen(), but this won't return correct value when the array is not properly null-terminated.
*-. Merge branches 'vmware-cursor-position' and ↵Pierre Ossman2021-03-1114-9/+115
|\ \ | | | | | | | | | 'vmware-cursor-position-vncviewer' of https://github.com/lhchavez/tigervnc
| | * Support the VMware Cursor Position extension on vncviewerlhchavez2021-03-117-3/+30
| |/ |/| | | | | | | | | | | | | This change makes it possible for re-synchronizing the remote cursor on the vncviewer when in fullscreen mode. This is done by locally moving the cursor position to what the server thinks it should be. Now SDL games should work!
generated by cgit v1.2.3 (git 2.39.1) at 2025-04-04 08:29:19 +0000