Replace local sha1 and md5 implementations with rpm's hash functionality
authorPanu Matilainen <pmatilai@laiskiainen.org>
Tue, 5 Apr 2011 08:43:40 +0000 (11:43 +0300)
committerPanu Matilainen <pmatilai@laiskiainen.org>
Tue, 5 Apr 2011 08:43:40 +0000 (11:43 +0300)
- implement an algorithm agnostic c++ interface to rpm's digest functions,
  based on the former md5/sha1 summation class interface
- minimally "port" md5 + sha1 summation classes on top of the new class
- the result is always used in string format, not bothering to wrap
  MD5/SHA1SumValue class fully, just typedef them as strings and fix
  users

13 files changed:
apt-pkg/Makefile.am
apt-pkg/Makefile.in
apt-pkg/acquire-item.cc
apt-pkg/contrib/cdromutl.cc
apt-pkg/contrib/md5.cc [deleted file]
apt-pkg/contrib/md5.h
apt-pkg/contrib/rhash.cc [new file with mode: 0644]
apt-pkg/contrib/rhash.h [new file with mode: 0644]
apt-pkg/contrib/sha1.cc [deleted file]
apt-pkg/contrib/sha1.h
apt-pkg/rpm/rpmhandler.cc
test/hash.cc
tools/cached_md5.cc

index 74abffe..188ba2b 100644 (file)
@@ -32,13 +32,13 @@ libapt_pkg_la_SOURCES = \
        contrib/fileutl.h \
        contrib/hashes.cc \
        contrib/hashes.h \
-       contrib/md5.cc \
        contrib/md5.h \
        contrib/mmap.cc \
        contrib/mmap.h \
        contrib/progress.cc \
        contrib/progress.h \
-       contrib/sha1.cc \
+       contrib/rhash.cc \
+       contrib/rhash.h \
        contrib/sha1.h \
        contrib/sptr.h \
        contrib/sqlite.h \
index 9d50a93..f2ff706 100644 (file)
@@ -87,19 +87,19 @@ libapt_pkg_la_DEPENDENCIES = $(am__DEPENDENCIES_1)
 am__dirstamp = $(am__leading_dot)dirstamp
 am_libapt_pkg_la_OBJECTS = contrib/cdromutl.lo contrib/cmndline.lo \
        contrib/configuration.lo contrib/crc-16.lo contrib/error.lo \
-       contrib/fileutl.lo contrib/hashes.lo contrib/md5.lo \
-       contrib/mmap.lo contrib/progress.lo contrib/sha1.lo \
-       contrib/sqlite.lo contrib/strutl.lo rpm/raptheader.lo \
-       rpm/repomd.lo rpm/aptcallback.lo rpm/rpmhandler.lo \
-       rpm/rpmindexfile.lo rpm/rpmlistparser.lo rpm/rpmpackagedata.lo \
-       rpm/rpmpm.lo rpm/rpmrecords.lo rpm/rpmsrcrecords.lo \
-       rpm/rpmsystem.lo rpm/rpmversion.lo rpm/xmlutil.lo acquire.lo \
-       acquire-item.lo acquire-method.lo acquire-worker.lo \
-       algorithms.lo cachefile.lo clean.lo depcache.lo indexfile.lo \
-       init.lo luaiface.lo orderlist.lo packagemanager.lo pkgcache.lo \
-       pkgcachegen.lo pkgrecords.lo pkgsystem.lo policy.lo \
-       repository.lo sourcelist.lo srcrecords.lo tagfile.lo \
-       version.lo versionmatch.lo
+       contrib/fileutl.lo contrib/hashes.lo contrib/mmap.lo \
+       contrib/progress.lo contrib/rhash.lo contrib/sqlite.lo \
+       contrib/strutl.lo rpm/raptheader.lo rpm/repomd.lo \
+       rpm/aptcallback.lo rpm/rpmhandler.lo rpm/rpmindexfile.lo \
+       rpm/rpmlistparser.lo rpm/rpmpackagedata.lo rpm/rpmpm.lo \
+       rpm/rpmrecords.lo rpm/rpmsrcrecords.lo rpm/rpmsystem.lo \
+       rpm/rpmversion.lo rpm/xmlutil.lo acquire.lo acquire-item.lo \
+       acquire-method.lo acquire-worker.lo algorithms.lo cachefile.lo \
+       clean.lo depcache.lo indexfile.lo init.lo luaiface.lo \
+       orderlist.lo packagemanager.lo pkgcache.lo pkgcachegen.lo \
+       pkgrecords.lo pkgsystem.lo policy.lo repository.lo \
+       sourcelist.lo srcrecords.lo tagfile.lo version.lo \
+       versionmatch.lo
 libapt_pkg_la_OBJECTS = $(am_libapt_pkg_la_OBJECTS)
 libapt_pkg_la_LINK = $(LIBTOOL) --tag=CXX $(AM_LIBTOOLFLAGS) \
        $(LIBTOOLFLAGS) --mode=link $(CXXLD) $(AM_CXXFLAGS) \
@@ -299,13 +299,13 @@ libapt_pkg_la_SOURCES = \
        contrib/fileutl.h \
        contrib/hashes.cc \
        contrib/hashes.h \
-       contrib/md5.cc \
        contrib/md5.h \
        contrib/mmap.cc \
        contrib/mmap.h \
        contrib/progress.cc \
        contrib/progress.h \
-       contrib/sha1.cc \
+       contrib/rhash.cc \
+       contrib/rhash.h \
        contrib/sha1.h \
        contrib/sptr.h \
        contrib/sqlite.h \
@@ -520,13 +520,11 @@ contrib/fileutl.lo: contrib/$(am__dirstamp) \
        contrib/$(DEPDIR)/$(am__dirstamp)
 contrib/hashes.lo: contrib/$(am__dirstamp) \
        contrib/$(DEPDIR)/$(am__dirstamp)
-contrib/md5.lo: contrib/$(am__dirstamp) \
-       contrib/$(DEPDIR)/$(am__dirstamp)
 contrib/mmap.lo: contrib/$(am__dirstamp) \
        contrib/$(DEPDIR)/$(am__dirstamp)
 contrib/progress.lo: contrib/$(am__dirstamp) \
        contrib/$(DEPDIR)/$(am__dirstamp)
