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

Re: [libvirt] [PATCH v2 08/12] spice: add support for listen type socket



Hey,

On Wed, May 11, 2016 at 05:08:27PM +0200, Pavel Hrdina wrote:
> @@ -7665,40 +7694,12 @@ qemuBuildGraphicsSPICECommandLine(virQEMUDriverConfigPtr cfg,
>              break;
>  
>          case VIR_DOMAIN_GRAPHICS_SPICE_CHANNEL_MODE_INSECURE:
> -            if (glisten->port <= 0) {
> -                virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> -                               _("spice insecure channels set in XML "
> -                                 "configuration, but plain port is not provided"));
> -                goto error;
> -            }
>              virBufferAsprintf(&opt, "plaintext-channel=%s,",
>                                virDomainGraphicsSpiceChannelNameTypeToString(i));
>              break;
>  
>          case VIR_DOMAIN_GRAPHICS_SPICE_CHANNEL_MODE_ANY:
> -            switch (defaultMode) {
> -            case VIR_DOMAIN_GRAPHICS_SPICE_CHANNEL_MODE_SECURE:
> -                if (glisten->tlsPort <= 0) {
> -                    virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> -                                   _("spice defaultMode secure requested in XML "
> -                                     "configuration but TLS port not provided"));
> -                    goto error;
> -                }
> -                break;
> -
> -            case VIR_DOMAIN_GRAPHICS_SPICE_CHANNEL_MODE_INSECURE:
> -                if (glisten->port <= 0) {
> -                    virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> -                                   _("spice defaultMode insecure requested in XML "
> -                                     "configuration but plain port not provided"));
> -                    goto error;
> -                }
> -                break;
> -
> -            case VIR_DOMAIN_GRAPHICS_SPICE_CHANNEL_MODE_ANY:
> -                /* don't care */
>              break;
> -            }
>          }
>      }

Didn't we lose the error when asking for plaintext channels without
setting a non-secure port?

Christophe

Attachment: signature.asc
Description: PGP signature


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