Re: VMware driver: SessionIsActive API / Sessions.ValidateSession permission

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

 



2016-02-16 16:13 GMT+01:00 Richard W.M. Jones <rjones@xxxxxxxxxx>:
> On Mon, Feb 15, 2016 at 11:22:27PM +0100, Matthias Bolte wrote:
>> 2016-02-11 12:59 GMT+01:00 Richard W.M. Jones <rjones@xxxxxxxxxx>:
>> > Is calling SessionIsActive necessary?  From my (very limited)
>> > understanding, it seems as if we might use 'SessionManager.
>> > currentSession' property instead, which doesn't require admin
>> > permissions.  Actually the code [see link above] already does this
>> > when ctx->hasSessionIsActive is false, but that doesn't apply to
>> > modern vCenter.
>>
>> SessionIsActive Is not necessary here, but it seemed to be the better
>> solution compared to this more hacky way of checking session object. I
>> wasn't aware that is would require admin level privileges.
>
> The patch works fine for me.  I believe the server I'm testing against
> is vCenter 5.5, although I'm not totally sure about that.
>
> ACK.
>
> Rich.

Thanks, pushed.

-- 
Matthias Bolte
http://photron.blogspot.com

--
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]