-contrib/sha1.lo: contrib/$(am__dirstamp) \
+contrib/rhash.lo: contrib/$(am__dirstamp) \
        contrib/$(DEPDIR)/$(am__dirstamp)
 contrib/sqlite.lo: contrib/$(am__dirstamp) \
        contrib/$(DEPDIR)/$(am__dirstamp)
@@ -573,14 +571,12 @@ mostlyclean-compile:
        -rm -f contrib/fileutl.lo
        -rm -f contrib/hashes.$(OBJEXT)
        -rm -f contrib/hashes.lo
-       -rm -f contrib/md5.$(OBJEXT)
-       -rm -f contrib/md5.lo
        -rm -f contrib/mmap.$(OBJEXT)
        -rm -f contrib/mmap.lo
        -rm -f contrib/progress.$(OBJEXT)
        -rm -f contrib/progress.lo
-       -rm -f contrib/sha1.$(OBJEXT)
-       -rm -f contrib/sha1.lo
+       -rm -f contrib/rhash.$(OBJEXT)
+       -rm -f contrib/rhash.lo
        -rm -f contrib/sqlite.$(OBJEXT)
        -rm -f contrib/sqlite.lo
        -rm -f contrib/strutl.$(OBJEXT)
@@ -646,10 +642,9 @@ distclean-compile:
 @AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/error.Plo@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/fileutl.Plo@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/hashes.Plo@am__quote@
-@AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/md5.Plo@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/mmap.Plo@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/progress.Plo@am__quote@
-@AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/sha1.Plo@am__quote@
+@AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/rhash.Plo@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/sqlite.Plo@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@contrib/$(DEPDIR)/strutl.Plo@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@rpm/$(DEPDIR)/aptcallback.Plo@am__quote@
index fa4ffee..3e409ff 100644 (file)
@@ -70,7 +70,7 @@ bool VerifyChecksums(string File,unsigned long Size,string MD5, string method)
         FileFd F(File, FileFd::ReadOnly);
         
         md5sum.AddFD(F.Fd(), F.Size());
