From: Matt Zimmerman <matt.zimmerman@canonical.com>
Date: Fri, 4 Feb 2005 03:00:00 +0000 (+0000)
Subject: allow SHA1Summation to process a file descriptor until EOF
X-Git-Tag: 0.7.24ubuntu1~360
X-Git-Url: https://git.saurik.com/apt.git/commitdiff_plain/dc50285d324d83548eae7e87b92d4f82b931fc30

allow SHA1Summation to process a file descriptor until EOF
  * Apply patch from Anthony Towns to allow SHA1Summation to process a file
    descriptor until EOF, rather than requiring that the length of input be
    specified (Closes: #291338)
---

diff --git a/apt-pkg/contrib/sha1.cc b/apt-pkg/contrib/sha1.cc
index a22b4d2b2..d7f606982 100644
--- a/apt-pkg/contrib/sha1.cc
+++ b/apt-pkg/contrib/sha1.cc
@@ -343,11 +343,16 @@ bool SHA1Summation::AddFD(int Fd,unsigned long Size)
 {
    unsigned char Buf[64 * 64];
    int Res = 0;
-   while (Size != 0)
+   int ToEOF = (Size == 0);
+   while (Size != 0 || ToEOF);
    {
-      Res = read(Fd,Buf,MIN(Size,sizeof(Buf)));
-      if (Res < 0 || (unsigned) Res != MIN(Size,sizeof(Buf)))
+      int n = sizeof(Buf);
+      if (!ToEOF) n = MIN(Size,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;
       Add(Buf,Res);
    }
diff --git a/debian/changelog b/debian/changelog
index e23460206..bf71480cc 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,6 +1,9 @@
 apt (0.6.31) hoary; urgency=low
 
   * Remove debugging output from apt.cron.daily (no one noticed?)
+  * Apply patch from Anthony Towns to allow SHA1Summation to process a file
+    descriptor until EOF, rather than requiring that the length of input be
+    specified (Closes: #291338)
 
  --