Просмотр исходного кода

* apt-pkg/contrib/hashes.cc, apt-pkg/contrib/md5.cc:
- Support reading until EOF if Size=0 to match behaviour of
SHA1Summation and SHA256Summation

Julian Andres Klode лет назад: 17
Родитель
Сommit
04f4e1a3b1
3 измененных файлов с 21 добавлено и 8 удалено
  1. 9 4
      apt-pkg/contrib/hashes.cc
  2. 9 4
      apt-pkg/contrib/md5.cc
  3. 3 0
      debian/changelog

+ 9 - 4
apt-pkg/contrib/hashes.cc

@@ -108,11 +108,16 @@ bool Hashes::AddFD(int Fd,unsigned long Size)
 {
 {
    unsigned char Buf[64*64];
    unsigned char Buf[64*64];
    int Res = 0;
    int Res = 0;
-   while (Size != 0)
+   int ToEOF = (Size == 0);
+   while (Size != 0 || ToEOF)
    {
    {
-      Res = read(Fd,Buf,min(Size,(unsigned long)sizeof(Buf)));
-      if (Res < 0 || (unsigned)Res != min(Size,(unsigned long)sizeof(Buf)))
-	 return false;
+      unsigned n = sizeof(Buf);
+      if (!ToEOF) n = min(Size,(unsigned long)n);
+      Res = read(Fd,Buf,n);
+      if (Res < 0 || (!ToEOF && (unsigned) Res != n)) // error, or short read
+         return false;
+      if (ToEOF && Res == 0) // EOF
+         break;
       Size -= Res;
       Size -= Res;
       MD5.Add(Buf,Res);
       MD5.Add(Buf,Res);
       SHA1.Add(Buf,Res);
       SHA1.Add(Buf,Res);

+ 9 - 4
apt-pkg/contrib/md5.cc

@@ -294,11 +294,16 @@ bool MD5Summation::AddFD(int Fd,unsigned long Size)
 {
 {
    unsigned char Buf[64*64];
    unsigned char Buf[64*64];
    int Res = 0;
    int Res = 0;
-   while (Size != 0)
+   int ToEOF = (Size == 0);
+   while (Size != 0 || ToEOF)
    {
    {
-      Res = read(Fd,Buf,min(Size,(unsigned long)sizeof(Buf)));
-      if (Res < 0 || (unsigned)Res != min(Size,(unsigned long)sizeof(Buf)))
-	 return false;
+      unsigned n = sizeof(Buf);
+      if (!ToEOF) n = min(Size,(unsigned long)n);
+      Res = read(Fd,Buf,n);
+      if (Res < 0 || (!ToEOF && (unsigned) Res != n)) // error, or short read
+         return false;
+      if (ToEOF && Res == 0) // EOF
+         break;
       Size -= Res;
       Size -= Res;
       Add(Buf,Res);
       Add(Buf,Res);
    }
    }

+ 3 - 0
debian/changelog

@@ -48,6 +48,9 @@ apt (0.7.22) UNRELEASED; urgency=low
   * configure-index: document Dir::Etc::SourceParts and some other options
   * configure-index: document Dir::Etc::SourceParts and some other options
     (Closes: #459605)
     (Closes: #459605)
   * Remove Eugene V. Lyubimkin from uploaders as requested.
   * Remove Eugene V. Lyubimkin from uploaders as requested.
+  * apt-pkg/contrib/hashes.cc, apt-pkg/contrib/md5.cc:
+    - Support reading until EOF if Size=0 to match behaviour of
+      SHA1Summation and SHA256Summation
 
 
  -- Julian Andres Klode <jak@debian.org>  Fri, 03 Jul 2009 08:27:35 +0200
  -- Julian Andres Klode <jak@debian.org>  Fri, 03 Jul 2009 08:27:35 +0200