-        if (md5sum.Result().Value() != MD5)
+        if (md5sum.Result() != MD5)
         {
            if (_config->FindB("Acquire::Verbose", false) == true)
               cout << "MD5Sum of "<<File<<" did not match what's in the checksum list and was redownloaded."<<endl;
@@ -81,7 +81,7 @@ bool VerifyChecksums(string File,unsigned long Size,string MD5, string method)
         FileFd F(File, FileFd::ReadOnly);
         
         sha1sum.AddFD(F.Fd(), F.Size());
-        if (sha1sum.Result().Value() != MD5)
+        if (sha1sum.Result() != MD5)
         {
            if (_config->FindB("Acquire::Verbose", false) == true)
               cout << "SHASum of "<<File<<" did not match what's in the checksum list and was redownloaded."<<endl;
index 524e217..d279a86 100644 (file)
@@ -221,7 +221,7 @@ bool IdentCdrom(string CD,string &Res,unsigned int Version)
    else
       sprintf(S,"-%u.debug",Version);
    
-   Res = Hash.Result().Value() + S;
+   Res = Hash.Result() + S;
    return true;   
 }
                                                                        /*}}}*/
diff --git a/apt-pkg/contrib/md5.cc b/apt-pkg/contrib/md5.cc
deleted file mode 100644 (file)
index 81b6102..0000000
+++ /dev/null
@@ -1,354 +0,0 @@
-// -*- mode: c++; mode: fold -*-
-// Description                                                         /*{{{*/
-// $Id: md5.cc,v 1.1 2002/07/23 17:54:51 niemeyer Exp $
-/* ######################################################################
-   
-   MD5Sum - MD5 Message Digest Algorithm.
-
-   This code implements the MD5 message-digest algorithm. The algorithm is 
-   due to Ron Rivest.  This code was written by Colin Plumb in 1993, no 
-   copyright is claimed. This code is in the public domain; do with it what 
-   you wish.
-   Equivalent code is available from RSA Data Security, Inc. This code has 
-   been tested against that, and is equivalent, except that you don't need to 
-   include two pages of legalese with every copy.
-
-   To compute the message digest of a chunk of bytes, instantiate the class,
-   and repeatedly call one of the Add() members. When finished the Result 
-   method will return the Hash and finalize the value.
-   
-   Changed so as no longer to depend on Colin Plumb's `usual.h' header
-   definitions; now uses stuff from dpkg's config.h.
-    - Ian Jackson <ijackson@nyx.cs.du.edu>.
-   
-   Changed into a C++ interface and made work with APT's config.h.
-    - Jason Gunthorpe <jgg@gpu.srv.ualberta.ca>
-   
-   Still in the public domain.
-
-   The classes use arrays of char that are a specific size. We cast those
-   arrays to uint8_t's and go from there. This allows us to advoid using
-   the uncommon inttypes.h in a public header or internally newing memory.
-   In theory if C9x becomes nicely accepted
-   
-   ##################################################################### */
-                                                                       /*}}}*/
-// Include Files                                                       /*{{{*/
-#include <apt-pkg/md5.h>
-#include <apt-pkg/strutl.h>
-
-#include <string.h>
-#include <unistd.h>
-#include <netinet/in.h>                          // For htonl
-#include <inttypes.h>
-#include <config.h>
-#include <system.h>
-                                                                       /*}}}*/
-
-// byteSwap - Swap bytes in a buffer                                   /*{{{*/
-// ---------------------------------------------------------------------
-/* Swap n 32 bit longs in given buffer */
-#ifdef WORDS_BIGENDIAN
-static void byteSwap(uint32_t *buf, unsigned words)
-{
-   uint8_t *p = (uint8_t *)buf;
-   
-   do 
-   {
-      *buf++ = (uint32_t)((unsigned)p[3] << 8 | p[2]) << 16 |
-        ((unsigned)p[1] << 8 | p[0]);
-      p += 4;
-   } while (--words);
-}
-#else
-#define byteSwap(buf,words)
-#endif
-                                                                       /*}}}*/
-// MD5Transform - Alters an existing MD5 hash                          /*{{{*/
-// ---------------------------------------------------------------------
-/* The core of the MD5 algorithm, this alters an existing MD5 hash to
-   reflect the addition of 16 longwords of new data. Add blocks
-   the data and converts bytes into longwords for this routine. */
-
-// The four core functions - F1 is optimized somewhat
-// #define F1(x, y, z) (x & y | ~x & z)
-#define F1(x, y, z) (z ^ (x & (y ^ z)))
-#define F2(x, y, z) F1(z, x, y)
-#define F3(x, y, z) (x ^ y ^ z)
-#define F4(x, y, z) (y ^ (x | ~z))
-
-// This is the central step in the MD5 algorithm.
-#define MD5STEP(f,w,x,y,z,in,s) \
-        (w += f(x,y,z) + in, w = (w<<s | w>>(32-s)) + x)
-
-static void MD5Transform(uint32_t buf[4], uint32_t const in[16])
-{
-   register uint32_t a, b, c, d;
-   
-   a = buf[0];
-   b = buf[1];
-   c = buf[2];
-   d = buf[3];
-   
-   MD5STEP(F1, a, b, c, d, in[0] + 0xd76aa478, 7);
-   MD5STEP(F1, d, a, b, c, in[1] + 0xe8c7b756, 12);
-   MD5STEP(F1, c, d, a, b, in[2] + 0x242070db, 17);
-   MD5STEP(F1, b, c, d, a, in[3] + 0xc1bdceee, 22);
-   MD5STEP(F1, a, b, c, d, in[4] + 0xf57c0faf, 7);
-   MD5STEP(F1, d, a, b, c, in[5] + 0x4787c62a, 12);
-   MD5STEP(F1, c, d, a, b, in[6] + 0xa8304613, 17);
-   MD5STEP(F1, b, c, d, a, in[7] + 0xfd469501, 22);
-   MD5STEP(F1, a, b, c, d, in[8] + 0x698098d8, 7);
-   MD5STEP(F1, d, a, b, c, in[9] + 0x8b44f7af, 12);
-   MD5STEP(F1, c, d, a, b, in[10] + 0xffff5bb1, 17);
-   MD5STEP(F1, b, c, d, a, in[11] + 0x895cd7be, 22);
-   MD5STEP(F1, a, b, c, d, in[12] + 0x6b901122, 7);
-   MD5STEP(F1, d, a, b, c, in[13] + 0xfd987193, 12);
-   MD5STEP(F1, c, d, a, b, in[14] + 0xa679438e, 17);
-   MD5STEP(F1, b, c, d, a, in[15] + 0x49b40821, 22);
-
-   MD5STEP(F2, a, b, c, d, in[1] + 0xf61e2562, 5);
-   MD5STEP(F2, d, a, b, c, in[6] + 0xc040b340, 9);
-   MD5STEP(F2, c, d, a, b, in[11] + 0x265e5a51, 14);
-   MD5STEP(F2, b, c, d, a, in[0] + 0xe9b6c7aa, 20);
-   MD5STEP(F2, a, b, c, d, in[5] + 0xd62f105d, 5);
-   MD5STEP(F2, d, a, b, c, in[10] + 0x02441453, 9);
-   MD5STEP(F2, c, d, a, b, in[15] + 0xd8a1e681, 14);
-   MD5STEP(F2, b, c, d, a, in[4] + 0xe7d3fbc8, 20);
-   MD5STEP(F2, a, b, c, d, in[9] + 0x21e1cde6, 5);
-   MD5STEP(F2, d, a, b, c, in[14] + 0xc33707d6, 9);
-   MD5STEP(F2, c, d, a, b, in[3] + 0xf4d50d87, 14);
-   MD5STEP(F2, b, c, d, a, in[8] + 0x455a14ed, 20);
-   MD5STEP(F2, a, b, c, d, in[13] + 0xa9e3e905, 5);
-   MD5STEP(F2, d, a, b, c, in[2] + 0xfcefa3f8, 9);
-   MD5STEP(F2, c, d, a, b, in[7] + 0x676f02d9, 14);
-   MD5STEP(F2, b, c, d, a, in[12] + 0x8d2a4c8a, 20);
-   
-   MD5STEP(F3, a, b, c, d, in[5] + 0xfffa3942, 4);
-   MD5STEP(F3, d, a, b, c, in[8] + 0x8771f681, 11);
-   MD5STEP(F3, c, d, a, b, in[11] + 0x6d9d6122, 16);
-   MD5STEP(F3, b, c, d, a, in[14] + 0xfde5380c, 23);
-   MD5STEP(F3, a, b, c, d, in[1] + 0xa4beea44, 4);
-   MD5STEP(F3, d, a, b, c, in[4] + 0x4bdecfa9, 11);
-   MD5STEP(F3, c, d, a, b, in[7] + 0xf6bb4b60, 16);
-   MD5STEP(F3, b, c, d, a, in[10] + 0xbebfbc70, 23);
-   MD5STEP(F3, a, b, c, d, in[13] + 0x289b7ec6, 4);
-   MD5STEP(F3, d, a, b, c, in[0] + 0xeaa127fa, 11);
-   MD5STEP(F3, c, d, a, b, in[3] + 0xd4ef3085, 16);
-   MD5STEP(F3, b, c, d, a, in[6] + 0x04881d05, 23);
-   MD5STEP(F3, a, b, c, d, in[9] + 0xd9d4d039, 4);
-   MD5STEP(F3, d, a, b, c, in[12] + 0xe6db99e5, 11);
-   MD5STEP(F3, c, d, a, b, in[15] + 0x1fa27cf8, 16);
-   MD5STEP(F3, b, c, d, a, in[2] + 0xc4ac5665, 23);
-   
-   MD5STEP(F4, a, b, c, d, in[0] + 0xf4292244, 6);
-   MD5STEP(F4, d, a, b, c, in[7] + 0x432aff97, 10);
-   MD5STEP(F4, c, d, a, b, in[14] + 0xab9423a7, 15);
-   MD5STEP(F4, b, c, d, a, in[5] + 0xfc93a039, 21);
-   MD5STEP(F4, a, b, c, d, in[12] + 0x655b59c3, 6);
-   MD5STEP(F4, d, a, b, c, in[3] + 0x8f0ccc92, 10);
-   MD5STEP(F4, c, d, a, b, in[10] + 0xffeff47d, 15);
-   MD5STEP(F4, b, c, d, a, in[1] + 0x85845dd1, 21);
-   MD5STEP(F4, a, b, c, d, in[8] + 0x6fa87e4f, 6);
-   MD5STEP(F4, d, a, b, c, in[15] + 0xfe2ce6e0, 10);
-   MD5STEP(F4, c, d, a, b, in[6] + 0xa3014314, 15);
-   MD5STEP(F4, b, c, d, a, in[13] + 0x4e0811a1, 21);
-   MD5STEP(F4, a, b, c, d, in[4] + 0xf7537e82, 6);
-   MD5STEP(F4, d, a, b, c, in[11] + 0xbd3af235, 10);
-   MD5STEP(F4, c, d, a, b, in[2] + 0x2ad7d2bb, 15);
-   MD5STEP(F4, b, c, d, a, in[9] + 0xeb86d391, 21);
-   
-   buf[0] += a;
-   buf[1] += b;
-   buf[2] += c;
-   buf[3] += d;
-}
-                                                                       /*}}}*/
-// MD5SumValue::MD5SumValue - Constructs the summation from a string   /*{{{*/
-// ---------------------------------------------------------------------
-/* The string form of a MD5 is a 32 character hex number */
-MD5SumValue::MD5SumValue(string Str)
-{
-   memset(Sum,0,sizeof(Sum));
-   Set(Str);
-}
-                                                                       /*}}}*/
-// MD5SumValue::MD5SumValue - Default constructor                      /*{{{*/
-// ---------------------------------------------------------------------
-/* Sets the value to 0 */
-MD5SumValue::MD5SumValue()
-{
-   memset(Sum,0,sizeof(Sum));
-}
-                                                                       /*}}}*/
-// MD5SumValue::Set - Set the sum from a string                                /*{{{*/
-// ---------------------------------------------------------------------
-/* Converts the hex string into a set of chars */
-bool MD5SumValue::Set(string Str)
-{
-   return Hex2Num(Str,Sum,sizeof(Sum));
-}
-                                                                       /*}}}*/
-// MD5SumValue::Value - Convert the number into a string               /*{{{*/
-// ---------------------------------------------------------------------
-/* Converts the set of chars into a hex string in lower case */
-string MD5SumValue::Value() const
-{
-   char Conv[16] = {'0','1','2','3','4','5','6','7','8','9','a','b',
-                    'c','d','e','f'};
-   char Result[33];
-   Result[32] = 0;
-   
-   // Convert each char into two letters
-   int J = 0;
-   int I = 0;
-   for (; I != 32; J++, I += 2)
-   {
-      Result[I] = Conv[Sum[J] >> 4];
-      Result[I + 1] = Conv[Sum[J] & 0xF];
-   } 
-
-   return string(Result);
-}
-                                                                       /*}}}*/
-// MD5SumValue::operator == - Comparitor                               /*{{{*/
-// ---------------------------------------------------------------------
-/* Call memcmp on the buffer */
-bool MD5SumValue::operator ==(const MD5SumValue &rhs) const
-{
-   return memcmp(Sum,rhs.Sum,sizeof(Sum)) == 0;
-}
-                                                                       /*}}}*/
-// MD5Summation::MD5Summation - Initialize the summer                  /*{{{*/
-// ---------------------------------------------------------------------
-/* This assigns the deep magic initial values */
-MD5Summation::MD5Summation()
-{
-   uint32_t *buf = (uint32_t *)Buf;
-   uint32_t *bytes = (uint32_t *)Bytes;
-   
-   buf[0] = 0x67452301;
-   buf[1] = 0xefcdab89;
-   buf[2] = 0x98badcfe;
-   buf[3] = 0x10325476;
-   
-   bytes[0] = 0;
-   bytes[1] = 0;
-   Done = false;
-}
-                                                                       /*}}}*/
-// MD5Summation::Add - 'Add' a data set to the hash                    /*{{{*/
-// ---------------------------------------------------------------------
-/* */
-bool MD5Summation::Add(const unsigned char *data,unsigned long len)
-{
-   if (Done == true)
-      return false;
-
-   uint32_t *buf = (uint32_t *)Buf;
-   uint32_t *bytes = (uint32_t *)Bytes;
-   uint32_t *in = (uint32_t *)In;
-
-   // Update byte count and carry (this could be done with a long long?)
-   uint32_t t = bytes[0];
-   if ((bytes[0] = t + len) < t)
-      bytes[1]++;      
-
-   // Space available (at least 1)
-   t = 64 - (t & 0x3f);        
-   if (t > len) 
-   {
-      memcpy((unsigned char *)in + 64 - t,data,len);
-      return true;
-   }
-
-   // First chunk is an odd size
-   memcpy((unsigned char *)in + 64 - t,data,t);
-   byteSwap(in, 16);
-   MD5Transform(buf,in);
-   data += t;
-   len -= t;
-   
-   // Process data in 64-byte chunks
-   while (len >= 64)
-   {
-      memcpy(in,data,64);
-      byteSwap(in,16);
-      MD5Transform(buf,in);
-      data += 64;
-      len -= 64;
-   }
-
-   // Handle any remaining bytes of data.
-   memcpy(in,data,len);
-
-   return true;   
-}
-                                                                       /*}}}*/
-// MD5Summation::AddFD - Add the contents of a FD to the hash          /*{{{*/
-// ---------------------------------------------------------------------
-/* */
-bool MD5Summation::AddFD(int Fd,unsigned long Size)
-{
-   unsigned char Buf[64*64];
-   int Res = 0;
-   while (Size != 0)
-   {
-      Res = read(Fd,Buf,std::min(Size,(unsigned long)sizeof(Buf)));
-      if (Res < 0 || (unsigned)Res != std::min(Size,(unsigned long)sizeof(Buf)))
-        return false;
-      Size -= Res;
-      Add(Buf,Res);
-   }
-   return true;
-}
-                                                                       /*}}}*/
-// MD5Summation::Result - Returns the value of the sum                 /*{{{*/
-// ---------------------------------------------------------------------
-/* Because this must add in the last bytes of the series it prevents anyone
-   from calling add after. */
-MD5SumValue MD5Summation::Result()
-{
-   uint32_t *buf = (uint32_t *)Buf;
-   uint32_t *bytes = (uint32_t *)Bytes;
-   uint32_t *in = (uint32_t *)In;
-   
-   if (Done == false)
-   {
-      // Number of bytes in In
-      int count = bytes[0] & 0x3f;     
-      unsigned char *p = (unsigned char *)in + count;
-      
-      // Set the first char of padding to 0x80.  There is always room.
-      *p++ = 0x80;
-      
-      // Bytes of padding needed to make 56 bytes (-8..55)
-      count = 56 - 1 - count;
-      
-      // Padding forces an extra block 
-      if (count < 0) 
-      {
-        memset(p,0,count + 8);
-        byteSwap(in, 16);
-        MD5Transform(buf,in);
-        p = (unsigned char *)in;
-        count = 56;
-      }
-      
-      memset(p, 0, count);
-      byteSwap(in, 14);
-      
-      // Append length in bits and transform
-      in[14] = bytes[0] << 3;
-      in[15] = bytes[1] << 3 | bytes[0] >> 29;
-      MD5Transform(buf,in);   
-      byteSwap(buf,4);
-      Done = true;
-   }
-   
-   MD5SumValue V;
-   memcpy(V.Sum,buf,16);
-   return V;
-}
-                                                                       /*}}}*/
index 8f650b4..669ed31 100644 (file)
@@ -3,20 +3,7 @@
 // $Id: md5.h,v 1.1 2002/07/23 17:54:51 niemeyer Exp $
 /* ######################################################################
    
-   MD5SumValue - Storage for a MD5Sum
-   MD5Summation - MD5 Message Digest Algorithm.
-   
-   This is a C++ interface to a set of MD5Sum functions. The class can
-   store a MD5Sum in 16 bytes of memory.
-   
-   A MD5Sum is used to generate a (hopefully) unique 16 byte number for a
-   block of data. This can be used to gaurd against corruption of a file.
-   MD5 should not be used for tamper protection, use SHA or something more
-   secure.
-   
-   There are two classes because computing a MD5 is not a continual 
-   operation unless 64 byte blocks are used. Also the summation requires an
-   extra 18*4 bytes to operate.
+   Legacy wrapper around raptHash class.
    
    ##################################################################### */
                                                                        /*}}}*/
 
 #include <string>
 #include <cstring>
+#include <apt-pkg/rhash.h>
 
 using std::string;
 
-class MD5Summation;
+typedef string MD5SumValue;
 
-class MD5SumValue
+class MD5Summation : public raptHash
 {
-   friend class MD5Summation;
-   unsigned char Sum[4*4];
-   
    public:
-
-   // Accessors
-   bool operator ==(const MD5SumValue &rhs) const; 
-   string Value() const;
-   inline operator string() const {return Value();}
-   bool Set(string Str);
-
-   MD5SumValue(string Str);
-   MD5SumValue();
+   MD5Summation() : raptHash(PGPHASHALGO_MD5) {};
 };
 
-class MD5Summation
-{
-   unsigned char Buf[4*4] __attribute__((aligned(8)));
-   unsigned char Bytes[2*4] __attribute__((aligned(8)));
-   unsigned char In[16*4] __attribute__((aligned(8)));
-   bool Done;
-   
-   public:
-
-   bool Add(const unsigned char *Data,unsigned long Size);
-   inline bool Add(const char *Data) {return Add((unsigned char *)Data,strlen(Data));}
-   bool AddFD(int Fd,unsigned long Size);
-   inline bool Add(const unsigned char *Beg,const unsigned char *End) 
-                  {return Add(Beg,End-Beg);}
-   MD5SumValue Result();
-   
-   MD5Summation();
-};
 
 #endif
diff --git a/apt-pkg/contrib/rhash.cc b/apt-pkg/contrib/rhash.cc
new file mode 100644 (file)
index 0000000..045d6c3
--- /dev/null
@@ -0,0 +1,65 @@
+// Include Files
+#include <apt-pkg/rhash.h>
+#include <apt-pkg/strutl.h>
+
+#include <string.h>
+#include <unistd.h>
+#include <inttypes.h>
+#include <config.h>
+#include <system.h>
+
+raptHash::raptHash(pgpHashAlgo HashAlgo)
+{
+   HashCtx = rpmDigestInit(HashAlgo, RPMDIGEST_NONE);
+}
+
+// raptHash::Result - Return checksum value                        /*{{{*/
+// ---------------------------------------------------------------------
+/* Add() may not be called after this */
+string raptHash::Result()
+{
+   void *data = NULL;
+   size_t dlen = 0;
+   string Value = "";
+   
+   rpmDigestFinal(HashCtx, &data, &dlen, 1);
+   if (data) {
+      Value = string((const char*)data);
+   }
+   HashCtx = NULL;
+
+   return Value;
+}
+                                                                       /*}}}*/
+// raptHash::Add - Adds content of buffer into the checksum        /*{{{*/
+// ---------------------------------------------------------------------
+/* May not be called after Result() is called */
+bool raptHash::Add(const unsigned char *data,unsigned long len)
+{
+   int rc;
+   if (HashCtx == NULL)
+      return false;
+
+   rc = rpmDigestUpdate(HashCtx, data, len);
+   
+   return (rc == 0);
+}
+                                                                       /*}}}*/
+// raptHash::AddFD - Add content of file into the checksum         /*{{{*/
+// ---------------------------------------------------------------------
+/* */
+bool raptHash::AddFD(int Fd,unsigned long Size)
+{
+   unsigned char Buf[64 * 64];
+   int Res = 0;
+   while (Size != 0)
+   {
+      Res = read(Fd,Buf,std::min(Size,(unsigned long)sizeof(Buf)));
+      if (Res < 0 || (unsigned) Res != std::min(Size,(unsigned long)sizeof(Buf)))
+        return false;
+      Size -= Res;
+      Add(Buf,Res);
+   }
+   return true;
+}
+                                                                       /*}}}*/
diff --git a/apt-pkg/contrib/rhash.h b/apt-pkg/contrib/rhash.h
new file mode 100644 (file)
index 0000000..3eb1373
--- /dev/null
@@ -0,0 +1,32 @@
+// Description                                                         /*{{{*/
+/* ######################################################################
+
+   This is a C++ interface to rpm's hash (aka digest) functions.
+
+   ##################################################################### */
+                                                                       /*}}}*/
+#ifndef APTPKG_RHASH_H
+#define APTPKG_RHASH_H
+
+#include <string>
+#include <rpm/rpmpgp.h>
+
+using std::string;
+
+class raptHash
+{
+   DIGEST_CTX HashCtx;
+   
+   public:
+
+   bool Add(const unsigned char *inbuf,unsigned long inlen);
+   inline bool Add(const char *Data) {return Add((unsigned char *)Data,strlen(Data));}
+   bool AddFD(int Fd,unsigned long Size);
+   inline bool Add(const unsigned char *Beg,const unsigned char *End) 
+                  {return Add(Beg,End-Beg);}
+   string Result();
+   
+   raptHash(pgpHashAlgo HashAlgo);
+};
+
+#endif
diff --git a/apt-pkg/contrib/sha1.cc b/apt-pkg/contrib/sha1.cc
deleted file mode 100644 (file)
index b8e3197..0000000
+++ /dev/null
@@ -1,352 +0,0 @@
-// -*- mode: c++; mode: fold -*-
-// Description                                                          /*{{{*/
-// $Id: sha1.cc,v 1.1 2002/07/23 17:54:51 niemeyer Exp $
-/* ######################################################################
-   
-   SHA1 - SHA-1 Secure Hash Algorithm.
-   
-   This file is a Public Domain wrapper for the Public Domain SHA1 
-   calculation code that is at it's end.
-
-   The algorithm was originally implemented by 
-   Steve Reid <sreid@sea-to-sky.net> and later modified by 
-   James H. Brown <jbrown@burgoyne.com>.
-   
-   Modifications for APT were done by Alfredo K. Kojima and Jason 
-   Gunthorpe.
-   
-   Still in the public domain.
-   
-   Test Vectors (from FIPS PUB 180-1)
-   "abc"
-   A9993E36 4706816A BA3E2571 7850C26C 9CD0D89D
-   "abcdbcdecdefdefgefghfghighijhijkijkljklmklmnlmnomnopnopq"
-   84983E44 1C3BD26E BAAE4AA1 F95129E5 E54670F1
-   A million repetitions of "a"
-   34AA973C D4C4DAA4 F61EEB2B DBAD2731 6534016F
-   
-   ##################################################################### 
- */
-                                                                       /*}}} */
-// Include Files                                                        /*{{{*/
-#include <apt-pkg/sha1.h>
-#include <apt-pkg/strutl.h>
-
-#include <string.h>
-#include <unistd.h>
-#include <inttypes.h>
-#include <config.h>
-#include <system.h>
-                                                                       /*}}}*/
-
-// SHA1Transform - Alters an existing SHA-1 hash                       /*{{{*/
-// ---------------------------------------------------------------------
-/* The core of the SHA-1 algorithm. This alters an existing SHA-1 hash to
-   reflect the addition of 16 longwords of new data. Other routines convert
-   incoming stream data into 16 long word chunks for this routine */
-
-#define rol(value,bits) (((value) << (bits)) | ((value) >> (32 - (bits))))
-
-/* blk0() and blk() perform the initial expand. */
-/* I got the idea of expanding during the round function from SSLeay */
-#ifndef WORDS_BIGENDIAN
-#define blk0(i) (block->l[i] = (rol(block->l[i],24)&0xFF00FF00) \
-    |(rol(block->l[i],8)&0x00FF00FF))
-#else
-#define blk0(i) block->l[i]
-#endif
-#define blk(i) (block->l[i&15] = rol(block->l[(i+13)&15]^block->l[(i+8)&15] \
-    ^block->l[(i+2)&15]^block->l[i&15],1))
-
-/* (R0+R1),R2,R3,R4 are the different operations used in SHA1 */
-#define R0(v,w,x,y,z,i) z+=((w&(x^y))^y)+blk0(i)+0x5A827999+rol(v,5);w=rol(w,30);
-#define R1(v,w,x,y,z,i) z+=((w&(x^y))^y)+blk(i)+0x5A827999+rol(v,5);w=rol(w,30);
-#define R2(v,w,x,y,z,i) z+=(w^x^y)+blk(i)+0x6ED9EBA1+rol(v,5);w=rol(w,30);
-#define R3(v,w,x,y,z,i) z+=(((w|x)&y)|(w&x))+blk(i)+0x8F1BBCDC+rol(v,5);w=rol(w,30);
-#define R4(v,w,x,y,z,i) z+=(w^x^y)+blk(i)+0xCA62C1D6+rol(v,5);w=rol(w,30);
-
-static void SHA1Transform(uint32_t state[5],uint8_t const buffer[64])
-{
-   uint32_t a,b,c,d,e;
-   typedef union
-   {
-      uint8_t c[64];
-      uint32_t l[16];
-   }
-   CHAR64LONG16;
-   CHAR64LONG16 *block;
-
-   uint8_t workspace[64];
-   block = (CHAR64LONG16 *)workspace;
-   memcpy(block,buffer,sizeof(workspace));
-
-   /* Copy context->state[] to working vars */
-   a = state[0];
-   b = state[1];
-   c = state[2];
-   d = state[3];
-   e = state[4];
-   
-   /* 4 rounds of 20 operations each. Loop unrolled. */
-   R0(a,b,c,d,e,0);
-   R0(e,a,b,c,d,1);
-   R0(d,e,a,b,c,2);
-   R0(c,d,e,a,b,3);
-   R0(b,c,d,e,a,4);
-   R0(a,b,c,d,e,5);
-   R0(e,a,b,c,d,6);
-   R0(d,e,a,b,c,7);
-   R0(c,d,e,a,b,8);
-   R0(b,c,d,e,a,9);
-   R0(a,b,c,d,e,10);
-   R0(e,a,b,c,d,11);
-   R0(d,e,a,b,c,12);
-   R0(c,d,e,a,b,13);
-   R0(b,c,d,e,a,14);
-   R0(a,b,c,d,e,15);
-   R1(e,a,b,c,d,16);
-   R1(d,e,a,b,c,17);
-   R1(c,d,e,a,b,18);
-   R1(b,c,d,e,a,19);
-   R2(a,b,c,d,e,20);
-   R2(e,a,b,c,d,21);
-   R2(d,e,a,b,c,22);
-   R2(c,d,e,a,b,23);
-   R2(b,c,d,e,a,24);
-   R2(a,b,c,d,e,25);
-   R2(e,a,b,c,d,26);
-   R2(d,e,a,b,c,27);
-   R2(c,d,e,a,b,28);
-   R2(b,c,d,e,a,29);
-   R2(a,b,c,d,e,30);
-   R2(e,a,b,c,d,31);
-   R2(d,e,a,b,c,32);
-   R2(c,d,e,a,b,33);
-   R2(b,c,d,e,a,34);
-   R2(a,b,c,d,e,35);
-   R2(e,a,b,c,d,36);
-   R2(d,e,a,b,c,37);
-   R2(c,d,e,a,b,38);
-   R2(b,c,d,e,a,39);
-   R3(a,b,c,d,e,40);
-   R3(e,a,b,c,d,41);
-   R3(d,e,a,b,c,42);
-   R3(c,d,e,a,b,43);
-   R3(b,c,d,e,a,44);
-   R3(a,b,c,d,e,45);
-   R3(e,a,b,c,d,46);
-   R3(d,e,a,b,c,47);
-   R3(c,d,e,a,b,48);
-   R3(b,c,d,e,a,49);
-   R3(a,b,c,d,e,50);
-   R3(e,a,b,c,d,51);
-   R3(d,e,a,b,c,52);
-   R3(c,d,e,a,b,53);
-   R3(b,c,d,e,a,54);
-   R3(a,b,c,d,e,55);
-   R3(e,a,b,c,d,56);
-   R3(d,e,a,b,c,57);
-   R3(c,d,e,a,b,58);
-   R3(b,c,d,e,a,59);
-   R4(a,b,c,d,e,60);
-   R4(e,a,b,c,d,61);
-   R4(d,e,a,b,c,62);
-   R4(c,d,e,a,b,63);
-   R4(b,c,d,e,a,64);
-   R4(a,b,c,d,e,65);
-   R4(e,a,b,c,d,66);
-   R4(d,e,a,b,c,67);
-   R4(c,d,e,a,b,68);
-   R4(b,c,d,e,a,69);
-   R4(a,b,c,d,e,70);
-   R4(e,a,b,c,d,71);
-   R4(d,e,a,b,c,72);
-   R4(c,d,e,a,b,73);
-   R4(b,c,d,e,a,74);
-   R4(a,b,c,d,e,75);
-   R4(e,a,b,c,d,76);
-   R4(d,e,a,b,c,77);
-   R4(c,d,e,a,b,78);
-   R4(b,c,d,e,a,79);
-   
-   /* Add the working vars back into context.state[] */
-   state[0] += a;
-   state[1] += b;
-   state[2] += c;
-   state[3] += d;
-   state[4] += e;   
-}
-                                                                       /*}}}*/
-
-// SHA1SumValue::SHA1SumValue - Constructs the summation from a string  /*{{{*/
-// ---------------------------------------------------------------------
-/* The string form of a SHA1 is a 40 character hex number */
-SHA1SumValue::SHA1SumValue(string Str)
-{
-   memset(Sum,0,sizeof(Sum));
-   Set(Str);
-}
-
-                                                                       /*}}} */
-// SHA1SumValue::SHA1SumValue - Default constructor                     /*{{{*/
-// ---------------------------------------------------------------------
-/* Sets the value to 0 */
-SHA1SumValue::SHA1SumValue()
-{
-   memset(Sum,0,sizeof(Sum));
-}
-
-                                                                       /*}}} */
-// SHA1SumValue::Set - Set the sum from a string                        /*{{{*/
-// ---------------------------------------------------------------------
-/* Converts the hex string into a set of chars */
-bool SHA1SumValue::Set(string Str)
-{
-   return Hex2Num(Str,Sum,sizeof(Sum));
-}
-
-                                                                       /*}}} */
-// SHA1SumValue::Value - Convert the number into a string               /*{{{*/
-// ---------------------------------------------------------------------
-/* Converts the set of chars into a hex string in lower case */
-string SHA1SumValue::Value() const
-{
-   char Conv[16] =
-      { '0','1','2','3','4','5','6','7','8','9','a','b',
-      'c','d','e','f'
-   };
-   char Result[41];
-   Result[40] = 0;
-
-   // Convert each char into two letters
-   int J = 0;
-   int I = 0;
-   for (; I != 40; J++,I += 2)
-   {
-      Result[I] = Conv[Sum[J] >> 4];
-      Result[I + 1] = Conv[Sum[J] & 0xF];
-   }
-
-   return string(Result);
-}
-
-                                                                       /*}}} */
-// SHA1SumValue::operator == - Comparator                               /*{{{*/
-// ---------------------------------------------------------------------
-/* Call memcmp on the buffer */
-bool SHA1SumValue::operator == (const SHA1SumValue & rhs) const
-{
-   return memcmp(Sum,rhs.Sum,sizeof(Sum)) == 0;
-}
-                                                                       /*}}}*/
-// SHA1Summation::SHA1Summation - Constructor                           /*{{{*/
-// ---------------------------------------------------------------------
-/* */
-SHA1Summation::SHA1Summation()
-{
-   uint32_t *state = (uint32_t *)State;
-   uint32_t *count = (uint32_t *)Count;
-   
-   /* SHA1 initialization constants */
-   state[0] = 0x67452301;
-   state[1] = 0xEFCDAB89;
-   state[2] = 0x98BADCFE;
-   state[3] = 0x10325476;
-   state[4] = 0xC3D2E1F0;
-   count[0] = 0;
-   count[1] = 0;
-   Done = false;
-}
-                                                                       /*}}}*/
-// SHA1Summation::Result - Return checksum value                        /*{{{*/
-// ---------------------------------------------------------------------
-/* Add() may not be called after this */
-SHA1SumValue SHA1Summation::Result()
-{
-   uint32_t *state = (uint32_t *)State;
-   uint32_t *count = (uint32_t *)Count;
-   
-   // Apply the padding
-   if (Done == false)
-   {
-      unsigned char finalcount[8];
-
-      for (unsigned i = 0; i < 8; i++)
-      {
-        // Endian independent
-        finalcount[i] = (unsigned char) ((count[(i >= 4 ? 0 : 1)]
-                                          >> ((3 - (i & 3)) * 8)) & 255);      
-      }
-      
-      Add((unsigned char *) "\200",1);
-      while ((count[0] & 504) != 448)
-        Add((unsigned char *) "\0",1);
-      
-      Add(finalcount,8);       /* Should cause a SHA1Transform() */
-      
-   }
-
-   Done = true;
-
-   // Transfer over the result
-   SHA1SumValue Value;
-   for (unsigned i = 0; i < 20; i++)
-   {
-      Value.Sum[i] = (unsigned char)
-        ((state[i >> 2] >> ((3 - (i & 3)) * 8)) & 255);
-   }
-   return Value;
-}
-                                                                       /*}}}*/
-// SHA1Summation::Add - Adds content of buffer into the checksum        /*{{{*/
-// ---------------------------------------------------------------------
-/* May not be called after Result() is called */
-bool SHA1Summation::Add(const unsigned char *data,unsigned long len)
-{
-   if (Done)
-      return false;
-
-   uint32_t *state = (uint32_t *)State;
-   uint32_t *count = (uint32_t *)Count;
-   uint8_t *buffer = (uint8_t *)Buffer;
-   uint32_t i,j;
-
-   j = (count[0] >> 3) & 63;
-   if ((count[0] += len << 3) < (len << 3))
-      count[1]++;
-   count[1] += (len >> 29);
-   if ((j + len) > 63)
-   {
-      memcpy(&buffer[j],data,(i = 64 - j));
-      SHA1Transform(state,buffer);
-      for (; i + 63 < len; i += 64)
-      {
-        SHA1Transform(state,&data[i]);
-      }
-      j = 0;
-   }
-   else
-      i = 0;
-   memcpy(&buffer[j],&data[i],len - i);
-   
-   return true;
-}
-                                                                       /*}}}*/
-// SHA1Summation::AddFD - Add content of file into the checksum         /*{{{*/
-// ---------------------------------------------------------------------
-/* */
-bool SHA1Summation::AddFD(int Fd,unsigned long Size)
-{
-   unsigned char Buf[64 * 64];
-   int Res = 0;
-   while (Size != 0)
-   {
-      Res = read(Fd,Buf,std::min(Size,(unsigned long)sizeof(Buf)));
-      if (Res < 0 || (unsigned) Res != std::min(Size,(unsigned long)sizeof(Buf)))
-        return false;
-      Size -= Res;
-      Add(Buf,Res);
-   }
-   return true;
-}
-                                                                       /*}}}*/
index 0dfabfa..d81ca73 100644 (file)
@@ -3,59 +3,24 @@
 // $Id: sha1.h,v 1.1 2002/07/23 17:54:51 niemeyer Exp $
 /* ######################################################################
 
-   SHA1SumValue - Storage for a SHA-1 hash.
-   SHA1Summation - SHA-1 Secure Hash Algorithm.
-   
-   This is a C++ interface to a set of SHA1Sum functions, that mirrors
-   the equivalent MD5 classes. 
+   Legacy wrapper around raptHash class.
 
    ##################################################################### */
                                                                        /*}}}*/
 #ifndef APTPKG_SHA1_H
 #define APTPKG_SHA1_H
 
