[libvirt] [PATCH] qemu: event: Fix retrieval of the "service" field from graphics events
Peter Krempa
pkrempa at redhat.com
Mon Jun 29 13:17:50 UTC 2015
On Mon, Jun 29, 2015 at 13:47:50 +0100, Daniel Berrange wrote:
> On Mon, Jun 29, 2015 at 02:42:50PM +0200, Peter Krempa wrote:
> > qemu's event has following format:
> >
> > {
> > "timestamp": {
> > "seconds": 1435580974,
> > "microseconds": 82226
> > },
> > "event": "SPICE_INITIALIZED",
> > "data": {
> > "server": {
> > "auth": "none",
> > "port": "5900",
> > "family": "ipv4",
> > "host": "127.0.0.1"
> > },
> > "client": {
> > "port": "53224",
> > "family": "ipv4",
> > "channel-type": 3,
> > "connection-id": 1113096064,
> > "host": "127.0.0.1",
> > "channel-id": 0,
> > "tls": false
> > }
> > }
> > }
> >
> > Our code tried to extract the "service" field but qemu reports it as
> > "port".
>
> Hmm, that's somewhat odd - did you check back historical versions of
> QEMU where this event was first introduced to see if it has always
> had this name ? It smells like the kind of thing that could have been
> a regression at some point
Actually the regression would be on the VNC side, since VNC uses
"service" and the function that I've modified handles both. I failed to
notice that fact at first.
I'll post a V2 with dual handling.
Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150629/0f6d19df/attachment-0001.sig>
More information about the libvir-list
mailing list