[Spice-devel] [PATCH 4/6] Add APIs for injecting a client connection socket

2011-12-12 Thread Daniel P. Berrange
From: "Daniel P. Berrange" Allow applications to pass a pre-accepted client socket file descriptor in. The new APIs are spice_server_add_ssl_client and spice_server_add_client * server/reds.c: Implement new APIs * server/spice.h: Define new APIs Signed-off-by: Daniel P. Berrange --- server/re

Re: [Spice-devel] [PATCH 4/6] Add APIs for injecting a client connection socket

2011-10-28 Thread Gerd Hoffmann
Hi, >SPICE_SERVER_0.10.0.a (a, b, c, etc) Looks good to me. >SPICE_SERVER_0.10.0.p1 (patch level 1, 2, 3, etc) Would work too. I'd prefer 'a' but its a matter of taste after all ;) >SPICE_SERVER_0.10.1(go straight for 0.10.1 release version) Don't think we'll go straight t

Re: [Spice-devel] [PATCH 4/6] Add APIs for injecting a client connection socket

2011-10-28 Thread Daniel P. Berrange
On Fri, Oct 28, 2011 at 03:03:57PM +0200, Gerd Hoffmann wrote: > Hi, > > > diff --git a/server/spice-server.syms b/server/spice-server.syms > > index f1374bd..c34ff86 100644 > > --- a/server/spice-server.syms > > +++ b/server/spice-server.syms > > @@ -89,5 +89,7 @@ global: > > spice_server_

Re: [Spice-devel] [PATCH 4/6] Add APIs for injecting a client connection socket

2011-10-28 Thread Gerd Hoffmann
Hi, > diff --git a/server/spice-server.syms b/server/spice-server.syms > index f1374bd..c34ff86 100644 > --- a/server/spice-server.syms > +++ b/server/spice-server.syms > @@ -89,5 +89,7 @@ global: > spice_server_record_set_mute; > spice_server_record_set_volume; > spice_server_get

[Spice-devel] [PATCH 4/6] Add APIs for injecting a client connection socket

2011-10-28 Thread Daniel P. Berrange
From: "Daniel P. Berrange" Allow applications to pass a pre-accepted client socket file descriptor in. The new APIs are spice_server_add_ssl_client and spice_server_add_client * server/reds.c: Implement new APIs * server/spice.h: Define new APIs Signed-off-by: Daniel P. Berrange --- server/re

Re: [Spice-devel] [PATCH 4/6] Add APIs for injecting a client connection socket

2011-10-26 Thread Marc-André Lureau
Hi To avoid duplication, it would be nice to make internal code use spice_server_add_client(), in reds_accept() for example. On Fri, Oct 21, 2011 at 5:11 PM, Daniel P. Berrange wrote: > + > +int spice_server_add_client(SpiceServer *s, int socket) Missing SPICE_GNUC_VISIBLE, coming in following

[Spice-devel] [PATCH 4/6] Add APIs for injecting a client connection socket

2011-10-21 Thread Daniel P. Berrange
From: "Daniel P. Berrange" Allow applications to pass a pre-accepted client socket file descriptor in. The new APIs are spice_server_add_ssl_client and spice_server_add_client * server/reds.c: Implement new APIs * server/spice.h: Define new APIs Signed-off-by: Daniel P. Berrange --- server/re