-#include <cstring>
 #include <string>
+#include <apt-pkg/rhash.h>
 
 using std::string;
 
-class SHA1Summation;
+typedef string SHA1SumValue;
 
-class SHA1SumValue
+class SHA1Summation : public raptHash
 {
-   friend class SHA1Summation;
-   unsigned char Sum[20];
-   
    public:
-
-   // Accessors
-   bool operator ==(const SHA1SumValue &rhs) const; 
-   string Value() const;
-   inline operator string() const {return Value();}
-   bool Set(string Str);
-
-   SHA1SumValue(string Str);
-   SHA1SumValue();
-};
-
-class SHA1Summation
-{
-   /* assumes 64-bit alignment just in case */
-   unsigned char Buffer[64] __attribute__((aligned(8)));
-   unsigned char State[5*4] __attribute__((aligned(8)));
-   unsigned char Count[2*4] __attribute__((aligned(8)));
-   bool Done;
-   
-   public:
-
-   bool Add(const unsigned char *inbuf,unsigned long inlen);
-   inline bool Add(const char *Data) {return Add((unsigned char *)Data,strlen(Data));}
-   bool AddFD(int Fd,unsigned long Size);
-   inline bool Add(const unsigned char *Beg,const unsigned char *End) 
-                  {return Add(Beg,End-Beg);}
-   SHA1SumValue Result();
-   
-   SHA1Summation();
+   SHA1Summation() : raptHash(PGPHASHALGO_SHA1) {};
 };
 
 #endif
