Re: [libosinfo 1/3] Every driver file in DB must provide MD5 checksum

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

 



On Wed, Dec 12, 2012 at 11:25 AM, Christophe Fergeau
<cfergeau@xxxxxxxxxx> wrote:
> On Wed, Dec 12, 2012 at 03:21:28AM +0200, Zeeshan Ali (Khattak) wrote:
>> From: "Zeeshan Ali (Khattak)" <zeeshanak@xxxxxxxxx>
>>
>> ---
>>  data/schemas/libosinfo.rng            |  9 +++++++++
>>  osinfo/osinfo_device_driver.c         | 18 ++++++++++++++++++
>>  osinfo/osinfo_device_driver_private.h |  3 +++
>>  osinfo/osinfo_loader.c                | 11 ++++++++---
>>  4 files changed, 38 insertions(+), 3 deletions(-)
>>
>> diff --git a/data/schemas/libosinfo.rng b/data/schemas/libosinfo.rng
>> index 51b0c20..819790a 100644
>> --- a/data/schemas/libosinfo.rng
>> +++ b/data/schemas/libosinfo.rng
>> @@ -415,6 +415,9 @@
>>        </optional>
>>        <zeroOrMore>
>>          <element name='file'>
>> +          <attribute name="md5">
>> +            <ref name='md5'/>
>> +          </attribute>
>>            <text/>
>>          </element>
>>        </zeroOrMore>
>> @@ -636,4 +639,10 @@
>>        <param name="pattern">dos|unix</param>
>>      </data>
>>    </define>
>> +
>> +  <define name='md5'>
>> +    <data type="string">
>> +      <param name="pattern">[0-9a-fA-F]{32}</param>
>> +    </data>
>> +  </define>
>
> I'd go with SHA256 from the start

Why? Do we expect these files to be huge? MD5 should suffice.

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124

_______________________________________________
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