Re: [PATCH] Add support for SPICE SASL

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

 



On Thu, Oct 17, 2013 at 02:46:46PM +0100, Daniel P. Berrange wrote:
> On Thu, Oct 17, 2013 at 03:38:04PM +0200, Christophe Fergeau wrote:
> > This adds a spice_sasl boolean option to qemu.conf to mimic what is
> > donc for VNC SASL support.
> 
> This is a dup of what I've already posted, but not yet pushed
> 
> https://www.redhat.com/archives/libvir-list/2013-September/msg00319.html
> 
> Now I've remembered, I'll push it :-)

Ah indeed, ACK to your patch fwiw. Feel free to squash the additions I made
to tests/ in your patch.

> 
> > I did not add a spice_sasl_dir option as the corresponding
> > vnc_sasl_dir option sets a SASL_CONF_DIR environment variable, but
> > I could not find any reference to that variable in either QEMU or
> > cyrus-sasl, and Google was not helpful either.
> 
> It is in the 'sasl_getconfpath_t' man page
> 
>   DESCRIPTION
>        sasl_getconfpath_t is used if the application wishes to use a  different
>        location  for the SASL configuration files. If this callback is not used
>        SASL  will  either  use  the  location  in  the   environment   variable
>        SASL_CONF_PATH  (provided  we  are  not  SUID  or SGID) or /etc/sasl2 by
>        default.
> 
> except we typoed  s/PATH/DIR/ so I'll fix that :-)

Ah, I indeed have more hits with that name! :)

Christophe

Attachment: pgpGT18M5aJ4j.pgp
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]