Re: [virt-manager PATCH] Fix UUID generation according to RFC 4122

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

 



On 08/16/2013 05:50 PM, Cole Robinson wrote:
> On 08/16/2013 08:53 AM, Martin Kletzander wrote:
>> In commit 74aa8c8a, I somehow made an off-by-one error even though
>> that code was almost copy-paste from libvirt's commit 396c4d34.
>> ---
>>  virtinst/util.py | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/virtinst/util.py b/virtinst/util.py
>> index f1bc5c3..0ddf051 100644
>> --- a/virtinst/util.py
>> +++ b/virtinst/util.py
>> @@ -378,8 +378,8 @@ def randomUUID(conn):
>>          return "00000000-1111-2222-3333-444444444444"
>>
>>      u = [random.randint(0, 255) for ignore in range(0, 16)]
>> -    u[7] = (u[7] & 0x0F) | (4 << 4)
>> -    u[9] = (u[9] & 0x3F) | (2 << 6)
>> +    u[6] = (u[6] & 0x0F) | (4 << 4)
>> +    u[8] = (u[8] & 0x3F) | (2 << 6)
>>      return "-".join(["%02x" * 4, "%02x" * 2, "%02x" * 2, "%02x" * 2,
>>                       "%02x" * 6]) % tuple(u)
>>
> 
> ACK
> 

Thanks both, pushed now.

Maritn

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux