Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
Acked-by: Jeremy White <jwhite@codeweavers.com>
|
|
Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
Acked-by: Jeremy White <jwhite@codeweavers.com>
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
This fixes a bug with --config=handling.
Removed old testing code as obsolete instead of updating unused code.
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
This is largely the work of Frediano Ziglio <fziglio@redhat.com>.
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
We were overly dependent on timing for success; now we
deliberately wait for our GC and drawing to finish.
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
When launching with no config file or arguments, the "You must provide
some authentication method" dialog appears. Clicking the Close button
would hang the application and it would never quit.
s->draw_command_in_progress was uninitialized, and flush_and_lock()
would get stuck in an infinite loop waiting for it to be 0.
Initialize the entire session struct to 0.
Signed-off-by: Brendan Shanks <bshanks@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
Quitting with the Quit button or closing the window always causes a
segfault for me. Destroying the surface before removing the interface
seems to resolve the problem.
log from valgrind:
==7741== Invalid read of size 8
==7741== at 0x6943517: red_qxl_destroy_primary_surface_sync (red-qxl.c:313)
==7741== by 0x6943517: red_qxl_destroy_primary_surface (red-qxl.c:338)
==7741== by 0x6943517: spice_qxl_destroy_primary_surface (red-qxl.c:696)
==7741== by 0x111F60: spice_destroy_primary (spice.c:506)
==7741== by 0x111F60: spice_end (spice.c:694)
==7741== by 0x10CBDA: main (main.c:154)
==7741== Address 0x88 is not stack'd, malloc'd or (recently) free'd
==7741==
==7741==
==7741== Process terminating with default action of signal 11 (SIGSEGV)
==7741== Access not within mapped region at address 0x88
==7741== at 0x6943517: red_qxl_destroy_primary_surface_sync (red-qxl.c:313)
==7741== by 0x6943517: red_qxl_destroy_primary_surface (red-qxl.c:338)
==7741== by 0x6943517: spice_qxl_destroy_primary_surface (red-qxl.c:696)
==7741== by 0x111F60: spice_destroy_primary (spice.c:506)
==7741== by 0x111F60: spice_end (spice.c:694)
==7741== by 0x10CBDA: main (main.c:154)
Signed-off-by: Brendan Shanks <bshanks@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
Acked-by: Frediano Ziglio <fziglio@redhat.com>
|
|
|
|
Provide some basic documentation as to how to use the new
spice-video-dummy driver.
|
|
This makes the dummy driver similar to the mode setting driver.
This work was done by Henri Verbeet, which was in turn based heavily on
the Xorg modesetting driver.
|
|
Do this using the spice_indent script which uses indent to apply
current Spice styles (as per Frediano Ziglio on 2019-05-01).
This also includes a fixup from Frediano which provides braces
around all ifs and removes extra parens from returns.
|
|
This gives x11spice access to a modern virtual/offscreen
frame buffer driver for the Xorg X server. We will then
make slight revisions to make it more suitable for use with Spice.
The overall result should be a more functional stack,
as compared to the existing xf86-video-qxl driver, as we gain
a wide range of Mesa features for free. It also simplifies
the overall process.
This is from https://gitlab.freedesktop.org/xorg/driver/xf86-video-dummy
as of 850c0516.
|
|
Fix build on Ubuntu 16.04
See merge request spice/x11spice!1
|
|
As -Werror flag is used in src/Makefile.am, next error occurs:
gui.c: In function ‘gui_report_error’:
gui.c:134:17: error: format not a string literal and no
format arguments [-Werror=format-security]
message);
Disable -Wformat-security in order to fix the build, before the code is
actually fixed.
Signed-off-by: Sam Protsenko <joe.skb7@gmail.com>
|
|
Libraries flags (like -lglib-2.0) should be provided in the end of
compile command. It's not how it's done right now in tests/Makefile.am,
because _LDFLAGS is used where we should actually use _LDADD. Due to
this, on Ubuntu 16.04 with GCC 5 we have next linking errors:
gcc -lxcb-damage -lxcb-xfixes -lxcb-render -lxcb-shape -lxcb
-lxcb-xtest -lxcb -lxcb-shm -lxcb -lxcb-util -lxcb -lgtk-x11-2.0
-lgdk-x11-2.0 -lpangocairo-1.0 -latk-1.0 -lcairo -lgdk_pixbuf-2.0
-lgio-2.0 -lpangoft2-1.0 -lpango-1.0 -lgobject-2.0 -lglib-2.0
-lfontconfig -lfreetype -lspice-server -lglib-2.0 -lpixman-1
-o x11spice_test tests.o x11spice_test.o xcb.o xdummy.o util.o
main.o -laudit
tests.o: In function `check_binary':
src/tests/tests.c:83: undefined reference to `g_log'
src/tests/tests.c:84: undefined reference to `g_test_skip'
tests.o: In function `test_common_start':
src/tests/tests.c:46: undefined reference to `g_test_build_filename'
src/tests/tests.c:56: undefined reference to `g_log'
src/tests/tests.c:57: undefined reference to `g_test_fail'
src/tests/tests.c:39: undefined reference to `g_test_skip'
src/tests/tests.c:48: undefined reference to `g_log'
src/tests/tests.c:49: undefined reference to `g_test_fail'
Use _LDADD instead of _LDFLAGS for libs, so that libraries are
specified in the end of compile line.
Signed-off-by: Sam Protsenko <joe.skb7@gmail.com>
|
|
Make it compile on the latest Fedora
|
|
|
|
Mostly unused parameters, make it compile with gcc version 6.3.1
|
|
|
|
|
|
If we are using the exit-on-disconnect feature, there may be
an exit() prior to the disconnect callback. This ensures
that the disconnect callback and audit functions fire even
in that case.
|
|
Also add a target to create an rpm.
|
|
These scripts save the current desktop background,
and then turn it red while a connection is active.
|
|
|
|
|
|
These scripts enable a user to do some action (such as changing
screen background color) upon connect and disconnect.
|
|
This makes view only the default, and requires an explicit
request to allow-control instead.
|
|
|
|
|
|
(Found by Coverity)
|
|
|
|
Avoids a potential race condition error.
|
|
|
|
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
|
|
This gets us an rpm.
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
|
|
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
|
|
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
|
|
Switch to TODO (Spice standard) where appropriate, remove
elsewhere.
|
|
This involves handling older glib, older gcc, and accomodating
a lack of the the autoconf macros for gconf.
Signed-off-by: Jeremy White <jwhite@codeweavers.com>
|
|
|
|
|