Perhaps a commit message would answer my question below... On 11/22/2016 06:09 AM, Nikolay Shirokovskiy wrote: > --- > src/conf/domain_conf.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c > index 6e008e2..fb6ff0b 100644 > --- a/src/conf/domain_conf.c > +++ b/src/conf/domain_conf.c > @@ -22534,7 +22534,10 @@ virDomainGraphicsDefFormat(virBufferPtr buf, > virBufferAsprintf(buf, " autoport='%s'", > def->data.vnc.autoport ? "yes" : "no"); > > - if (def->data.vnc.websocket) > + if (def->data.vnc.websocketGenerated && Wouldn't websocketGenerated imply an active domain? And a change of the websocket in the active xml to be some value? So is the purpose of this because if you have an active domain you don't want to display the websocket that was generated? And why is that? IOW: What are you trying to ensure with this patch? John > + (flags & VIR_DOMAIN_DEF_FORMAT_INACTIVE)) > + virBufferAddLit(buf, " websocket='-1'"); > + else if (def->data.vnc.websocket) > virBufferAsprintf(buf, " websocket='%d'", def->data.vnc.websocket); > > virDomainGraphicsListenDefFormatAddr(buf, glisten, flags); > -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
