summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Add tests for usb-acl-helperJonathon Jongsma2016-03-113-5/+330
|
* usb-acl-helper: add env var for specifying acl helper binaryJonathon Jongsma2016-03-111-1/+4
| | | | | Setting SPICE_USB_ACL_BINARY allows us to execute a custom mock acl helper binary for testing purposes.
* Rename spice_usb_acl_helper_open_acl() to _open_acl_async()Jonathon Jongsma2016-03-113-18/+18
| | | | | Follow established practice and make the function's behavior more explicit.
* Remove spice_usb_acl_helper_close_acl()Jonathon Jongsma2016-03-113-18/+0
| | | | | | This function is now only called after the open_acl() task completes, and in that scenario it basically does the same thing that the SpiceUsbAclHelper destructor does, so it's pointless. Remove it.
* usb-acl-helper: Avoid deadlock when cancelledJonathon Jongsma2016-03-111-1/+1
| | | | | | | | | | | cancelled_cb() (which is triggered when the GCancellable's "cancelled" signal is emitted) called spice_usb_acl_helper_close_acl(), which calls spice_usb_acl_helper_cleanup(), which in turn calls g_cancellable_disconnect(). Calling g_cancellable_disconnect() from within a "cancelled" handler results in a dealock, as mentioned in the documentation. Instead of closing the acl here, simply cancel the task here. The cleanup() call will happen when the SpiceUsbAclHelper object is destroyed.
* Introduce spice_usb_acl_helper_cancel()Jonathon Jongsma2016-03-113-1/+15
| | | | | | | | | | | This function explicitly cancels a open_acl() task. It is similar to close_acl(), but its behavior is more explicit. This function is very similar to the existing close_acl() function but it requires that the task has not already been completed. Also, although it releases its reference on priv->result to avoid circular references, it doesn't call cleanup(), preferring instead to wait for the destructor to clean up. This makes the close_acl() function essentially pointless and will be removed in the following commit.
* Update spice-common submoduleMarc-André Lureau2016-03-101-0/+0
| | | | | | Pick codegen build fix Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
* spice-gtk: Fix the signedness of a couple of mm-time tracesFrancois Gouget2016-02-231-2/+2
| | | | | Signed-off-by: Francois Gouget <fgouget@codeweavers.com> Acked-by: Marc-André Lureau <mlureau@redhat.com>
* namedpipelistener: do not leak the error messageIgnacio Casal Quinteiro2016-02-231-0/+1
|
* Simplify spice_usb_device_manager_device_to_libdev()Christophe Fergeau2016-02-181-6/+1
| | | | | | The Windows-specific version duplicates some code from spice_usb_device_equal_libdev(), this commit switches to using that helper instead.
* Simplify spice_usb_device_equal_libdev()Christophe Fergeau2016-02-181-9/+4
| | | | | | The Windows-specific version duplicates some code from spice_usb_device_manager_libdev_match(), this commit switches to using that helper instead.
* gtk: add spice-widget GL scanout supportMarc-André Lureau2016-02-166-21/+779
| | | | | | | | | | | | | | | | Hook to spice-glib events to show the GL scanout. The opengl context is created with egl, and is currently x11-only (supporting wayland with bare-egl doesn't seem trivial). Using GtkGLArea is left for a future series, since SpiceDisplay widget is a GtkDrawingArea and can't be replaced without breaking ABI. Furthermore, GtkGLArea won't work on non-egl contexts, so this approach is necessary on gtk+ < 3.16 or X11 (because gdk/x11 uses glx). Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Acked-by: Victor Toso <victortoso@redhat.com> Acked-by: Pavel Grunt <pgrunt@redhat.com>
* glib: add local GL scanout supportMarc-André Lureau2016-02-136-3/+196
| | | | | | | | | | | | | | Add spice-glib support for gl scanout messages. A note about SpiceGlScanout: it is struct with scanout details, registered as a boxed type, with associated gl-scanout property. That way, it doesn't need a seperate signal for change notification and the current scanout can be retrieve with gobject getter. Since boxed property are always duplicated by g_object_get(), an additional spice_display_get_gl_scanout() method returns the current scanout without duplication (that's what spice-gtk display widget will use). Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
* build-sys: check for epoxy-egl supportMarc-André Lureau2016-02-131-0/+26
| | | | | | | The following code requires epoxy with egl support. Check for pkg-config and egl headers. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
* Add spice_channel_unix_read_fd()Marc-André Lureau2016-02-132-0/+71
| | | | | | Utility function used in the messages with socket ancillary fd. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
* build-sys: bump to spice-protocol 0.12.11Marc-André Lureau2016-02-131-1/+1
| | | | | | Needed for GL messages Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com>
* typo: s/GTask/GSimpleAsyncResultFabiano Fidêncio2016-02-121-1/+1
| | | | | | | | | Seems that I messed up with cd0c1008316e90bce925e1448ffcabb366e88f8f while rebasing my GTask series on top of this patch, causing a build breakage due to non-intentional insertion of a GTask replacing a GSimpleAsyncResult. Acked-by: Pavel Grunt <pgrunt@redhat.com>
* Use #ifdef {HAVE,USE}_FOO instead of #if {HAVE,USE}_FOOFabiano Fidêncio2016-02-127-26/+26
| | | | | | | | | | | | | | | | | | | While "#if USE_FOO" checks for the value of the variable USE_FOO, "#ifdef" checks whether USE_FOO is defined or not. It means, if we had something like: #define USE_FOO 0, we would have: #if USE_FOO /* Any code in here would NOT be compiled */ #endif #ifdef USE_FOO /* Any code in here would be compiled */ #endif No problem was faced on spice-gtk till now because either USE_FOO is not defined or defined as 1, but let's try to have it standardized. Acked-by: Eduardo Lima (Etrunko) <etrunko@redhat.com>
* gtk: replace configure-event for size-allocateMarc-André Lureau2016-02-031-7/+4
| | | | | | | | | | For size-allocate, it's simpler to connect to the signal rather than override the default signal handler since there is no automatic chaining to parent handler otherwise. Doing this removes the need to have a window. Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Acked-by: Victor Toso <victortoso@redhat.com>
* usb-device-{manager,widget}: Add counter of free channelsFabiano Fidêncio2016-01-222-5/+47
| | | | | | | | | | As the message showed when the last usbredir channel is taken can be a bit confusing, let's add a counter of free channels to the widget's label. In order to add the counter, a new property for SpiceUsbDeviceManager was introduced ("free-channels"). Related: rhbz#1298772
* Remove GSLice usageFabiano Fidêncio2016-01-227-21/+22
| | | | | It's being slowly deprecated in glib https://bugzilla.gnome.org/show_bug.cgi?id=754687
* gstaudio: set output parameter to NULL on errorVictor Toso2016-01-191-0/+4
| | | | | | | | This is not really triggered in the current code but this is usually expected in case of errors; Also, the same function on record side already does this. Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* build-sys: silence vala deprecated function usage warningsMarc-André Lureau2016-01-141-0/+1
| | | | | | | This silence usage of deprecated glib declarations from vala generated code. Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* Update spice-commonMarc-Andre Lureau2016-01-141-0/+0
| | | | | | | | | | | | | | | This is an update to fix build issues with spice-protocol.git. generated_server_demarshallers.c: In function ‘parse_msgc_display_gl_draw_done’: generated_server_demarshallers.c:767:23: error: ‘SpiceMsgcDisplayGlDrawDone’ undeclared (first use in this function) mem_size = sizeof(SpiceMsgcDisplayGlDrawDone); ... Also needed for upcoming gl-scanout messages. Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* channel-main: remove unused header rect.hVictor Toso2016-01-071-1/+0
| | | | | | | It was included in be8ff99571478deb5c8d116134f65ed2b788dbd3 but with no use. Acked-by: Marc-André Lureau <mlureau@redhat.com>
* build-sys: remove xrandrMarc-André Lureau2016-01-042-8/+0
| | | | | | Not needed anymore (probably since gnome-rr was removed in 30986505b) Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
* Grab keyboard based on session focus.snir sheriber2015-12-213-2/+44
| | | | | | | | | | | | | | | | | | | | When using multiple monitors moving mouse between monitors releases keyboard grab. Reproduce bug -Open multiple monitors remote-viewer session -Click on one of the monitors to get focus & keyboard-grab -Move mouse to another monitor and try keyboard command (do not click) At this point all keyboard commands are being executed on the client machine instead of the remote machine I added keyboard_has_focus and mouse_has_pointer variables at the session and now these properties are being tested for the session instead for the current widget (works also when using alt-tab). Resolves: rhbz#1275231 Acked-by: Pavel Grunt <pgrunt@redhat.com>
* ppc: Fix spicy-screenshot colors on BE machinesLukas Venhoda2015-12-211-0/+6
| | | | | | Save screenshot in proper endianess xRGB on LE machine and BGRx on BE machine Acked-by: Pavel Grunt <pgrunt@redhat.com>
* ppc: Fix colors on ppc when using GLZLukas Venhoda2015-12-211-1/+1
| | | | | Fixes color order on PowerPC when using GLZ image compression. Acked-by: Pavel Grunt <pgrunt@redhat.com>
* ppc: Fix message endianessFabiano Fidêncio2015-12-211-24/+37
|
* ppc: Fix header endianessFabiano Fidêncio2015-12-211-10/+10
|
* build-sys: Use sasl check from spice-commonPavel Grunt2015-12-182-45/+2
| | | | | | Support only libsasl2 providing libsasl2.pc file Acked-by: Christophe Fergeau <cfergeau@redhat.com>
* Call spice_marshaller_flush on message writeJavier Celaya2015-12-101-0/+1
| | | | | | | | | spice_marshaller_flush must be called before writing a message to calculate pointer offsets. This is only an issue when ptr submarshallers are used, which currently is only used by Tunnel::service_add. Since this is disabled by default, this buug is not going to be hit by the current codebase.
* gst: Fix typo in preprocessor symbolChristophe Fergeau2015-12-091-2/+2
| | | | | When the GStreamer backend is used, HAVE_GSTAUDIO is defined, not HAVE_GST_AUDIO.
* smartcard: Fix compilation with older libcacardChristophe Fergeau2015-12-092-0/+8
| | | | | | | | e0c2182937 dropped some #include from channel-smartcard.c and spicy.c as they were redundant with the headers already included from spice-common/common/messages.h. While this is true for the newer libcacard.h header, we need to include more headers to support older libcacard, or compilation breaks.
* spice-gtk: Recover from GStreamer audio initialization errorsFrancois Gouget2015-12-091-5/+7
| | | | | | | gst_init() will terminate the program in case of a failure so it's better to use gst_init_check() so we can recover from errors. Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
* build-sys: Warn about missing GStreamer 1.0 audio runtime elementsFrancois Gouget2015-12-081-1/+5
| | | | | | | | Their absence should not be reported as an error since they are not needed for the build, but the developer will need them to test the code so it's appropriate to warn him. Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
* build-sys: Check for GStreamer 1.0 audio with SPICE_CHECK_GSTREAMER()Christophe Fergeau2015-12-081-11/+8
| | | | Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
* build-sys: Allow simultaneous support for Pulse and GStreamer audioFrancois Gouget2015-12-083-40/+46
| | | | | | | | | | Rather than GStreamer/PulseAudio backend being mutually exclusive at compile-time, this commit allows to enable both at the same time. PulseAudio will then be favoured, with a fallback to GStreamer if it's not available. Note that --with-audio is kept for backward compatibility. Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
* Use libcacard.h if possibleMarc-André Lureau2015-12-083-10/+5
| | | | | | Drop unneeded includes (that are already undirectly included). Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com>
* Remove unmaintained spicy.nsisMarc-André Lureau2015-12-084-236/+3
| | | | | | Remove gtkrc too, and cleanup EXTRA_DIST ($(desktop_in_files) is unused) Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com>
* Update spice-commonMarc-André Lureau2015-12-081-0/+0
| | | | For new libcacard.h usage
* Fix use after free of uriMarc-André Lureau2015-12-081-1/+1
| | | | | | | | "path" points to "uri" memory, move free(uri) later. Found thanks to valgrind :) Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com>
* Update spice-common submoduleChristophe Fergeau2015-11-251-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | The next commit is going to use SPICE_WARNING which was added recently to spice-common This submodule update changes: Christophe Fergeau (5): build-sys: Add gio-2.0 to SPICE_CHECK_GLIB2 build-sys: Fix error in SPICE_CHECK_LZ4 description build-sys: Set automake conditional in SPICE_CHECK_SMARTCARD build-sys: Rename SUPPORT_GL to HAVE_GL build-sys: Add missing # to comment Francois Gouget (3): build-sys: Add the SPICE_WARNING() and SPICE_PRINT_MESSAGES m4 macros build-sys: Add SPICE_CHECK_GSTREAMER() build-sys: Add SPICE_CHECK_GSTREAMER_ELEMENTS() Lukas Venhoda (4): ssl-verify: Only check addr length when using IP addr m4: Require glib version >= 2.22 ssl-verify: Changed IPv4 hostname to IPv6 canvas_base: Remove redundant switch case block
* build-sys: Use SPICE_WARNING() to issue the DBus warningFrancois Gouget2015-11-251-3/+3
| | | | Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
* build-sys: Prefix the configure audio GStreamer variables with GSTAUDIO_Francois Gouget2015-11-242-11/+11
| | | | | | | | This lets us avoid naming conflicts when using GStreamer for other purposes. Signed-off-by: Francois Gouget <fgouget@codeweavers.com> Acked-by: Victor Toso <victortoso@redhat.com>
* file transfer: send cancel message when user cancels a transferJay.han2015-11-191-1/+2
| | | | | | | | | When a user cancel's a file transfer, we should send a STATUS_CANCELED message rather than an STATUS_ERROR message. The latter triggers a bug in the vdagent where the file remains opened by the vdagent in the guest and cannot be removed. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com>
* Be more specific in the case of authentication errorMarek Kasik2015-11-193-5/+31
| | | | | | | | This patch adds SPICE_CLIENT_ERROR_AUTH_NEEDS_USERNAME error for the case when authentication fails because of missing username. This can happen when GSSAPI method is used. https://bugs.freedesktop.org/show_bug.cgi?id=92994
* spicy: Add support for changing compressionPavel Grunt2015-11-181-0/+63
|
* channel-display: Add spice_display_change_preferred_compressionPavel Grunt2015-11-185-7/+38
|
generated by cgit (git 2.34.1) at 2024-05-19 14:38:36 +0000