summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Release 0.7.30.7.3Hans de Goede2011-02-113-2/+10
|
* Drop static_title.bmp from windows/Makefile.amHans de Goede2011-02-111-1/+0
|
* update required minimal libcacard to 0.1.2Alon Levy2011-02-101-1/+1
|
* client/smartcard: libcacard dropped ReaderAddResponseAlon Levy2011-02-102-18/+78
| | | | | | | | | | | | uses VSC_Error with code==VSC_SUCCESS instead. This means that the VSC_Error message is overloaded. Instead of the other option of adding a message id, since the connection is TCP so no messages may be dropped or reordered, by having each message followed by a response there is no ambiguity. Still this commit adds a queue for messages that we only have one of which outstanding at a time, i.e. send, wait for response, send the next, etc. This further simplifies the logic, while not adding much overhead since only when spicec starts up it has a situation where it needs to send two events (ReaderAdd and ATR for Card Insert).
* server/smartcard: don't push our own error on reader addAlon Levy2011-02-101-3/+3
| | | | | | | | | | | | | | The device already sends one. There are actually two connections going on: server to client - this is the smartcard channel, it reuses the VSC protocol. server to device - this is an internal connection using VSC too. We generally just passthrough all messages from the client to the device, and from the device to the client. We only rewrite the reader_id because the device knows about a single id (it is actually a card id), and we may manage more then one in the future. Bottom line is that there was an extra VSC_Error message reaching the client.
* client/smartcard: ignore VSC_InitAlon Levy2011-02-101-0/+2
|
* server/smartcard: ignore VSC_Init from clientAlon Levy2011-02-101-0/+3
|
* server/smartcard: print instead of assert on bad reader_id in ↵Alon Levy2011-02-101-1/+3
| | | | smartcard_char_device_on_message_from_device
* server/smartcard: libcacard uses network byte order, so we must tooAlon Levy2011-02-101-6/+19
|
* client/smartcard: s/reader_id_t/uint32_t/ (libcacard changed)Alon Levy2011-02-102-11/+11
|
* server/smartcard: libcacard removed ReaderAddResponseAlon Levy2011-02-101-42/+4
|
* server/smartcard: s/reader_id_t/uint32_t/ (libcacard changed)Alon Levy2011-02-101-6/+6
|
* server/red_worker: fix used but uninitialized warning (gcc 4.6.0)Alon Levy2011-02-071-1/+1
|
* spicec: Remove spice-client watermark (rhbz#662450)Hans de Goede2011-02-029-1060/+2
| | | | | | | | | This patch stops us from drawing the spice client watermark at the top of the virtual machine view. We have had requests through several channels to remove this as it has little added value, and is seen as annoying by some. Given that we now also have a bugzilla for this I think it is time we really remove it. (cherry picked from commit 392ed65dda1a28c85f04ebb09924c6ce83dbdf2b)
* spice-client migration: fix minor for old migration support.Uri Lublin2011-01-272-5/+5
| | | | | | | For not too old spice-migration, minor is 1. For older (ancient) spice-migration, minor is 0. Affects only VM migration while a spice client is connected.
* client/windows: don't allocate console unless requiredAlon Levy2011-01-271-9/+27
|
* client: fix broken vs2008 buildAlon Levy2011-01-272-1/+14
|
* client: --help should not need platform initializationAlon Levy2011-01-272-33/+62
| | | | | | separate initialization into before command line parsing and after, call later only if command line parsing succeeds (in particular, it "fails" if --help is given).
* demarshaller/marshaller fix gcc 4.6.0Alon Levy2011-01-272-9/+21
| | | | | | | | | | python_modules/demarshal.py and marshal.py fixes for gcc 4.6.0 warning about set but unused variables. The fixes disable creating of variables mem_size when they are not used (demarshall) and declaring a src variable when the message doesn't use it (marshal). You need to touch *.proto after applying this (should add a Makefile dependency).
* codegen: avoid creating out if not used (fix gcc 4.6.0 warning)Alon Levy2011-01-271-3/+5
|
* client: gcc 4.6.0: two more unused variable fixesAlon Levy2011-01-272-3/+0
|
* client/cegui: cegui 0.6.0 gcc 4.6.0 related fixAlon Levy2011-01-273-0/+8
| | | | | cegui doesn't include stddef required for ptrdiff_t type, we include it for it.
* client/glz_decoder.cpp: gcc 4.6.0 unused fixesAlon Levy2011-01-271-5/+0
|
* client/display_channel: gcc 4.6.0 unused fixesAlon Levy2011-01-271-6/+12
|
* common/sw_canvas: remove unused error valAlon Levy2011-01-271-2/+1
| | | | | | | | | | | This is the only unused var change I'll want to revisit eventually, I'm submitting anyway since it doesn't change current behavior. I'm talking about ignoring the return value from canvas creation. Adding a print is possible but I didn't test (may be too verbose, also preferable to be a debug print if so, and we don't have that option in the code atm - probably an environment variable will do, or adding some spice_server_set_logging_level api, maybe even spice_server_set_logging_fd?)
* common/canvas_base.c: remove unused variablesAlon Levy2011-01-271-12/+0
|
* client/server: warning fixes (gcc 4.6.0)Alon Levy2011-01-275-21/+11
| | | | | gcc 4.6.0 added "[-Werror=unused-but-set-variable]", this and the next few fixes tend to that. Mostly harmless.
* client/server: add missing USE_TUNNELAlon Levy2011-01-252-0/+6
| | | | | disable some code that only makes sense when USE_TUNNEL is defined in client and server channel security level setting.
* client/server: add missing smartchannel channel security handlingAlon Levy2011-01-252-0/+9
| | | | | The name to channel id mapping for the smartcard channel is missing, add it in client and server.
* Update license header for server/red_parse_qxl.cHans de Goede2011-01-211-6/+6
| | | | This one mistakenly had a GPL header rather then an LGPL header.
* Drop unnecessary X11 and alsa requires from spice-server.pcHans de Goede2011-01-211-3/+0
|
* Release 0.7.20.7.2Hans de Goede2011-01-193-2/+11
|
* server: remove dep on CEGUIHans de Goede2011-01-191-2/+0
|
* client: CEGUI: search for CEGUI-0.6 to work in F15 (rawhide)Alon Levy2011-01-192-5/+16
|
* client: log subject-host mismatch, and raise ssl warnings to errorsAlon Levy2011-01-171-5/+5
|
* client inputs: stop blinking keyboard when out of focusMarc-André Lureau2011-01-172-1/+15
| | | | | | | | | | We could introduce another boolean to prevent changes, or just reuse _active_modifiers_event = true to prevent further update. Additionaly this patch restore the keyboard state when focusing out, which is fine when dealing with full remote desktop, but should be reconsidered if/when SPICE supports remote windows managed by client window manager for instance, imho.
* common, canvas_get_jpeg_alpha: let top_down be any valueAlon Levy2011-01-171-1/+1
| | | | | | | Allow top_down flag to have any value, only ASSERT it is positive when needs to be positive and zero otherwise. Allows older server bug of sending 4 instead of 1 in top down flag to not affect newer clients (previous patch fixes server).
* server/red_worker: use 1, not 4 when lz_encoding a top down imageAlon Levy2011-01-171-2/+3
|
* server: remove dep on libcacardAlon Levy2011-01-171-1/+0
| | | | | | Only the client needs this, and erronously using SPICE_REQUIRES results in libcacard dep in spice-server.pc, which is then dragged into qemu when linking, beeing used instead of the builtin libcacard.
* client/cmd_line_parser: fix wrong reporting of bad argument in --bla=val caseAlon Levy2011-01-171-2/+5
| | | | | | | | | | | We use get_opt_long, which allows non ambiguous abbreviations, but since we didn't like that we have code that checks for abbreviations and issues an error. But that code only handled separate argument and key like: --bla value and didn't handle them in the same arguemnts, like: --bla=value This patch fixes that, and gives a slightly better error report (it still contains the =value part though)
* configure.ac: use AC_LANG_SOURCE in AC_COMPILE_IFELSE, silence remaining ↵Alon Levy2011-01-171-3/+3
| | | | warnings
* Release 0.7.10.7.1Hans de Goede2010-12-173-2/+9
|
* Makefiles: fix server/tests/test_util.h not being included in make distHans de Goede2010-12-171-2/+2
|
* server: Update SPICE_SERVER_VERSIONHans de Goede2010-12-171-1/+1
|
* Update NEWSHans de Goede2010-12-171-1/+0
| | | | | The NEWS file wrongly mentioned mingw32 support, but this is only present in master, not in the 0.8 branch.
* Release 0.7.00.7.0Hans de Goede2010-12-162-3/+9
|
* spicec-x11: Let the window manager place our window the 1st time (rhbz#662407)Hans de Goede2010-12-162-1/+8
| | | | | | | | | | | | | | | | | | | | | | | The problem is that RedWindow::show calls the XLib MoveWindow function on the window after it has been mapped, moving it to the location in _show_pos. This is seen by the window manager as the application saying I know exactly where I want my window to be placed, don't do placing for me. Which causes the client window to always be shown at pos 0x0, even though that may not be the best location. What this patch does is: 1) It makes RedWindow::show not call MoveWindow when a window is first created normally and then shown 2) It makes RedWindow::show still call MoveWindow when: -when the window has been shown before, and was hidden for some reason (controller interface), and is now being re-shown so that it ends up being re-shown at its old position -when the window is a fullscreen window (screen.cpp always calls move on the window before showing it to set its position) -when the user switch from windowed mode -> fullscreen -> windowed mode again, to make sure that the windowed mode window is shown in the same position as before switching to fullscreen mode
* spicec-x11: Add a class hint to our window managet hintsHans de Goede2010-12-161-1/+10
| | | | | | This helps people who want to tell their windowmanager to do something special with spicec, like make it sticky, or whatever, see: https://bugzilla.redhat.com/show_bug.cgi?id=662452#c4
* spicec: Add a --title cmdline option (rhbz#662452)Hans de Goede2010-12-161-0/+6
|
* server/red_worker: fix worker->drawable_countAlon Levy2010-12-161-1/+1
| | | | | | | | | | | | | drawable_count was becoming negative. It tracks the number of items in the worker->current_list ring. It was decremented correctly, but incremented only in several cases. The cases it wasn't incremented where: red_current_add_equal found an equivalent drawable by moving the increment to where the item is added to current_list, in __current_add_drawable, the accounting remains correct. This has no affect other then correct accounting, as drawable_count isn't used for anything.