Re: [PATCH spice-server] reds: Check link header magic without waiting all header

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On 01/26/2017 06:20 PM, Uri Lublin wrote:
> > Hi Frediano,
> >
> > I'd replace "all" in subject with "for the whole"
> >
> > On 01/26/2017 05:54 PM, Frediano Ziglio wrote:
> >> This allows the connection to early fail in case initial bytes
> >> are not correct.
> >> This allows for instance VNC client to graceful fail connecting
> >> to a spice-server. This happens easily as the two protocols
> >> share the same range of ports.
> >>
> >> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> >> Tested-by: Daniel P. Berrange <berrange@xxxxxxxxxx>
> >
> 
> I wrote the following python script to test it, till vnc
> client is ready:
> (And added some debug messages in spice-server)
> 
> ---
> import socket
> s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
> s.connect(('localhost', 5924))
> s.send("RFB ")
> magic = s.recv(4)
> print 'magic is ', magic
> ---
> 
> Uri.

Seems to work. I mean, the program get not stuck which
was the intention of the patch.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]