summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* server: not reading command rings before RED_WORKER_MESSAGE_START, RHBZ #718713Yonit Halperin2011-07-051-1/+11
| | | | | | | On migration, destroy_surfaces is called from qxl (qxl_hard_reset), before the device was loaded (on destination). handle_dev_destroy_surfaces led to red_process_commands, which read the qxl command ring (which appeared to be not empty), and then when processing the command it accessed unmapped memory.
* server: replace redundant code with red_cursor_resetYonit Halperin2011-07-051-47/+24
| | | | | In addition (1) make handle_dev_destroy_surfaces call red_release_cursor (2) call red_wait_outgoing_item(cursor_channel) only after adding msgs to pipe
* server: removing local cursor, this solves RHBZ #714801Yonit Halperin2011-07-051-114/+14
| | | | | | | | | | | When the worker was stoped, the cursor was copied from guest ram to the host ram, and its corresponding qxl command was released. This is unecessary, since the qxl ram still exists (we keep references to the surfaces in the same manner). It also led to BSOD on guest upon migration: the device tracks cursor set commands and it stores a reference to the last one. Then, it replays it to the destination server when migrating to it. However, the command the qxl replayed has already been released from the pci by the original worker, upon STOP.
* tests/migrate: update to use argparse and run spicy by defaultAlon Levy2011-07-051-32/+56
| | | | | | | | | Most helpful change is that it prints help if you do --help. It can take --image parameter, and --client parameter, defaults to spicy. Incidentally, spicy switch host seems broken - if you run with --client spicec (assuming spicec is in the PATH), it works fine, but using the default spicy (or --client spicy) it fails after a single switch.
* server/tests/basic_event_loop: strange assert failure; fixAlon Levy2011-07-051-1/+1
|
* server/smartcard: register channel only when hardware is availableAlon Levy2011-07-052-6/+9
|
* Revert "server/smartcard: don't register the channel if no hardware emulated"Alon Levy2011-06-291-4/+1
| | | | | | | | This reverts commit 456ff9f8d5d27ac763e89e67a100661084695dc5. That patch just disabled the smartcard channel completely because the check was done *before* the initialization of the qemu smartcard devices, not after.
* red_worker: remove 2 empty functionsChristophe Fergeau2011-06-281-12/+0
| | | | | red_worker was using some share/unshare stream_buf functions whose body is empty. This commit removes them and their use.
* server/smartcard: don't register the channel if no hardware emulatedAlon Levy2011-06-231-1/+4
|
* server/smartcard: handle BaseChannel messagesAlon Levy2011-06-231-0/+6
| | | | | | According to spice.proto the smartcard channel can receive acks and any other message defined in BaseChannel. While the spicec implementation didn't send an ACK spice-gtk does, so handle it.
* don't #include config.h in generated header filesChristophe Fergeau2011-06-221-3/+4
| | | | | | | | 7e30572ab adds a #include <config.h> to the beginning of generated files. It also does this for generated headers and enums files, which is not wanted, especially if it's an installed file. This commit only adds this include for the non-header non-enum case (hopefully, enums are only generated for use in a .h file).
* s/__visible__/SPICE_GNUC_VISIBLEChristophe Fergeau2011-06-224-59/+59
| | | | | The C specification reserves use of identifiers starting with __ to the compiler so we shouldn't use one such symbol.
* add check for pyparsingChristophe Fergeau2011-06-222-3/+18
| | | | | | Check both in configure.ac (after checking if we need to rebuild the marshalling files) and in the python script using pyparsing (for people modifying .proto files in tarballs)
* sndworker: check the caps before sending volume messagesMarc-André Lureau2011-06-221-10/+46
|
* sndworker: add AudioVolume/AudioMute messagesMarc-André Lureau2011-06-225-4/+202
| | | | | | | | | | | | | | | | | | | These messages allow the guest to send the audio device volume to the client. It uses an arbitrary scale of 16bits, which works good enough for now. Save VolumeState in {Playback,Record}State, so that we can send the current volume on channel connection. Note about future improvements: - add exact dB support - add client to guest volume change Updated since v2: - bumped record and playback interface minor version to allow conditional compilation Updated since v1: - sync record volume on connection too
* python: remove c-ism trailing ;Marc-André Lureau2011-06-215-25/+25
|
* codegen: typedef the protocol enumsMarc-André Lureau2011-06-211-4/+12
| | | | | | | Commit 9d5ef9beeca722b2ceff7d15aaa3aaaaf07ecfbf in spice-protocol introduced a typedef manually in the generated enums.h header. This patch adds them automatically to all enums during enums.h generation.
* client: fix for redundant shift+f11 RHBZ #674532Yonit Halperin2011-06-143-10/+27
| | | | | | | | | | | After shift+F11, both in Windows 7 and xp clients, WM_KEYUP events were missing for SHIFT and F11. For F11 it was less important since unpress_all was preformed for all keys. However, we perform sync for all the keyboard modifiers and the GetKeyboardState returns "down" for shift. In windows7 client, we sometimes received afterwards a F11 KEYDOWN event repetition, and this caused SHIFT+F11 to be called again. Not performing hiding of the windows while changing client resolutions, solved the problem of missing events, and I don't see any difference in how spice looks while toggling to full screen. Using GetAsyncKeyState, returns "UP" for shift in windows 7, and helps avoid performing shift+f11 again, if there is an F11 repetition before we receive the KEYUP event for shift.
* client/windows: enable image randomization (ASLR) rhbz#701111Arnon Gilboa2011-05-301-2/+2
| | | | | | Enable image randomized base address, hindering some types of security attacks by making it more difficult for an attacker to predict target addresses.
* client/windows: remove slash from x64 build dirArnon Gilboa2011-05-301-4/+4
| | | | otherwise x64 is built in root if REDC_BUILD_DIR is not defined
* server/smartcard: fix use after freeAlon Levy2011-05-241-1/+1
|
* client/windows: remove precompiled header for common.h (fix broken windows ↵Arnon Gilboa2011-05-221-4/+4
| | | | | | | | debug build) -Release currently doesn't use precompiled headers at all -Debug is broken since common/*.c files don't include common.h -PCH can be enabled for all but specifically-chosen c-files
* common: add WIN64 ifdef for spice_bit_find_msb (fix broken windows x64 build)Arnon Gilboa2011-05-221-2/+2
| | | | | inline __asm is not supported in x64, so use the naive implementation until x64 asm implemented.
* client: fix flipped video in Linux guest on windows client, RHBZ #667689Yonit Halperin2011-05-191-3/+5
| | | | | | Video streams from Linux guests are oriented top-down, where gdi_canvas_put_image always received display context for down-top oriented bitmap. I fixed create_bitmap to consider the stream orientation.
* common: use INLINE instead of inlineArnon Gilboa2011-05-127-38/+20
| | | | | | needed for spice/common files used by the client, server & qxl driver. in windows _inline works for both c/c++, while inline is c++ only. compiling the client with mixed c/c++ code required this define.
* client/windows: undef SIZE_MAX in stdint.hArnon Gilboa2011-05-121-1/+1
| | | | eliminating redefinition warning
* common: fix ssl_verify windows build errorsArnon Gilboa2011-05-122-2/+8
|
* client/windows: add common\ssl_verify.c/h to projectArnon Gilboa2011-05-121-2/+10
| | | | disable WarnAsError, due to c/c++ warnings
* client/windows: inc version to 0,9,0,0Arnon Gilboa2011-05-121-4/+4
|
* client/windows: init PACKAGE_VERSIONArnon Gilboa2011-05-122-3/+5
| | | | | in windows, we set PACKAGE_VERSION to the binary version since we don't have config.h as generated by linux configure
* client: fix return code when missing hostAlon Levy2011-05-121-0/+1
|
* server/red_worker: stat_now cleanupAlon Levy2011-05-091-1/+2
| | | | static, remove inline, add (void), and empty line after function variables.
* server/red_worker: fix typo (lats_send_time)Alon Levy2011-05-091-4/+4
|
* server/tests: show port to connect toAlon Levy2011-05-091-1/+3
|
* server: bump SPICE_SERVER_VERSION to 0.9.0Marc-André Lureau2011-05-081-1/+1
| | | | SASL functions will be released with 0.9.0
* server: Unset executable bit of red_tunnel_worker.hZeeshan Ali (Khattak)2011-05-051-0/+0
|
* build: fix gettimeofday warningMarc-André Lureau2011-05-031-0/+1
| | | | | | CC test_playback.o test_playback.c: In function ‘playback_timer_cb’: test_playback.c:56:5: warning: implicit declaration of function ‘gettimeofday’
* client: make use of ssl_verify.cMarc-André Lureau2011-05-034-385/+36
| | | | | | | Fixed since v1: - don't include C code, rather use the common lib - add missing spice_openssl_verify_free() call - keep the extra-parsing of subject for error reporting
* common: add ssl_verify.c common codeMarc-André Lureau2011-05-033-0/+543
| | | | | | | | | | | | | | | | | | Code adapter from RedPeer::ssl_verify_callback() and used by spice-gtk. Since v1: - fixed Makefile.am - added config.h include - autoconf alloca added in patch series - moved int escape inside for loop - added a failed case when missing assignment - replaced strlen () by -1 - skip spaces after comma - c++ guards I didn't use bool, because openSSL uses int, and it is more future proof for error reporting.
* common: add ring_get_length() for debugging purposesMarc-André Lureau2011-05-031-0/+13
| | | | | Please notice it has a "static" modifier, like the rest of the inlined functions in ring.h, so it won't warn if it isn't used.
* common: mem.h add alloca definitionMarc-André Lureau2011-05-032-0/+31
| | | | | We don't support the autoconf ALLOCA/C_ALLOC fallback. If one day, someone cares for a weird platform, he can fix it.
* common: spice_memdup could accept NULLMarc-André Lureau2011-05-031-0/+4
| | | | (this patch is not to solve a crash fix, but to align with glib API)
* common: add windows.h where required, make gdi_handlers staticMarc-André Lureau2011-05-034-2/+8
| | | | | This patch has not been verified with VS/brew. It should be safe hopefully. Compilation is fine with mingw32/spice-gtk.
* build: require c99Marc-André Lureau2011-05-032-1/+4
| | | | Because we use c99: stdbool.h, inttypes.h, bool, variadic macros, // comments, ...
* move get_time_stamp to main_channel.cChristophe Fergeau2011-05-032-7/+7
| | | | | There is only one user of get_time_stamp from spice_common.h so it's not really useful to keep it there.
* use standard do { } while (0) for spice_common.h macrosChristophe Fergeau2011-05-031-11/+11
|
* move WARN and WARN_ONCE to spice_common.hChristophe Fergeau2011-05-034-15/+7
|
* common: use PANIC from spice_common.hChristophe Fergeau2011-05-033-20/+0
|
* common,server: use ASSERT from spice_common.hChristophe Fergeau2011-05-039-36/+7
| | | | | | | spice_common.h provides an ASSERT macro, no need to duplicate it in many places. For now client/debug.h keeps its own copy since debug.h and spice_common.h have clashes on other macros which are trickier to unify.
* server: use gcc builtin for atomic get/set bitChristophe Fergeau2011-05-031-2/+4
|
generated by cgit (git 2.34.1) at 2024-06-10 11:56:15 +0000