On Wed, Dec 12, 2012 at 04:06:10PM +0200, Zeeshan Ali (Khattak) wrote: > On Wed, Dec 12, 2012 at 11:25 AM, Christophe Fergeau > <[email protected]> wrote: > > On Wed, Dec 12, 2012 at 03:21:28AM +0200, Zeeshan Ali (Khattak) wrote: > >> From: "Zeeshan Ali (Khattak)" <[email protected]> > >> > >> --- > >> 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.
As you said in one of your other email, we don't really know what creative use applications would make of these checksums, md5 is old, unsecure, ... so I'm not sure it's a good idea to use this in new code. Christophe
pgp7miu0c5787.pgp
Description: PGP signature
_______________________________________________ virt-tools-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/virt-tools-list
