[libvirt] [PATCH] daemon: fix logic bug with virAsprintf

Eric Blake eblake at redhat.com
Thu Feb 16 00:14:44 UTC 2012


On 02/15/2012 12:21 AM, Wen Congyang wrote:
> At 02/15/2012 07:03 AM, Eric Blake Wrote:
>> Regression introduced in commit 7033c5f2, due to improper conversion
>> from snprintf to virAsprintf.
>>
>> * daemon/remote.c (remoteDispatchAuthList): Check return value
>> correctly.
>> ---
>>
>> This one's embarrassing.  I think I broke polkit authorization in
>> 0.9.10.  :(
>>
>>  daemon/remote.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/daemon/remote.c b/daemon/remote.c
>> index 9c61306..724db23 100644
>> --- a/daemon/remote.c
>> +++ b/daemon/remote.c
>> @@ -2052,7 +2052,7 @@ remoteDispatchAuthList(virNetServerPtr server ATTRIBUTE_UNUSED,
>>          } else if (callerUid == 0) {
>>              char *ident;
>>              if (virAsprintf(&ident, "pid:%lld,uid:%d",
>> -                            (long long) callerPid, callerUid) == 0) {
>> +                            (long long) callerPid, callerUid) >= 0) {
>>                  VIR_INFO("Bypass polkit auth for privileged client %s",
>>                           ident);
>>                  if (virNetServerClientSetIdentity(client, ident) < 0)
> 
> ACK

Thanks; pushed.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120215/77e56bd8/attachment-0001.sig>


More information about the libvir-list mailing list