index fcbcd4f..cccbeb1 100644 (file)
@@ -579,7 +579,7 @@ string RPMSingleFileHandler::MD5Sum() const
    FileFd File(sFilePath, FileFd::ReadOnly);
    MD5.AddFD(File.Fd(), File.Size());
    File.Close();
-   return MD5.Result().Value();
+   return MD5.Result();
 }
 
 bool RPMSingleFileHandler::ChangeLog(vector<ChangeLogEntry* > &ChangeLogs) const
@@ -723,7 +723,7 @@ string RPMDirHandler::MD5Sum() const
    FileFd File(sFilePath, FileFd::ReadOnly);
    MD5.AddFD(File.Fd(), File.Size());
    File.Close();
-   return MD5.Result().Value();
+   return MD5.Result();
 }
 
 
index 5334c03..1dd47cc 100644 (file)
@@ -9,8 +9,8 @@ template <class T> void Test(const char *In,const char *Out)
 {
    T Sum;
    Sum.Add(In);
-   cout << Sum.Result().Value() << endl;
-   if (stringcasecmp(Sum.Result().Value(),Out) != 0)
+   cout << Sum.Result() << endl;
+   if (stringcasecmp(Sum.Result(),Out) != 0)
       abort();
 }
 
@@ -34,8 +34,8 @@ template <class T> void TestMill(const char *Out)
       }
    }
    
-   cout << Sum.Result().Value() << endl;
-   if (stringcasecmp(Sum.Result().Value(),Out) != 0)
+   cout << Sum.Result() << endl;
+   if (stringcasecmp(Sum.Result(),Out) != 0)
       abort();
 }
 
index 5bf789b..98a57a2 100644 (file)
@@ -90,7 +90,7 @@ void CachedMD5::MD5ForFile(string FileName, time_t TimeStamp, char *buf)
       MD5.AddFD(File.Fd(), File.Size());
       File.Close();
       FileData Data;
-      Data.MD5 = MD5.Result().Value();
+      Data.MD5 = MD5.Result();
       Data.TimeStamp = TimeStamp;
       MD5Table[FileName] = Data;
       strcpy(buf, Data.MD5.c_str());