]> git.saurik.com Git - apt.git/commitdiff
Merge remote-tracking branch 'upstream/debian/experimental' into feature/acq-trans
authorMichael Vogt <mvo@ubuntu.com>
Tue, 7 Oct 2014 14:38:03 +0000 (16:38 +0200)
committerMichael Vogt <mvo@ubuntu.com>
Tue, 7 Oct 2014 14:38:03 +0000 (16:38 +0200)
Conflicts:
apt-pkg/acquire-item.cc

1  2 
apt-pkg/acquire-item.cc
apt-pkg/acquire-item.h
cmdline/apt-get.cc
debian/apt.postinst
test/integration/framework
test/integration/test-compressed-indexes
test/integration/test-pdiff-usage

diff --combined apt-pkg/acquire-item.cc
index 9c1609048538de5b6e9cd644dfa8effce37966a8,779f828d3133a6806979704be4ec8543beb72f0e..5d0a00055501ca1235746209a7cb8ca76a5cd9c8
@@@ -44,9 -44,6 +44,9 @@@
  #include <sstream>
  #include <stdio.h>
  #include <ctime>
 +#include <sys/types.h>
 +#include <pwd.h>
 +#include <grp.h>
  
  #include <apti18n.h>
                                                                        /*}}}*/
@@@ -65,47 -62,19 +65,47 @@@ static void printHashSumComparision(std
        std::cerr <<  "\t- " << hs->toStr() << std::endl;
  }
                                                                        /*}}}*/
 +static void ChangeOwnerAndPermissionOfFile(char const * const requester, char const * const file, char const * const user, char const * const group, mode_t const mode)
 +{
 +   // ensure the file is owned by root and has good permissions
 +   struct passwd const * const pw = getpwnam(user);
 +   struct group const * const gr = getgrnam(group);
 +   if (getuid() == 0) // if we aren't root, we can't chown, so don't try it
 +   {
 +      if (pw != NULL && gr != NULL && chown(file, pw->pw_uid, gr->gr_gid) != 0)
 +       _error->WarningE(requester, "chown to %s:%s of file %s failed", user, group, file);
 +   }
 +   if (chmod(file, mode) != 0)
 +      _error->WarningE(requester, "chmod 0%o of file %s failed", mode, file);
 +}
 +static std::string GetPartialFileName(std::string const &file)
 +{
 +   std::string DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 +   DestFile += file;
 +   return DestFile;
 +}
 +static std::string GetPartialFileNameFromURI(std::string const &uri)
 +{
 +   return GetPartialFileName(URItoFileName(uri));
 +}
 +
  
  // Acquire::Item::Item - Constructor                                  /*{{{*/
  #if __GNUC__ >= 4
        #pragma GCC diagnostic push
        #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
  #endif
 -pkgAcquire::Item::Item(pkgAcquire *Owner, HashStringList const &ExpectedHashes) :
 -   Owner(Owner), FileSize(0), PartialSize(0), Mode(0), ID(0), Complete(false),
 -   Local(false), QueueCounter(0), ExpectedAdditionalItems(0),
 -   ExpectedHashes(ExpectedHashes)
 +pkgAcquire::Item::Item(pkgAcquire *Owner,
 +                       HashStringList const &ExpectedHashes,
 +                       pkgAcqMetaBase *TransactionManager)
 +   : Owner(Owner), FileSize(0), PartialSize(0), Mode(0), ID(0), Complete(false),
 +     Local(false), QueueCounter(0), TransactionManager(TransactionManager),
 +     ExpectedAdditionalItems(0), ExpectedHashes(ExpectedHashes)
  {
     Owner->Add(this);
     Status = StatIdle;
 +   if(TransactionManager != NULL)
 +      TransactionManager->Add(this);
  }
  #if __GNUC__ >= 4
        #pragma GCC diagnostic pop
@@@ -125,15 -94,15 +125,15 @@@ pkgAcquire::Item::~Item(
     fetch this object */
  void pkgAcquire::Item::Failed(string Message,pkgAcquire::MethodConfig *Cnf)
  {
 -   Status = StatIdle;
 -   ErrorText = LookupTag(Message,"Message");
 +   if(ErrorText == "")
 +      ErrorText = LookupTag(Message,"Message");
     UsedMirror =  LookupTag(Message,"UsedMirror");
     if (QueueCounter <= 1)
     {
        /* This indicates that the file is not available right now but might
           be sometime later. If we do a retry cycle then this should be
         retried [CDROMs] */
 -      if (Cnf->LocalOnly == true &&
 +      if (Cnf != NULL && Cnf->LocalOnly == true &&
          StringToBool(LookupTag(Message,"Transient-Failure"),false) == true)
        {
         Status = StatIdle;
        }
  
        Status = StatError;
 +      Complete = false;
        Dequeue();
 -   }   
 +   }
 +   else
 +      Status = StatIdle;
  
     // report mirror failure back to LP if we actually use a mirror
     string FailReason = LookupTag(Message, "FailReason");
@@@ -175,7 -141,7 +175,7 @@@ void pkgAcquire::Item::Done(string Mess
  {
     // We just downloaded something..
     string FileName = LookupTag(Message,"Filename");
 -   UsedMirror =  LookupTag(Message,"UsedMirror");
 +   UsedMirror = LookupTag(Message,"UsedMirror");
     if (Complete == false && !Local && FileName == DestFile)
     {
        if (Owner->Log != 0)
  // ---------------------------------------------------------------------
  /* This helper function is used by a lot of item methods as their final
     step */
 -void pkgAcquire::Item::Rename(string From,string To)
 +bool pkgAcquire::Item::Rename(string From,string To)
  {
     if (rename(From.c_str(),To.c_str()) != 0)
     {
        snprintf(S,sizeof(S),_("rename failed, %s (%s -> %s)."),strerror(errno),
              From.c_str(),To.c_str());
        Status = StatError;
 -      ErrorText = S;
 +      ErrorText += S;
 +      return false;
     }   
 +   return true;
  }
                                                                        /*}}}*/
 +
 +void pkgAcquire::Item::QueueURI(ItemDesc &Item)
 +{
 +   if (RealFileExists(DestFile))
 +      ChangeOwnerAndPermissionOfFile("GetPartialFileName", DestFile.c_str(), "_apt", "root", 0600);
 +   Owner->Enqueue(Item);
 +}
 +void pkgAcquire::Item::Dequeue()
 +{
 +   Owner->Dequeue(this);
 +}
 +
  bool pkgAcquire::Item::RenameOnError(pkgAcquire::Item::RenameOnErrorState const error)/*{{{*/
  {
     if(FileExists(DestFile))
         Status = StatError;
         // do not report as usually its not the mirrors fault, but Portal/Proxy
         break;
 +      case SignatureError:
 +       ErrorText = _("Signature error");
 +       Status = StatError;
 +       break;
 +      case NotClearsigned:
 +         ErrorText = _("Does not start with a cleartext signature");
 +       Status = StatError;
 +       break;
     }
     return false;
 +}
 +                                                                      /*}}}*/
 +void pkgAcquire::Item::SetActiveSubprocess(const std::string &subprocess)/*{{{*/
 +{
 +      ActiveSubprocess = subprocess;
 +#if __GNUC__ >= 4
 +      #pragma GCC diagnostic push
 +      #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
 +#endif
 +        Mode = ActiveSubprocess.c_str();
 +#if __GNUC__ >= 4
 +      #pragma GCC diagnostic pop
 +#endif
  }
                                                                        /*}}}*/
  // Acquire::Item::ReportMirrorFailure                                 /*{{{*/
@@@ -310,6 -241,118 +310,6 @@@ void pkgAcquire::Item::ReportMirrorFail
     }
  }
                                                                        /*}}}*/
 -// AcqSubIndex::AcqSubIndex - Constructor                             /*{{{*/
 -// ---------------------------------------------------------------------
 -/* Get a sub-index file based on checksums from a 'master' file and
 -   possibly query additional files */
 -pkgAcqSubIndex::pkgAcqSubIndex(pkgAcquire *Owner, string const &URI,
 -                               string const &URIDesc, string const &ShortDesc,
 -                               HashStringList const &ExpectedHashes)
 -   : Item(Owner, ExpectedHashes)
 -{
 -   /* XXX: Beware: Currently this class does nothing (of value) anymore ! */
 -   Debug = _config->FindB("Debug::pkgAcquire::SubIndex",false);
 -
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(URI);
 -
 -   Desc.URI = URI;
 -   Desc.Description = URIDesc;
 -   Desc.Owner = this;
 -   Desc.ShortDesc = ShortDesc;
 -
 -   QueueURI(Desc);
 -
 -   if(Debug)
 -      std::clog << "pkgAcqSubIndex: " << Desc.URI << std::endl;
 -}
 -                                                                      /*}}}*/
 -// AcqSubIndex::Custom600Headers - Insert custom request headers      /*{{{*/
 -// ---------------------------------------------------------------------
 -/* The only header we use is the last-modified header. */
 -string pkgAcqSubIndex::Custom600Headers() const
 -{
 -   string Final = _config->FindDir("Dir::State::lists");
 -   Final += URItoFileName(Desc.URI);
 -
 -   struct stat Buf;
 -   if (stat(Final.c_str(),&Buf) != 0)
 -      return "\nIndex-File: true\nFail-Ignore: true\n";
 -   return "\nIndex-File: true\nFail-Ignore: true\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
 -}
 -                                                                      /*}}}*/
 -void pkgAcqSubIndex::Failed(string Message,pkgAcquire::MethodConfig * /*Cnf*/)/*{{{*/
 -{
 -   if(Debug)
 -      std::clog << "pkgAcqSubIndex failed: " << Desc.URI << " with " << Message << std::endl;
 -
 -   Complete = false;
 -   Status = StatDone;
 -   Dequeue();
 -
 -   // No good Index is provided
 -}
 -                                                                      /*}}}*/
 -void pkgAcqSubIndex::Done(string Message,unsigned long long Size,HashStringList const &Hashes,        /*{{{*/
 -                         pkgAcquire::MethodConfig *Cnf)
 -{
 -   if(Debug)
 -      std::clog << "pkgAcqSubIndex::Done(): " << Desc.URI << std::endl;
 -
 -   string FileName = LookupTag(Message,"Filename");
 -   if (FileName.empty() == true)
 -   {
 -      Status = StatError;
 -      ErrorText = "Method gave a blank filename";
 -      return;
 -   }
 -
 -   if (FileName != DestFile)
 -   {
 -      Local = true;
 -      Desc.URI = "copy:" + FileName;
 -      QueueURI(Desc);
 -      return;
 -   }
 -
 -   Item::Done(Message, Size, Hashes, Cnf);
 -
 -   string FinalFile = _config->FindDir("Dir::State::lists")+URItoFileName(Desc.URI);
 -
 -   /* Downloaded invalid transindex => Error (LP: #346386) (Closes: #627642) */
 -   indexRecords SubIndexParser;
 -   if (FileExists(DestFile) == true && !SubIndexParser.Load(DestFile)) {
 -      Status = StatError;
 -      ErrorText = SubIndexParser.ErrorText;
 -      return;
 -   }
 -
 -   // success in downloading the index
 -   // rename the index
 -   if(Debug)
 -      std::clog << "Renaming: " << DestFile << " -> " << FinalFile << std::endl;
 -   Rename(DestFile,FinalFile);
 -   chmod(FinalFile.c_str(),0644);
 -   DestFile = FinalFile;
 -
 -   if(ParseIndex(DestFile) == false)
 -      return Failed("", NULL);
 -
 -   Complete = true;
 -   Status = StatDone;
 -   Dequeue();
 -   return;
 -}
 -                                                                      /*}}}*/
 -bool pkgAcqSubIndex::ParseIndex(string const &IndexFile)              /*{{{*/
 -{
 -   indexRecords SubIndexParser;
 -   if (FileExists(IndexFile) == false || SubIndexParser.Load(IndexFile) == false)
 -      return false;
 -   // so something with the downloaded index
 -   return true;
 -}
 -                                                                      /*}}}*/
  // AcqDiffIndex::AcqDiffIndex - Constructor                           /*{{{*/
  // ---------------------------------------------------------------------
  /* Get the DiffIndex file first and see if there are patches available
   * the original packages file
   */
  pkgAcqDiffIndex::pkgAcqDiffIndex(pkgAcquire *Owner,
 +                                 pkgAcqMetaBase *TransactionManager,
                                   IndexTarget const * const Target,
                                 HashStringList const &ExpectedHashes,
                                   indexRecords *MetaIndexParser)
 -   : pkgAcqBaseIndex(Owner, Target, ExpectedHashes, MetaIndexParser)
 +   : pkgAcqBaseIndex(Owner, TransactionManager, Target, ExpectedHashes, 
 +                     MetaIndexParser), PackagesFileReadyInPartial(false)
  {
     
     Debug = _config->FindB("Debug::pkgAcquire::Diffs",false);
  
     RealURI = Target->URI;
     Desc.Owner = this;
--   Desc.Description = Target->Description + "/DiffIndex";
++   Desc.Description = Target->Description + ".diff/Index";
     Desc.ShortDesc = Target->ShortDesc;
     Desc.URI = Target->URI + ".diff/Index";
  
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(Desc.URI);
 +   DestFile = GetPartialFileNameFromURI(Desc.URI);
  
     if(Debug)
        std::clog << "pkgAcqDiffIndex: " << Desc.URI << std::endl;
        Desc.URI.substr(0,strlen("file:/")) == "file:/")
     {
        // we don't have a pkg file or we don't want to queue
-       if(Debug)
-        std::clog << "No index file, local or canceld by user" << std::endl;
-       Failed("", NULL);
+       Failed("No index file, local or canceld by user", NULL);
        return;
     }
  
@@@ -371,7 -411,7 +369,7 @@@ string pkgAcqDiffIndex::Custom600Header
  {
     string Final = _config->FindDir("Dir::State::lists");
     Final += URItoFileName(Desc.URI);
--   
++
     if(Debug)
        std::clog << "Custom600Header-IMS: " << Final << std::endl;
  
                                                                        /*}}}*/
  bool pkgAcqDiffIndex::ParseDiffIndex(string IndexDiffFile)            /*{{{*/
  {
+    // failing here is fine: our caller will take care of trying to
+    // get the complete file if patching fails
     if(Debug)
        std::clog << "pkgAcqDiffIndex::ParseIndexDiff() " << IndexDiffFile
         << std::endl;
  
-    pkgTagSection Tags;
-    string ServerSha1;
-    vector<DiffInfo> available_patches;
-    
     FileFd Fd(IndexDiffFile,FileFd::ReadOnly);
     pkgTagFile TF(&Fd);
     if (_error->PendingError() == true)
        return false;
  
-    if(TF.Step(Tags) == true)
-    {
-       bool found = false;
-       DiffInfo d;
-       string size;
+    pkgTagSection Tags;
+    if(unlikely(TF.Step(Tags) == false))
+       return false;
  
-       string const tmp = Tags.FindS("SHA1-Current");
+    HashStringList ServerHashes;
+    unsigned long long ServerSize = 0;
+    for (char const * const * type = HashString::SupportedHashes(); *type != NULL; ++type)
+    {
+       std::string tagname = *type;
+       tagname.append("-Current");
+       std::string const tmp = Tags.FindS(tagname.c_str());
+       if (tmp.empty() == true)
+        continue;
+       string hash;
+       unsigned long long size;
        std::stringstream ss(tmp);
-       ss >> ServerSha1 >> size;
-       unsigned long const ServerSize = atol(size.c_str());
+       ss >> hash >> size;
+       if (unlikely(hash.empty() == true))
+        continue;
+       if (unlikely(ServerSize != 0 && ServerSize != size))
+        continue;
+       ServerHashes.push_back(HashString(*type, hash));
+       ServerSize = size;
+    }
  
-       FileFd fd(CurrentPackagesFile, FileFd::ReadOnly);
-       SHA1Summation SHA1;
-       SHA1.AddFD(fd);
-       string const local_sha1 = SHA1.Result();
+    if (ServerHashes.usable() == false)
+    {
+       if (Debug == true)
+        std::clog << "pkgAcqDiffIndex: " << IndexDiffFile << ": Did not find a good hashsum in the index" << std::endl;
+       return false;
+    }
  
-       if(local_sha1 == ServerSha1)
+    if (ServerHashes != HashSums())
+    {
+       if (Debug == true)
 +      {
-        // we have the same sha1 as the server so we are done here
-        if(Debug)
-           std::clog << "Package file is up-to-date" << std::endl;
-          // ensure we have no leftovers from previous runs
-          std::string Partial = _config->FindDir("Dir::State::lists");
-          Partial += "partial/" + URItoFileName(RealURI);
-          unlink(Partial.c_str());
-        // list cleanup needs to know that this file as well as the already
-        // present index is ours, so we create an empty diff to save it for us
-        new pkgAcqIndexDiffs(Owner, TransactionManager, Target, 
-                               ExpectedHashes, MetaIndexParser, 
-                               ServerSha1, available_patches);
-        return true;
+        std::clog << "pkgAcqDiffIndex: " << IndexDiffFile << ": Index has different hashes than parser, probably older, so fail pdiffing" << std::endl;
++         printHashSumComparision(CurrentPackagesFile, ServerHashes, HashSums());
 +      }
-       else
+       return false;
+    }
+    if (ServerHashes.VerifyFile(CurrentPackagesFile) == true)
+    {
+       // we have the same sha1 as the server so we are done here
+       if(Debug)
 -       std::clog << "pkgAcqDiffIndex: Package file is up-to-date" << std::endl;
++       std::clog << "pkgAcqDiffIndex: Package file " << CurrentPackagesFile << " is up-to-date" << std::endl;
++
+       // list cleanup needs to know that this file as well as the already
+       // present index is ours, so we create an empty diff to save it for us
 -      new pkgAcqIndexDiffs(Owner, Target, ExpectedHashes, MetaIndexParser);
++      new pkgAcqIndexDiffs(Owner, TransactionManager, Target,
++                           ExpectedHashes, MetaIndexParser);
+       return true;
+    }
+    FileFd fd(CurrentPackagesFile, FileFd::ReadOnly);
+    Hashes LocalHashesCalc;
+    LocalHashesCalc.AddFD(fd);
+    HashStringList const LocalHashes = LocalHashesCalc.GetHashStringList();
+    if(Debug)
+       std::clog << "Server-Current: " << ServerHashes.find(NULL)->toStr() << " and we start at "
+        << fd.Name() << " " << fd.FileSize() << " " << LocalHashes.find(NULL)->toStr() << std::endl;
+    // parse all of (provided) history
+    vector<DiffInfo> available_patches;
+    bool firstAcceptedHashes = true;
+    for (char const * const * type = HashString::SupportedHashes(); *type != NULL; ++type)
+    {
+       if (LocalHashes.find(*type) == NULL)
+        continue;
+       std::string tagname = *type;
+       tagname.append("-History");
+       std::string const tmp = Tags.FindS(tagname.c_str());
+       if (tmp.empty() == true)
+        continue;
+       string hash, filename;
+       unsigned long long size;
+       std::stringstream ss(tmp);
+       while (ss >> hash >> size >> filename)
        {
-        if(Debug)
-           std::clog << "SHA1-Current: " << ServerSha1 << " and we start at "<< fd.Name() << " " << fd.Size() << " " << local_sha1 << std::endl;
+        if (unlikely(hash.empty() == true || filename.empty() == true))
+           continue;
  
-        // check the historie and see what patches we need
-        string const history = Tags.FindS("SHA1-History");
-        std::stringstream hist(history);
-        while(hist >> d.sha1 >> size >> d.file)
+        // see if we have a record for this file already
+        std::vector<DiffInfo>::iterator cur = available_patches.begin();
+        for (; cur != available_patches.end(); ++cur)
         {
-           // read until the first match is found
-           // from that point on, we probably need all diffs
-           if(d.sha1 == local_sha1) 
-              found=true;
-           else if (found == false)
+           if (cur->file != filename || unlikely(cur->result_size != size))
               continue;
-           if(Debug)
-              std::clog << "Need to get diff: " << d.file << std::endl;
-           available_patches.push_back(d);
+           cur->result_hashes.push_back(HashString(*type, hash));
+           break;
         }
-        if (available_patches.empty() == false)
+        if (cur != available_patches.end())
+           continue;
+        if (firstAcceptedHashes == true)
+        {
+           DiffInfo next;
+           next.file = filename;
+           next.result_hashes.push_back(HashString(*type, hash));
+           next.result_size = size;
+           next.patch_size = 0;
+           available_patches.push_back(next);
+        }
+        else
         {
-           // patching with too many files is rather slow compared to a fast download
-           unsigned long const fileLimit = _config->FindI("Acquire::PDiffs::FileLimit", 0);
-           if (fileLimit != 0 && fileLimit < available_patches.size())
-           {
-              if (Debug)
-                 std::clog << "Need " << available_patches.size() << " diffs (Limit is " << fileLimit
-                       << ") so fallback to complete download" << std::endl;
-              return false;
-           }
-           // see if the patches are too big
-           found = false; // it was true and it will be true again at the end
-           d = *available_patches.begin();
-           string const firstPatch = d.file;
-           unsigned long patchesSize = 0;
-           std::stringstream patches(Tags.FindS("SHA1-Patches"));
-           while(patches >> d.sha1 >> size >> d.file)
-           {
-              if (firstPatch == d.file)
-                 found = true;
-              else if (found == false)
-                 continue;
-              patchesSize += atol(size.c_str());
-           }
-           unsigned long const sizeLimit = ServerSize * _config->FindI("Acquire::PDiffs::SizeLimit", 100);
-           if (sizeLimit > 0 && (sizeLimit/100) < patchesSize)
-           {
-              if (Debug)
-                 std::clog << "Need " << patchesSize << " bytes (Limit is " << sizeLimit/100
-                       << ") so fallback to complete download" << std::endl;
-              return false;
-           }
+           if (Debug == true)
+              std::clog << "pkgAcqDiffIndex: " << IndexDiffFile << ": File " << filename
+                 << " wasn't in the list for the first parsed hash! (history)" << std::endl;
+           break;
         }
        }
+       firstAcceptedHashes = false;
+    }
+    if (unlikely(available_patches.empty() == true))
+    {
+       if (Debug)
+        std::clog << "pkgAcqDiffIndex: " << IndexDiffFile << ": "
+           << "Couldn't find any patches for the patch series." << std::endl;
+       return false;
+    }
+    for (char const * const * type = HashString::SupportedHashes(); *type != NULL; ++type)
+    {
+       if (LocalHashes.find(*type) == NULL)
+        continue;
+       std::string tagname = *type;
+       tagname.append("-Patches");
+       std::string const tmp = Tags.FindS(tagname.c_str());
+       if (tmp.empty() == true)
+        continue;
  
-       // we have something, queue the next diff
-       if(found)
+       string hash, filename;
+       unsigned long long size;
+       std::stringstream ss(tmp);
+       while (ss >> hash >> size >> filename)
        {
-          // FIXME: make this use the method
-          PackagesFileReadyInPartial = true;
-        std::string const Partial = GetPartialFileNameFromURI(RealURI);
-          FileFd From(CurrentPackagesFile, FileFd::ReadOnly);
-          FileFd To(Partial, FileFd::WriteEmpty);
-          if(CopyFile(From, To) == false)
-             return _error->Errno("CopyFile", "failed to copy");
-          if(Debug)
-             std::cerr << "Done copying " << CurrentPackagesFile
-                       << " -> " << Partial
-                       << std::endl;
-        // queue the diffs
-        string::size_type const last_space = Description.rfind(" ");
-        if(last_space != string::npos)
-           Description.erase(last_space, Description.size()-last_space);
-        /* decide if we should download patches one by one or in one go:
-           The first is good if the server merges patches, but many don't so client
-           based merging can be attempt in which case the second is better.
-           "bad things" will happen if patches are merged on the server,
-           but client side merging is attempt as well */
-        bool pdiff_merge = _config->FindB("Acquire::PDiffs::Merge", true);
-        if (pdiff_merge == true)
-        {
-           // reprepro adds this flag if it has merged patches on the server
-           std::string const precedence = Tags.FindS("X-Patch-Precedence");
-           pdiff_merge = (precedence != "merged");
-        }
+        if (unlikely(hash.empty() == true || filename.empty() == true))
+           continue;
  
-        if (pdiff_merge == false)
-          {
-           new pkgAcqIndexDiffs(Owner, TransactionManager, Target, ExpectedHashes, 
-                                  MetaIndexParser,
-                                  ServerSha1, available_patches);
-          }
-          else
+        // see if we have a record for this file already
+        std::vector<DiffInfo>::iterator cur = available_patches.begin();
+        for (; cur != available_patches.end(); ++cur)
         {
-           std::vector<pkgAcqIndexMergeDiffs*> *diffs = new std::vector<pkgAcqIndexMergeDiffs*>(available_patches.size());
-           for(size_t i = 0; i < available_patches.size(); ++i)
-              (*diffs)[i] = new pkgAcqIndexMergeDiffs(Owner,
-                                                        TransactionManager,
-                                                        Target,
-                                                        ExpectedHashes,
-                                                        MetaIndexParser,
-                                                        available_patches[i],
-                                                        diffs);
+           if (cur->file != filename)
+              continue;
+           if (unlikely(cur->patch_size != 0 && cur->patch_size != size))
+              continue;
+           cur->patch_hashes.push_back(HashString(*type, hash));
+           cur->patch_size = size;
+           break;
         }
-          Complete = false;
-          Status = StatDone;
-          Dequeue();
-          return true;
+        if (cur != available_patches.end())
+              continue;
+        if (Debug == true)
+           std::clog << "pkgAcqDiffIndex: " << IndexDiffFile << ": File " << filename
+              << " wasn't in the list for the first parsed hash! (patches)" << std::endl;
+        break;
        }
     }
-    // Nothing found, report and return false
-    // Failing here is ok, if we return false later, the full
-    // IndexFile is queued
+    bool foundStart = false;
+    for (std::vector<DiffInfo>::iterator cur = available_patches.begin();
+        cur != available_patches.end(); ++cur)
+    {
+       if (LocalHashes != cur->result_hashes)
+        continue;
+       available_patches.erase(available_patches.begin(), cur);
+       foundStart = true;
+       break;
+    }
+    if (foundStart == false || unlikely(available_patches.empty() == true))
+    {
+       if (Debug)
+        std::clog << "pkgAcqDiffIndex: " << IndexDiffFile << ": "
+           << "Couldn't find the start of the patch series." << std::endl;
+       return false;
+    }
+    // patching with too many files is rather slow compared to a fast download
+    unsigned long const fileLimit = _config->FindI("Acquire::PDiffs::FileLimit", 0);
+    if (fileLimit != 0 && fileLimit < available_patches.size())
+    {
+       if (Debug)
+        std::clog << "Need " << available_patches.size() << " diffs (Limit is " << fileLimit
+           << ") so fallback to complete download" << std::endl;
+       return false;
+    }
+    // calculate the size of all patches we have to get
+    // note that all sizes are uncompressed, while we download compressed files
+    unsigned long long patchesSize = 0;
+    for (std::vector<DiffInfo>::const_iterator cur = available_patches.begin();
+        cur != available_patches.end(); ++cur)
+       patchesSize += cur->patch_size;
+    unsigned long long const sizeLimit = ServerSize * _config->FindI("Acquire::PDiffs::SizeLimit", 100);
+    if (false && sizeLimit > 0 && (sizeLimit/100) < patchesSize)
+    {
+       if (Debug)
+        std::clog << "Need " << patchesSize << " bytes (Limit is " << sizeLimit/100
+           << ") so fallback to complete download" << std::endl;
+       return false;
+    }
++   // FIXME: make this use the method
++   PackagesFileReadyInPartial = true;
++   std::string const Partial = GetPartialFileNameFromURI(RealURI);
++   
++   FileFd From(CurrentPackagesFile, FileFd::ReadOnly);
++   FileFd To(Partial, FileFd::WriteEmpty);
++   if(CopyFile(From, To) == false)
++      return _error->Errno("CopyFile", "failed to copy");
 +   
-       std::clog << "Can't find a patch in the index file" << std::endl;
-    return false;
 +   if(Debug)
 -      new pkgAcqIndexDiffs(Owner, Target, ExpectedHashes, MetaIndexParser,
 -          available_patches);
++      std::cerr << "Done copying " << CurrentPackagesFile
++                << " -> " << Partial
++                << std::endl;
++
+    // we have something, queue the diffs
+    string::size_type const last_space = Description.rfind(" ");
+    if(last_space != string::npos)
+       Description.erase(last_space, Description.size()-last_space);
+    /* decide if we should download patches one by one or in one go:
+       The first is good if the server merges patches, but many don't so client
+       based merging can be attempt in which case the second is better.
+       "bad things" will happen if patches are merged on the server,
+       but client side merging is attempt as well */
+    bool pdiff_merge = _config->FindB("Acquire::PDiffs::Merge", true);
+    if (pdiff_merge == true)
+    {
+       // reprepro adds this flag if it has merged patches on the server
+       std::string const precedence = Tags.FindS("X-Patch-Precedence");
+       pdiff_merge = (precedence != "merged");
+    }
+    if (pdiff_merge == false)
+    {
 -       (*diffs)[i] = new pkgAcqIndexMergeDiffs(Owner, Target,
++      new pkgAcqIndexDiffs(Owner, TransactionManager, Target, ExpectedHashes, 
++                           MetaIndexParser, available_patches);
+    }
+    else
+    {
+       std::vector<pkgAcqIndexMergeDiffs*> *diffs = new std::vector<pkgAcqIndexMergeDiffs*>(available_patches.size());
+       for(size_t i = 0; i < available_patches.size(); ++i)
++       (*diffs)[i] = new pkgAcqIndexMergeDiffs(Owner, TransactionManager,
++               Target,
+              ExpectedHashes,
+              MetaIndexParser,
+              available_patches[i],
+              diffs);
+    }
+    Complete = false;
+    Status = StatDone;
+    Dequeue();
+    return true;
  }
                                                                        /*}}}*/
 -void pkgAcqDiffIndex::Failed(string Message,pkgAcquire::MethodConfig * /*Cnf*/)/*{{{*/
 +void pkgAcqDiffIndex::Failed(string Message,pkgAcquire::MethodConfig * Cnf)/*{{{*/
  {
     if(Debug)
        std::clog << "pkgAcqDiffIndex failed: " << Desc.URI << " with " << Message << std::endl
                << "Falling back to normal index file acquire" << std::endl;
  
 -   new pkgAcqIndex(Owner, Target, ExpectedHashes, MetaIndexParser);
 +   new pkgAcqIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser);
  
 -   Complete = false;
 +   Item::Failed(Message,Cnf);
     Status = StatDone;
 -   Dequeue();
  }
                                                                        /*}}}*/
  void pkgAcqDiffIndex::Done(string Message,unsigned long long Size,HashStringList const &Hashes,       /*{{{*/
  
     Item::Done(Message, Size, Hashes, Cnf);
  
 -   FinalFile = _config->FindDir("Dir::State::lists")+URItoFileName(RealURI);
 +   // verify the index target
 +   if(Target && Target->MetaKey != "" && MetaIndexParser && Hashes.usable())
 +   {
 +      std::string IndexMetaKey  = Target->MetaKey + ".diff/Index";
 +      indexRecords::checkSum *Record = MetaIndexParser->Lookup(IndexMetaKey);
 +      if(Record && Record->Hashes.usable() && Hashes != Record->Hashes)
 +      {
 +         RenameOnError(HashSumMismatch);
 +         printHashSumComparision(RealURI, Record->Hashes, Hashes);
 +         Failed(Message, Cnf);
 +         return;
 +      }
 +
 +   }
 +
+    string FinalFile;
 -   // success in downloading the index
 -   // rename the index
 -   FinalFile += string(".IndexDiff");
 -   if(Debug)
 -      std::clog << "Renaming: " << DestFile << " -> " << FinalFile 
 -              << std::endl;
 -   Rename(DestFile,FinalFile);
 -   chmod(FinalFile.c_str(),0644);
 -   DestFile = FinalFile;
++   FinalFile = _config->FindDir("Dir::State::lists");
++   FinalFile += URItoFileName(Desc.URI);
++   if(StringToBool(LookupTag(Message,"IMS-Hit"),false))
++      DestFile = FinalFile;
     if(!ParseDiffIndex(DestFile))
 -      return Failed("Parsing pdiff Index failed", NULL);
 +      return Failed("Message: Couldn't parse pdiff index", Cnf);
 +
 +   // queue for final move
-    string FinalFile;
-    FinalFile = _config->FindDir("Dir::State::lists")+URItoFileName(RealURI);
-    FinalFile += string(".IndexDiff");
 +   TransactionManager->TransactionStageCopy(this, DestFile, FinalFile);
  
     Complete = true;
     Status = StatDone;
   * for each diff and the index
   */
  pkgAcqIndexDiffs::pkgAcqIndexDiffs(pkgAcquire *Owner,
 +                                   pkgAcqMetaBase *TransactionManager,
                                     struct IndexTarget const * const Target,
                                     HashStringList const &ExpectedHashes,
                                     indexRecords *MetaIndexParser,
-                                  string ServerSha1,
                                   vector<DiffInfo> diffs)
 -   : pkgAcqBaseIndex(Owner, Target, ExpectedHashes, MetaIndexParser),
 +   : pkgAcqBaseIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser),
-      available_patches(diffs), ServerSha1(ServerSha1)
+      available_patches(diffs)
  {
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(Target->URI);
 +   DestFile = GetPartialFileNameFromURI(Target->URI);
  
     Debug = _config->FindB("Debug::pkgAcquire::Diffs",false);
  
  
     if(available_patches.empty() == true)
     {
 -      // we are done (yeah!)
 +      // we are done (yeah!), check hashes against the final file
 +      DestFile = _config->FindDir("Dir::State::lists");
 +      DestFile += URItoFileName(Target->URI);
        Finish(true);
     }
     else
@@@ -652,22 -768,20 +759,22 @@@ void pkgAcqIndexDiffs::Failed(string Me
     if(Debug)
        std::clog << "pkgAcqIndexDiffs failed: " << Desc.URI << " with " << Message << std::endl
                << "Falling back to normal index file acquire" << std::endl;
 -   new pkgAcqIndex(Owner, Target, ExpectedHashes, MetaIndexParser);
 +   new pkgAcqIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser);
     Finish();
  }
                                                                        /*}}}*/
  // Finish - helper that cleans the item out of the fetcher queue      /*{{{*/
  void pkgAcqIndexDiffs::Finish(bool allDone)
  {
 +   if(Debug)
 +      std::clog << "pkgAcqIndexDiffs::Finish(): " 
 +                << allDone << " "
 +                << Desc.URI << std::endl;
 +
     // we restore the original name, this is required, otherwise
     // the file will be cleaned
     if(allDone) 
     {
 -      DestFile = _config->FindDir("Dir::State::lists");
 -      DestFile += URItoFileName(RealURI);
 -
        if(HashSums().usable() && !HashSums().VerifyFile(DestFile))
        {
         RenameOnError(HashSumMismatch);
         return;
        }
  
-       PartialFile = GetPartialFileNameFromURI(RealURI);
-       DestFile = _config->FindDir("Dir::State::lists");
-       DestFile += URItoFileName(RealURI);
-       // this happens if we have a up-to-date indexfile
-       if(!FileExists(PartialFile))
-          PartialFile = DestFile;
-       TransactionManager->TransactionStageCopy(this, PartialFile, DestFile);
 +      // queue for copy
++      std::string FinalFile = _config->FindDir("Dir::State::lists");
++      FinalFile += URItoFileName(RealURI);
++      TransactionManager->TransactionStageCopy(this, DestFile, FinalFile);
 +
        // this is for the "real" finish
        Complete = true;
        Status = StatDone;
                                                                        /*}}}*/
  bool pkgAcqIndexDiffs::QueueNextDiff()                                        /*{{{*/
  {
 -
     // calc sha1 of the just patched file
 -   string FinalFile = _config->FindDir("Dir::State::lists");
 -   FinalFile += URItoFileName(RealURI);
 +   std::string const FinalFile = GetPartialFileNameFromURI(RealURI);
 +
 +   if(!FileExists(FinalFile))
 +   {
 +      Failed("Message: No FinalFile " + FinalFile + " available", NULL);
 +      return false;
 +   }
  
     FileFd fd(FinalFile, FileFd::ReadOnly);
-    SHA1Summation SHA1;
-    SHA1.AddFD(fd);
-    string local_sha1 = string(SHA1.Result());
+    Hashes LocalHashesCalc;
+    LocalHashesCalc.AddFD(fd);
+    HashStringList const LocalHashes = LocalHashesCalc.GetHashStringList();
     if(Debug)
-       std::clog << "QueueNextDiff: " 
-               << FinalFile << " (" << local_sha1 << ")"<<std::endl;
+       std::clog << "QueueNextDiff: " << FinalFile << " (" << LocalHashes.find(NULL)->toStr() << ")" << std::endl;
+    if (unlikely(LocalHashes.usable() == false || ExpectedHashes.usable() == false))
+    {
+       Failed("Local/Expected hashes are not usable", NULL);
+       return false;
+    }
  
 +
     // final file reached before all patches are applied
-    if(local_sha1 == ServerSha1)
+    if(LocalHashes == ExpectedHashes)
     {
        Finish(true);
        return true;
  
     // remove all patches until the next matching patch is found
     // this requires the Index file to be ordered
-    for(vector<DiffInfo>::iterator I=available_patches.begin();
+    for(vector<DiffInfo>::iterator I = available_patches.begin();
         available_patches.empty() == false &&
          I != available_patches.end() &&
-         I->sha1 != local_sha1;
+         I->result_hashes != LocalHashes;
         ++I)
     {
        available_patches.erase(I);
     // error checking and falling back if no patch was found
     if(available_patches.empty() == true)
     {
-       Failed("Message: No patches available", NULL);
+       Failed("No patches left to reach target", NULL);
        return false;
     }
  
     // queue the right diff
     Desc.URI = RealURI + ".diff/" + available_patches[0].file + ".gz";
     Desc.Description = Description + " " + available_patches[0].file + string(".pdiff");
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(RealURI + ".diff/" + available_patches[0].file);
 +   DestFile = GetPartialFileNameFromURI(RealURI + ".diff/" + available_patches[0].file);
  
     if(Debug)
        std::clog << "pkgAcqIndexDiffs::QueueNextDiff(): " << Desc.URI << std::endl;
-    
     QueueURI(Desc);
  
     return true;
@@@ -770,12 -874,23 +876,23 @@@ void pkgAcqIndexDiffs::Done(string Mess
  
     Item::Done(Message, Size, Hashes, Cnf);
  
 -   string FinalFile;
 -   FinalFile = _config->FindDir("Dir::State::lists")+URItoFileName(RealURI);
 +   // FIXME: verify this download too before feeding it to rred
 +   std::string const FinalFile = GetPartialFileNameFromURI(RealURI);
  
     // success in downloading a diff, enter ApplyDiff state
     if(State == StateFetchDiff)
     {
+       FileFd fd(DestFile, FileFd::ReadOnly, FileFd::Gzip);
+       class Hashes LocalHashesCalc;
+       LocalHashesCalc.AddFD(fd);
+       HashStringList const LocalHashes = LocalHashesCalc.GetHashStringList();
+       if (fd.Size() != available_patches[0].patch_size ||
+           available_patches[0].patch_hashes != LocalHashes)
+       {
+        Failed("Patch has Size/Hashsum mismatch", NULL);
+        return;
+       }
  
        // rred excepts the patch as $FinalFile.ed
        Rename(DestFile,FinalFile+".ed");
        Local = true;
        Desc.URI = "rred:" + FinalFile;
        QueueURI(Desc);
 -      ActiveSubprocess = "rred";
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic push
 -      #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
 -#endif
 -      Mode = "rred";
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic pop
 -#endif
 +      SetActiveSubprocess("rred");
        return;
     } 
  
  
        // see if there is more to download
        if(available_patches.empty() == false) {
 -       new pkgAcqIndexDiffs(Owner, Target,
 +       new pkgAcqIndexDiffs(Owner, TransactionManager, Target,
                              ExpectedHashes, MetaIndexParser,
-                               ServerSha1, available_patches);
+                             available_patches);
         return Finish();
        } else 
 +         // update
 +         DestFile = FinalFile;
         return Finish(true);
     }
  }
                                                                        /*}}}*/
  // AcqIndexMergeDiffs::AcqIndexMergeDiffs - Constructor                       /*{{{*/
  pkgAcqIndexMergeDiffs::pkgAcqIndexMergeDiffs(pkgAcquire *Owner,
 +                                             pkgAcqMetaBase *TransactionManager,
                                               struct IndexTarget const * const Target,
                                               HashStringList const &ExpectedHashes,
                                               indexRecords *MetaIndexParser,
                                               DiffInfo const &patch,
                                               std::vector<pkgAcqIndexMergeDiffs*> const * const allPatches)
 -   : pkgAcqBaseIndex(Owner, Target, ExpectedHashes, MetaIndexParser),
 +  : pkgAcqBaseIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser),
       patch(patch), allPatches(allPatches), State(StateFetchDiff)
  {
 -
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(Target->URI);
 -
     Debug = _config->FindB("Debug::pkgAcquire::Diffs",false);
  
     RealURI = Target->URI;
  
     Desc.URI = RealURI + ".diff/" + patch.file + ".gz";
     Desc.Description = Description + " " + patch.file + string(".pdiff");
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(RealURI + ".diff/" + patch.file);
 +
 +   DestFile = GetPartialFileNameFromURI(RealURI + ".diff/" + patch.file);
  
     if(Debug)
        std::clog << "pkgAcqIndexMergeDiffs: " << Desc.URI << std::endl;
     QueueURI(Desc);
  }
                                                                        /*}}}*/
 -void pkgAcqIndexMergeDiffs::Failed(string Message,pkgAcquire::MethodConfig * /*Cnf*/)/*{{{*/
 +void pkgAcqIndexMergeDiffs::Failed(string Message,pkgAcquire::MethodConfig * Cnf)/*{{{*/
  {
     if(Debug)
        std::clog << "pkgAcqIndexMergeDiffs failed: " << Desc.URI << " with " << Message << std::endl;
 -   Complete = false;
 +
 +   Item::Failed(Message,Cnf);
     Status = StatDone;
 -   Dequeue();
  
     // check if we are the first to fail, otherwise we are done here
     State = StateDoneDiff;
     // first failure means we should fallback
     State = StateErrorDiff;
     std::clog << "Falling back to normal index file acquire" << std::endl;
 -   new pkgAcqIndex(Owner, Target, ExpectedHashes, MetaIndexParser);
 +   new pkgAcqIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser);
  }
                                                                        /*}}}*/
  void pkgAcqIndexMergeDiffs::Done(string Message,unsigned long long Size,HashStringList const &Hashes, /*{{{*/
  
     Item::Done(Message,Size,Hashes,Cnf);
  
 -   string const FinalFile = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 +   // FIXME: verify download before feeding it to rred
 +   string const FinalFile = GetPartialFileNameFromURI(RealURI);
  
     if (State == StateFetchDiff)
     {
+       FileFd fd(DestFile, FileFd::ReadOnly, FileFd::Gzip);
+       class Hashes LocalHashesCalc;
+       LocalHashesCalc.AddFD(fd);
+       HashStringList const LocalHashes = LocalHashesCalc.GetHashStringList();
+       if (fd.Size() != patch.patch_size || patch.patch_hashes != LocalHashes)
+       {
+        Failed("Patch has Size/Hashsum mismatch", NULL);
+        return;
+       }
        // rred expects the patch as $FinalFile.ed.$patchname.gz
        Rename(DestFile, FinalFile + ".ed." + patch.file + ".gz");
  
        Local = true;
        Desc.URI = "rred:" + FinalFile;
        QueueURI(Desc);
 -      ActiveSubprocess = "rred";
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic push
 -      #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
 -#endif
 -      Mode = "rred";
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic pop
 -#endif
 +      SetActiveSubprocess("rred");
        return;
     }
     // success in download/apply all diffs, clean up
         return;
        }
  
 +
 +      std::string FinalFile = _config->FindDir("Dir::State::lists");
 +      FinalFile += URItoFileName(RealURI);
 +
        // move the result into place
        if(Debug)
 -       std::clog << "Moving patched file in place: " << std::endl
 +       std::clog << "Queue patched file in place: " << std::endl
                   << DestFile << " -> " << FinalFile << std::endl;
 -      Rename(DestFile, FinalFile);
 -      chmod(FinalFile.c_str(), 0644);
  
 -      // otherwise lists cleanup will eat the file
 -      DestFile = FinalFile;
 +      // queue for copy by the transaction manager
 +      TransactionManager->TransactionStageCopy(this, DestFile, FinalFile);
  
        // ensure the ed's are gone regardless of list-cleanup
        for (std::vector<pkgAcqIndexMergeDiffs *>::const_iterator I = allPatches->begin();
            I != allPatches->end(); ++I)
        {
 -          std::string patch = FinalFile + ".ed." + (*I)->patch.file + ".gz";
 +            std::string const PartialFile = GetPartialFileNameFromURI(RealURI);
 +          std::string patch = PartialFile + ".ed." + (*I)->patch.file + ".gz";
 +            std::cerr << patch << std::endl;
            unlink(patch.c_str());
        }
  
     }
  }
                                                                        /*}}}*/
 +// AcqBaseIndex::VerifyHashByMetaKey - verify hash for the given metakey /*{{{*/
 +bool pkgAcqBaseIndex::VerifyHashByMetaKey(HashStringList const &Hashes)
 +{
 +   if(MetaKey != "" && Hashes.usable())
 +   {
 +      indexRecords::checkSum *Record = MetaIndexParser->Lookup(MetaKey);
 +      if(Record && Record->Hashes.usable() && Hashes != Record->Hashes)
 +      {
 +         printHashSumComparision(RealURI, Record->Hashes, Hashes);
 +         return false;
 +      }
 +   }
 +   return true;
 +}
 +                                                                      /*}}}*/
  // AcqIndex::AcqIndex - Constructor                                   /*{{{*/
  // ---------------------------------------------------------------------
 -/* The package file is added to the queue and a second class is 
 -   instantiated to fetch the revision file */   
 +/* The package file is added to the queue and a second class is
 +   instantiated to fetch the revision file */
  pkgAcqIndex::pkgAcqIndex(pkgAcquire *Owner,
                         string URI,string URIDesc,string ShortDesc,
 -                       HashStringList const  &ExpectedHash, string comprExt)
 -   : pkgAcqBaseIndex(Owner, NULL, ExpectedHash, NULL), RealURI(URI)
 +                       HashStringList const  &ExpectedHash)
 +   : pkgAcqBaseIndex(Owner, 0, NULL, ExpectedHash, NULL)
  {
 -   if(comprExt.empty() == true)
 -   {
 -      // autoselect the compression method
 -      std::vector<std::string> types = APT::Configuration::getCompressionTypes();
 -      for (std::vector<std::string>::const_iterator t = types.begin(); t != types.end(); ++t)
 -       comprExt.append(*t).append(" ");
 -      if (comprExt.empty() == false)
 -       comprExt.erase(comprExt.end()-1);
 -   }
 -   CompressionExtension = comprExt;
 +   RealURI = URI;
  
 +   AutoSelectCompression();
     Init(URI, URIDesc, ShortDesc);
 +
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "New pkgIndex with TransactionManager "
 +                << TransactionManager << std::endl;
  }
 -pkgAcqIndex::pkgAcqIndex(pkgAcquire *Owner, IndexTarget const *Target,
 -                       HashStringList const &ExpectedHash, 
 +                                                                      /*}}}*/
 +// AcqIndex::AcqIndex - Constructor                                   /*{{{*/
 +pkgAcqIndex::pkgAcqIndex(pkgAcquire *Owner,
 +                         pkgAcqMetaBase *TransactionManager,
 +                         IndexTarget const *Target,
 +                       HashStringList const &ExpectedHash,
                           indexRecords *MetaIndexParser)
 -   : pkgAcqBaseIndex(Owner, Target, ExpectedHash, MetaIndexParser), 
 -     RealURI(Target->URI)
 +   : pkgAcqBaseIndex(Owner, TransactionManager, Target, ExpectedHash,
 +                     MetaIndexParser)
  {
 +   RealURI = Target->URI;
 +
     // autoselect the compression method
 +   AutoSelectCompression();
 +   Init(Target->URI, Target->Description, Target->ShortDesc);
 +
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "New pkgIndex with TransactionManager "
 +                << TransactionManager << std::endl;
 +}
 +                                                                      /*}}}*/
 +// AcqIndex::AutoSelectCompression - Select compression                       /*{{{*/
 +void pkgAcqIndex::AutoSelectCompression()
 +{
     std::vector<std::string> types = APT::Configuration::getCompressionTypes();
 -   CompressionExtension = "";
 +   CompressionExtensions = "";
     if (ExpectedHashes.usable())
     {
 -      for (std::vector<std::string>::const_iterator t = types.begin(); t != types.end(); ++t)
 -       if (*t == "uncompressed" || MetaIndexParser->Exists(string(Target->MetaKey).append(".").append(*t)) == true)
 -          CompressionExtension.append(*t).append(" ");
 +      for (std::vector<std::string>::const_iterator t = types.begin();
 +           t != types.end(); ++t)
 +      {
 +         std::string CompressedMetaKey = string(Target->MetaKey).append(".").append(*t);
 +         if (*t == "uncompressed" ||
 +             MetaIndexParser->Exists(CompressedMetaKey) == true)
 +            CompressionExtensions.append(*t).append(" ");
 +      }
     }
     else
     {
        for (std::vector<std::string>::const_iterator t = types.begin(); t != types.end(); ++t)
 -       CompressionExtension.append(*t).append(" ");
 +       CompressionExtensions.append(*t).append(" ");
     }
 -   if (CompressionExtension.empty() == false)
 -      CompressionExtension.erase(CompressionExtension.end()-1);
 -
 -   Init(Target->URI, Target->Description, Target->ShortDesc);
 +   if (CompressionExtensions.empty() == false)
 +      CompressionExtensions.erase(CompressionExtensions.end()-1);
  }
                                                                        /*}}}*/
  // AcqIndex::Init - defered Constructor                                       /*{{{*/
 -void pkgAcqIndex::Init(string const &URI, string const &URIDesc, string const &ShortDesc) {
 -   Decompression = false;
 -   Erase = false;
 +void pkgAcqIndex::Init(string const &URI, string const &URIDesc,
 +                       string const &ShortDesc)
 +{
 +   Stage = STAGE_DOWNLOAD;
  
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(URI);
 +   DestFile = GetPartialFileNameFromURI(URI);
  
 -   std::string const comprExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
 -   std::string MetaKey;
 -   if (comprExt == "uncompressed")
 +   CurrentCompressionExtension = CompressionExtensions.substr(0, CompressionExtensions.find(' '));
 +   if (CurrentCompressionExtension == "uncompressed")
     {
        Desc.URI = URI;
        if(Target)
     }
     else
     {
 -      Desc.URI = URI + '.' + comprExt;
 -      DestFile = DestFile + '.' + comprExt;
 +      Desc.URI = URI + '.' + CurrentCompressionExtension;
 +      DestFile = DestFile + '.' + CurrentCompressionExtension;
        if(Target)
 -         MetaKey = string(Target->MetaKey) + '.' + comprExt;
 +         MetaKey = string(Target->MetaKey) + '.' + CurrentCompressionExtension;
     }
  
     // load the filesize
        indexRecords::checkSum *Record = MetaIndexParser->Lookup(MetaKey);
        if(Record)
           FileSize = Record->Size;
 -      
 +
        InitByHashIfNeeded(MetaKey);
     }
  
  }
                                                                        /*}}}*/
  // AcqIndex::AdjustForByHash - modify URI for by-hash support         /*{{{*/
 -// ---------------------------------------------------------------------
 -/* */
  void pkgAcqIndex::InitByHashIfNeeded(const std::string MetaKey)
  {
     // TODO:
  /* The only header we use is the last-modified header. */
  string pkgAcqIndex::Custom600Headers() const
  {
 -   std::string const compExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
 -   string Final = _config->FindDir("Dir::State::lists");
 -   Final += URItoFileName(RealURI);
 -   if (_config->FindB("Acquire::GzipIndexes",false))
 -      Final += compExt;
 -   
 -   string msg = "\nIndex-File: true";
 +   string Final = GetFinalFilename();
  
 +   string msg = "\nIndex-File: true";
     struct stat Buf;
     if (stat(Final.c_str(),&Buf) == 0)
        msg += "\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
     return msg;
  }
                                                                        /*}}}*/
 -void pkgAcqIndex::Failed(string Message,pkgAcquire::MethodConfig *Cnf)        /*{{{*/
 +// pkgAcqIndex::Failed - getting the indexfile failed                 /*{{{*/
 +void pkgAcqIndex::Failed(string Message,pkgAcquire::MethodConfig *Cnf)
  {
 -   size_t const nextExt = CompressionExtension.find(' ');
 +   size_t const nextExt = CompressionExtensions.find(' ');
     if (nextExt != std::string::npos)
     {
 -      CompressionExtension = CompressionExtension.substr(nextExt+1);
 +      CompressionExtensions = CompressionExtensions.substr(nextExt+1);
        Init(RealURI, Desc.Description, Desc.ShortDesc);
        return;
     }
  
     // on decompression failure, remove bad versions in partial/
 -   if (Decompression && Erase) {
 -      string s = _config->FindDir("Dir::State::lists") + "partial/";
 -      s.append(URItoFileName(RealURI));
 -      unlink(s.c_str());
 +   if (Stage == STAGE_DECOMPRESS_AND_VERIFY)
 +   {
 +      unlink(EraseFileName.c_str());
     }
  
     Item::Failed(Message,Cnf);
 +
 +   /// cancel the entire transaction
 +   TransactionManager->AbortTransaction();
  }
                                                                        /*}}}*/
 -// pkgAcqIndex::GetFinalFilename - Return the full final file path      /*{{{*/
 -std::string pkgAcqIndex::GetFinalFilename(std::string const &URI,
 -                                          std::string const &compExt)
 +// pkgAcqIndex::GetFinalFilename - Return the full final file path    /*{{{*/
 +std::string pkgAcqIndex::GetFinalFilename() const
  {
     std::string FinalFile = _config->FindDir("Dir::State::lists");
 -   FinalFile += URItoFileName(URI);
 +   FinalFile += URItoFileName(RealURI);
     if (_config->FindB("Acquire::GzipIndexes",false) == true)
 -      FinalFile += '.' + compExt;
 +      FinalFile += '.' + CurrentCompressionExtension;
     return FinalFile;
  }
                                                                        /*}}}*/
  // AcqIndex::ReverifyAfterIMS - Reverify index after an ims-hit               /*{{{*/
 -void pkgAcqIndex::ReverifyAfterIMS(std::string const &FileName)
 +void pkgAcqIndex::ReverifyAfterIMS()
  {
 -   std::string const compExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
 +   // update destfile to *not* include the compression extension when doing
 +   // a reverify (as its uncompressed on disk already)
 +   DestFile = GetPartialFileNameFromURI(RealURI);
 +
 +   // adjust DestFile if its compressed on disk
     if (_config->FindB("Acquire::GzipIndexes",false) == true)
 -      DestFile += compExt;
 +      DestFile += '.' + CurrentCompressionExtension;
  
 -   string FinalFile = GetFinalFilename(RealURI, compExt);
 -   Rename(FinalFile, FileName);
 -   Decompression = true;
 -   Desc.URI = "copy:" + FileName;
 +   // copy FinalFile into partial/ so that we check the hash again
 +   string FinalFile = GetFinalFilename();
 +   Stage = STAGE_DECOMPRESS_AND_VERIFY;
 +   Desc.URI = "copy:" + FinalFile;
     QueueURI(Desc);
  }
                                                                        /*}}}*/
 +// AcqIndex::ValidateFile - Validate the content of the downloaded file       /*{{{*/
 +bool pkgAcqIndex::ValidateFile(const std::string &FileName)
 +{
 +   // FIXME: this can go away once we only ever download stuff that
 +   //        has a valid hash and we never do GET based probing
 +   // FIXME2: this also leaks debian-isms into the code and should go therefore
 +
 +   /* Always validate the index file for correctness (all indexes must
 +    * have a Package field) (LP: #346386) (Closes: #627642) 
 +    */
 +   FileFd fd(FileName, FileFd::ReadOnly, FileFd::Extension);
 +   // Only test for correctness if the content of the file is not empty
 +   // (empty is ok)
 +   if (fd.Size() > 0)
 +   {
 +      pkgTagSection sec;
 +      pkgTagFile tag(&fd);
 +      
 +      // all our current indexes have a field 'Package' in each section
 +      if (_error->PendingError() == true ||
 +          tag.Step(sec) == false ||
 +          sec.Exists("Package") == false)
 +         return false;
 +   }
 +   return true;
 +}
 +                                                                      /*}}}*/
  // AcqIndex::Done - Finished a fetch                                  /*{{{*/
  // ---------------------------------------------------------------------
  /* This goes through a number of states.. On the initial fetch the
     method could possibly return an alternate filename which points
     to the uncompressed version of the file. If this is so the file
     is copied into the partial directory. In all other cases the file
 -   is decompressed with a gzip uri. */
 -void pkgAcqIndex::Done(string Message,unsigned long long Size,HashStringList const &Hashes,
 +   is decompressed with a compressed uri. */
 +void pkgAcqIndex::Done(string Message,
 +                       unsigned long long Size,
 +                       HashStringList const &Hashes,
                       pkgAcquire::MethodConfig *Cfg)
  {
     Item::Done(Message,Size,Hashes,Cfg);
 -   std::string const compExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
  
 -   if (Decompression == true)
 +   switch(Stage) 
     {
 -      if (ExpectedHashes.usable() && ExpectedHashes != Hashes)
 -      {
 -         Desc.URI = RealURI;
 -       RenameOnError(HashSumMismatch);
 -       printHashSumComparision(RealURI, ExpectedHashes, Hashes);
 -         return;
 -      }
 +      case STAGE_DOWNLOAD:
 +         StageDownloadDone(Message, Hashes, Cfg);
 +         break;
 +      case STAGE_DECOMPRESS_AND_VERIFY:
 +         StageDecompressDone(Message, Hashes, Cfg);
 +         break;
 +   }
 +}
 +                                                                      /*}}}*/
 +// AcqIndex::StageDownloadDone - Queue for decompress and verify      /*{{{*/
 +void pkgAcqIndex::StageDownloadDone(string Message,
 +                                    HashStringList const &Hashes,
 +                                    pkgAcquire::MethodConfig *Cfg)
 +{
 +   // First check if the calculcated Hash of the (compressed) downloaded
 +   // file matches the hash we have in the MetaIndexRecords for this file
 +   if(VerifyHashByMetaKey(Hashes) == false)
 +   {
 +      RenameOnError(HashSumMismatch);
 +      Failed(Message, Cfg);
 +      return;
 +   }
  
 -      // FIXME: this can go away once we only ever download stuff that
 -      //        has a valid hash and we never do GET based probing
 -      //
 -      /* Always verify the index file for correctness (all indexes must
 -       * have a Package field) (LP: #346386) (Closes: #627642) 
 -       */
 -      FileFd fd(DestFile, FileFd::ReadOnly, FileFd::Extension);
 -      // Only test for correctness if the file is not empty (empty is ok)
 -      if (fd.Size() > 0)
 -      {
 -         pkgTagSection sec;
 -         pkgTagFile tag(&fd);
 -         
 -         // all our current indexes have a field 'Package' in each section
 -         if (_error->PendingError() == true || tag.Step(sec) == false || sec.Exists("Package") == false)
 -         {
 -            RenameOnError(InvalidFormat);
 -            return;
 -         }
 -      }
 -       
 -      // Done, move it into position
 -      string FinalFile = GetFinalFilename(RealURI, compExt);
 -      Rename(DestFile,FinalFile);
 -      chmod(FinalFile.c_str(),0644);
 +   Complete = true;
  
 -      /* We restore the original name to DestFile so that the clean operation
 -         will work OK */
 -      DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -      DestFile += URItoFileName(RealURI);
 -      if (_config->FindB("Acquire::GzipIndexes",false))
 -         DestFile += '.' + compExt;
 -
 -      // Remove the compressed version.
 -      if (Erase == true)
 -       unlink(DestFile.c_str());
 -
 -      return;
 -   }
 -
 -   Erase = false;
 -   Complete = true;
 -   
     // Handle the unzipd case
     string FileName = LookupTag(Message,"Alt-Filename");
     if (FileName.empty() == false)
     {
 -      Decompression = true;
 +      Stage = STAGE_DECOMPRESS_AND_VERIFY;
        Local = true;
        DestFile += ".decomp";
        Desc.URI = "copy:" + FileName;
        QueueURI(Desc);
 -      ActiveSubprocess = "copy";
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic push
 -      #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
 -#endif
 -      Mode = "copy";
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic pop
 -#endif
 +      SetActiveSubprocess("copy");
        return;
     }
  
        ErrorText = "Method gave a blank filename";
     }
  
 -   if (FileName == DestFile)
 -      Erase = true;
 -   else
 +   // Methods like e.g. "file:" will give us a (compressed) FileName that is
 +   // not the "DestFile" we set, in this case we uncompress from the local file
 +   if (FileName != DestFile)
        Local = true;
 +   else
 +      EraseFileName = FileName;
  
 -   // do not reverify cdrom sources as apt-cdrom may rewrite the Packages
 -   // file when its doing the indexcopy
 -   if (RealURI.substr(0,6) == "cdrom:" &&
 -       StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
 -      return;
 -
 -   // The files timestamp matches, for non-local URLs reverify the local
 -   // file, for local file, uncompress again to ensure the hashsum is still
 -   // matching the Release file
 -   if (!Local && StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
 +   // we need to verify the file against the current Release file again
 +   // on if-modfied-since hit to avoid a stale attack against us
 +   if(StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
     {
 -      // set destfile to the final destfile
 -      if(_config->FindB("Acquire::GzipIndexes",false) == false)
 -      {
 -         DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -         DestFile += URItoFileName(RealURI);
 -      }
 +      // do not reverify cdrom sources as apt-cdrom may rewrite the Packages
 +      // file when its doing the indexcopy
 +      if (RealURI.substr(0,6) == "cdrom:")
 +         return;
  
 -      ReverifyAfterIMS(FileName);
 +      // The files timestamp matches, reverify by copy into partial/
 +      EraseFileName = "";
 +      ReverifyAfterIMS();
        return;
     }
 -   string decompProg;
  
 -   // If we enable compressed indexes, queue for hash verification
 +   // If we have compressed indexes enabled, queue for hash verification
     if (_config->FindB("Acquire::GzipIndexes",false))
     {
 -      DestFile = _config->FindDir("Dir::State::lists");
 -      DestFile += URItoFileName(RealURI) + '.' + compExt;
 -
 -      Decompression = true;
 +      DestFile = GetPartialFileNameFromURI(RealURI + '.' + CurrentCompressionExtension);
 +      EraseFileName = "";
 +      Stage = STAGE_DECOMPRESS_AND_VERIFY;
        Desc.URI = "copy:" + FileName;
        QueueURI(Desc);
 -
 +      SetActiveSubprocess("copy");
        return;
      }
  
     // get the binary name for your used compression type
 -   decompProg = _config->Find(string("Acquire::CompressionTypes::").append(compExt),"");
 -   if(decompProg.empty() == false);
 -   else if(compExt == "uncompressed")
 +   string decompProg;
 +   if(CurrentCompressionExtension == "uncompressed")
        decompProg = "copy";
 -   else {
 -      _error->Error("Unsupported extension: %s", compExt.c_str());
 +   else
 +      decompProg = _config->Find(string("Acquire::CompressionTypes::").append(CurrentCompressionExtension),"");
 +   if(decompProg.empty() == true)
 +   {
 +      _error->Error("Unsupported extension: %s", CurrentCompressionExtension.c_str());
        return;
     }
  
 -   Decompression = true;
 +   // queue uri for the next stage
 +   Stage = STAGE_DECOMPRESS_AND_VERIFY;
     DestFile += ".decomp";
     Desc.URI = decompProg + ":" + FileName;
     QueueURI(Desc);
 +   SetActiveSubprocess(decompProg);
 +}
 +                                                                      /*}}}*/
 +// pkgAcqIndex::StageDecompressDone - Final verification              /*{{{*/
 +void pkgAcqIndex::StageDecompressDone(string Message,
 +                                      HashStringList const &Hashes,
 +                                      pkgAcquire::MethodConfig *Cfg)
 +{
 +   if (ExpectedHashes.usable() && ExpectedHashes != Hashes)
 +   {
 +      Desc.URI = RealURI;
 +      RenameOnError(HashSumMismatch);
 +      printHashSumComparision(RealURI, ExpectedHashes, Hashes);
 +      Failed(Message, Cfg);
 +      return;
 +   }
  
 -   ActiveSubprocess = decompProg;
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic push
 -      #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
 -#endif
 -      Mode = ActiveSubprocess.c_str();
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic pop
 -#endif
 +   if(!ValidateFile(DestFile))
 +   {
 +      RenameOnError(InvalidFormat);
 +      Failed(Message, Cfg);
 +      return;
 +   }
 +
 +   // remove the compressed version of the file
 +   unlink(EraseFileName.c_str());
 +
 +   // Done, queue for rename on transaction finished
 +   TransactionManager->TransactionStageCopy(this, DestFile, GetFinalFilename());
 +
 +   return;
  }
                                                                        /*}}}*/
  // AcqIndexTrans::pkgAcqIndexTrans - Constructor                      /*{{{*/
  // ---------------------------------------------------------------------
  /* The Translation file is added to the queue */
  pkgAcqIndexTrans::pkgAcqIndexTrans(pkgAcquire *Owner,
 -                          string URI,string URIDesc,string ShortDesc) 
 -  : pkgAcqIndex(Owner, URI, URIDesc, ShortDesc, HashStringList(), "")
 +                          string URI,string URIDesc,string ShortDesc)
 +  : pkgAcqIndex(Owner, URI, URIDesc, ShortDesc, HashStringList())
  {
  }
 -pkgAcqIndexTrans::pkgAcqIndexTrans(pkgAcquire *Owner, IndexTarget const * const Target,
 -                       HashStringList const &ExpectedHashes, indexRecords *MetaIndexParser)
 -  : pkgAcqIndex(Owner, Target, ExpectedHashes, MetaIndexParser)
 +pkgAcqIndexTrans::pkgAcqIndexTrans(pkgAcquire *Owner,
 +                                   pkgAcqMetaBase *TransactionManager,
 +                                   IndexTarget const * const Target,
 +                                   HashStringList const &ExpectedHashes,
 +                                   indexRecords *MetaIndexParser)
 +   : pkgAcqIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser)
  {
     // load the filesize
     indexRecords::checkSum *Record = MetaIndexParser->Lookup(string(Target->MetaKey));
  }
                                                                        /*}}}*/
  // AcqIndexTrans::Custom600Headers - Insert custom request headers    /*{{{*/
 -// ---------------------------------------------------------------------
  string pkgAcqIndexTrans::Custom600Headers() const
  {
 -   std::string const compExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
 -   string Final = _config->FindDir("Dir::State::lists");
 -   Final += URItoFileName(RealURI);
 -   if (_config->FindB("Acquire::GzipIndexes",false))
 -      Final += compExt;
 +   string Final = GetFinalFilename();
  
     struct stat Buf;
     if (stat(Final.c_str(),&Buf) != 0)
  }
                                                                        /*}}}*/
  // AcqIndexTrans::Failed - Silence failure messages for missing files /*{{{*/
 -// ---------------------------------------------------------------------
 -/* */
  void pkgAcqIndexTrans::Failed(string Message,pkgAcquire::MethodConfig *Cnf)
  {
 -   size_t const nextExt = CompressionExtension.find(' ');
 +   size_t const nextExt = CompressionExtensions.find(' ');
     if (nextExt != std::string::npos)
     {
 -      CompressionExtension = CompressionExtension.substr(nextExt+1);
 +      CompressionExtensions = CompressionExtensions.substr(nextExt+1);
        Init(RealURI, Desc.Description, Desc.ShortDesc);
        Status = StatIdle;
        return;
     }
  
 -   if (Cnf->LocalOnly == true || 
 +   Item::Failed(Message,Cnf);
 +
 +   // FIXME: this is used often (e.g. in pkgAcqIndexTrans) so refactor
 +   if (Cnf->LocalOnly == true ||
         StringToBool(LookupTag(Message,"Transient-Failure"),false) == false)
 -   {      
 +   {
        // Ignore this
        Status = StatDone;
 -      Complete = false;
 -      Dequeue();
 -      return;
     }
 +}
 +                                                                      /*}}}*/
 +// AcqMetaBase::Add - Add a item to the current Transaction           /*{{{*/
 +void pkgAcqMetaBase::Add(Item *I)
 +{
 +   Transaction.push_back(I);
 +}
 +                                                                      /*}}}*/
 +// AcqMetaBase::AbortTransaction - Abort the current Transaction      /*{{{*/
 +void pkgAcqMetaBase::AbortTransaction()
 +{
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "AbortTransaction: " << TransactionManager << std::endl;
  
 -   Item::Failed(Message,Cnf);
 +   // ensure the toplevel is in error state too
 +   for (std::vector<Item*>::iterator I = Transaction.begin();
 +        I != Transaction.end(); ++I)
 +   {
 +      if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +         std::clog << "  Cancel: " << (*I)->DestFile << std::endl;
 +      // the transaction will abort, so stop anything that is idle
 +      if ((*I)->Status == pkgAcquire::Item::StatIdle)
 +         (*I)->Status = pkgAcquire::Item::StatDone;
 +
 +      // kill files in partial
 +      std::string const PartialFile = GetPartialFileName(flNotDir((*I)->DestFile));
 +      if(FileExists(PartialFile))
 +         Rename(PartialFile, PartialFile + ".FAILED");
 +   }
 +}
 +                                                                      /*}}}*/
 +// AcqMetaBase::TransactionHasError - Check for errors in Transaction /*{{{*/
 +bool pkgAcqMetaBase::TransactionHasError()
 +{
 +   for (pkgAcquire::ItemIterator I = Transaction.begin();
 +        I != Transaction.end(); ++I)
 +      if((*I)->Status != pkgAcquire::Item::StatDone &&
 +         (*I)->Status != pkgAcquire::Item::StatIdle)
 +         return true;
 +
 +   return false;
 +}
 +                                                                      /*}}}*/
 +// AcqMetaBase::CommitTransaction - Commit a transaction              /*{{{*/
 +void pkgAcqMetaBase::CommitTransaction()
 +{
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "CommitTransaction: " << this << std::endl;
 +
 +   // move new files into place *and* remove files that are not
 +   // part of the transaction but are still on disk
 +   for (std::vector<Item*>::iterator I = Transaction.begin();
 +        I != Transaction.end(); ++I)
 +   {
 +      if((*I)->PartialFile != "")
 +      {
 +       if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +          std::clog << "mv " << (*I)->PartialFile << " -> "<< (*I)->DestFile << " "
 +             << (*I)->DescURI() << std::endl;
 +
 +       Rename((*I)->PartialFile, (*I)->DestFile);
 +       ChangeOwnerAndPermissionOfFile("CommitTransaction", (*I)->DestFile.c_str(), "root", "root", 0644);
 +
 +      } else {
 +         if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +            std::clog << "rm "
 +                      <<  (*I)->DestFile
 +                      << " "
 +                      << (*I)->DescURI()
 +                      << std::endl;
 +         unlink((*I)->DestFile.c_str());
 +      }
 +      // mark that this transaction is finished
 +      (*I)->TransactionManager = 0;
 +   }
 +}
 +                                                                      /*}}}*/
 +// AcqMetaBase::TransactionStageCopy - Stage a file for copying               /*{{{*/
 +void pkgAcqMetaBase::TransactionStageCopy(Item *I,
 +                                          const std::string &From,
 +                                          const std::string &To)
 +{
 +   I->PartialFile = From;
 +   I->DestFile = To;
 +}
 +                                                                      /*}}}*/
 +// AcqMetaBase::TransactionStageRemoval - Sage a file for removal     /*{{{*/
 +void pkgAcqMetaBase::TransactionStageRemoval(Item *I,
 +                                             const std::string &FinalFile)
 +{
 +   I->PartialFile = "";
 +   I->DestFile = FinalFile; 
  }
                                                                        /*}}}*/
 -pkgAcqMetaSig::pkgAcqMetaSig(pkgAcquire *Owner,                               /*{{{*/
 +// AcqMetaBase::GenerateAuthWarning - Check gpg authentication error  /*{{{*/
 +bool pkgAcqMetaBase::CheckStopAuthentication(const std::string &RealURI,
 +                                             const std::string &Message)
 +{
 +   // FIXME: this entire function can do now that we disallow going to
 +   //        a unauthenticated state and can cleanly rollback
 +
 +   string Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 +
 +   if(FileExists(Final))
 +   {
 +      Status = StatTransientNetworkError;
 +      _error->Warning(_("An error occurred during the signature "
 +                        "verification. The repository is not updated "
 +                        "and the previous index files will be used. "
 +                        "GPG error: %s: %s\n"),
 +                      Desc.Description.c_str(),
 +                      LookupTag(Message,"Message").c_str());
 +      RunScripts("APT::Update::Auth-Failure");
 +      return true;
 +   } else if (LookupTag(Message,"Message").find("NODATA") != string::npos) {
 +      /* Invalid signature file, reject (LP: #346386) (Closes: #627642) */
 +      _error->Error(_("GPG error: %s: %s"),
 +                    Desc.Description.c_str(),
 +                    LookupTag(Message,"Message").c_str());
 +      Status = StatError;
 +      return true;
 +   } else {
 +      _error->Warning(_("GPG error: %s: %s"),
 +                      Desc.Description.c_str(),
 +                      LookupTag(Message,"Message").c_str());
 +   }
 +   // gpgv method failed
 +   ReportMirrorFailure("GPGFailure");
 +   return false;
 +}
 +                                                                      /*}}}*/
 +// AcqMetaSig::AcqMetaSig - Constructor                                       /*{{{*/
 +pkgAcqMetaSig::pkgAcqMetaSig(pkgAcquire *Owner,
 +                             pkgAcqMetaBase *TransactionManager,
                             string URI,string URIDesc,string ShortDesc,
 -                           string MetaIndexURI, string MetaIndexURIDesc,
 -                           string MetaIndexShortDesc,
 +                             string MetaIndexFile,
                             const vector<IndexTarget*>* IndexTargets,
                             indexRecords* MetaIndexParser) :
 -   Item(Owner, HashStringList()), RealURI(URI), MetaIndexURI(MetaIndexURI),
 -   MetaIndexURIDesc(MetaIndexURIDesc), MetaIndexShortDesc(MetaIndexShortDesc),
 -   MetaIndexParser(MetaIndexParser), IndexTargets(IndexTargets)
 +   pkgAcqMetaBase(Owner, IndexTargets, MetaIndexParser,
 +                  HashStringList(), TransactionManager),
 +   RealURI(URI), MetaIndexFile(MetaIndexFile), URIDesc(URIDesc),
 +   ShortDesc(ShortDesc)
  {
     DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(URI);
 +   DestFile += URItoFileName(RealURI);
  
 -   // remove any partial downloaded sig-file in partial/. 
 -   // it may confuse proxies and is too small to warrant a 
 +   // remove any partial downloaded sig-file in partial/.
 +   // it may confuse proxies and is too small to warrant a
     // partial download anyway
     unlink(DestFile.c_str());
  
 +   // set the TransactionManager
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "New pkgAcqMetaSig with TransactionManager "
 +                << TransactionManager << std::endl;
 +
     // Create the item
     Desc.Description = URIDesc;
     Desc.Owner = this;
     Desc.ShortDesc = ShortDesc;
     Desc.URI = URI;
 -      
 -   string Final = _config->FindDir("Dir::State::lists");
 -   Final += URItoFileName(RealURI);
 -   if (RealFileExists(Final) == true)
 -   {
 -      // File was already in place.  It needs to be re-downloaded/verified
 -      // because Release might have changed, we do give it a different
 -      // name than DestFile because otherwise the http method will
 -      // send If-Range requests and there are too many broken servers
 -      // out there that do not understand them
 -      LastGoodSig = DestFile+".reverify";
 -      Rename(Final,LastGoodSig);
 -   }
 -
 -   // we expect the indextargets + one additional Release file
 -   ExpectedAdditionalItems = IndexTargets->size() + 1;
  
     QueueURI(Desc);
  }
                                                                        /*}}}*/
  pkgAcqMetaSig::~pkgAcqMetaSig()                                               /*{{{*/
  {
 -   // if the file was never queued undo file-changes done in the constructor
 -   if (QueueCounter == 1 && Status == StatIdle && FileSize == 0 && Complete == false &&
 -       LastGoodSig.empty() == false)
 -   {
 -      string const Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 -      if (RealFileExists(Final) == false && RealFileExists(LastGoodSig) == true)
 -       Rename(LastGoodSig, Final);
 -   }
 -
  }
                                                                        /*}}}*/
  // pkgAcqMetaSig::Custom600Headers - Insert custom request headers    /*{{{*/
  // ---------------------------------------------------------------------
 -/* The only header we use is the last-modified header. */
  string pkgAcqMetaSig::Custom600Headers() const
  {
 +   string FinalFile = _config->FindDir("Dir::State::lists");
 +   FinalFile += URItoFileName(RealURI);
 +
     struct stat Buf;
 -   if (stat(LastGoodSig.c_str(),&Buf) != 0)
 +   if (stat(FinalFile.c_str(),&Buf) != 0)
        return "\nIndex-File: true";
  
     return "\nIndex-File: true\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
  }
 -
 -void pkgAcqMetaSig::Done(string Message,unsigned long long Size, HashStringList const &Hashes,
 +                                                                      /*}}}*/
 +// pkgAcqMetaSig::Done - The signature was downloaded/verified                /*{{{*/
 +// ---------------------------------------------------------------------
 +/* The only header we use is the last-modified header. */
 +void pkgAcqMetaSig::Done(string Message,unsigned long long Size,
 +                         HashStringList const &Hashes,
                         pkgAcquire::MethodConfig *Cfg)
  {
     Item::Done(Message, Size, Hashes, Cfg);
  
 -   string FileName = LookupTag(Message,"Filename");
 -   if (FileName.empty() == true)
 +   if(AuthPass == false)
     {
 -      Status = StatError;
 -      ErrorText = "Method gave a blank filename";
 +      if(CheckDownloadDone(Message, RealURI) == true)
 +      {
 +         // destfile will be modified to point to MetaIndexFile for the
 +         // gpgv method, so we need to save it here
 +         MetaIndexFileSignature = DestFile;
 +         QueueForSignatureVerify(MetaIndexFile, MetaIndexFileSignature);
 +      }
        return;
     }
 -
 -   if (FileName != DestFile)
 +   else
     {
 -      // We have to copy it into place
 -      Local = true;
 -      Desc.URI = "copy:" + FileName;
 -      QueueURI(Desc);
 -      return;
 +      if(CheckAuthDone(Message, RealURI) == true)
 +      {
 +         std::string FinalFile = _config->FindDir("Dir::State::lists");
 +         FinalFile += URItoFileName(RealURI);
 +         TransactionManager->TransactionStageCopy(this, MetaIndexFileSignature, FinalFile);
 +      }
     }
 -
 -   Complete = true;
 -
 -   // at this point pkgAcqMetaIndex takes over
 -   ExpectedAdditionalItems = 0;
 -
 -   // put the last known good file back on i-m-s hit (it will
 -   // be re-verified again)
 -   // Else do nothing, we have the new file in DestFile then
 -   if(StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
 -      Rename(LastGoodSig, DestFile);
 -
 -   // queue a pkgAcqMetaIndex to be verified against the sig we just retrieved
 -   new pkgAcqMetaIndex(Owner, MetaIndexURI, MetaIndexURIDesc, 
 -                     MetaIndexShortDesc,  DestFile, IndexTargets, 
 -                     MetaIndexParser);
 -
  }
                                                                        /*}}}*/
  void pkgAcqMetaSig::Failed(string Message,pkgAcquire::MethodConfig *Cnf)/*{{{*/
  {
     string Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
  
 -   // at this point pkgAcqMetaIndex takes over
 -   ExpectedAdditionalItems = 0;
 +   // check if we need to fail at this point 
 +   if (AuthPass == true && CheckStopAuthentication(RealURI, Message))
 +         return;
  
 -   // if we get a network error we fail gracefully
 -   if(Status == StatTransientNetworkError)
 -   {
 -      Item::Failed(Message,Cnf);
 -      // move the sigfile back on transient network failures 
 -      if(FileExists(LastGoodSig))
 -       Rename(LastGoodSig,Final);
 +   // FIXME: meh, this is not really elegant
 +   string InReleaseURI = RealURI.replace(RealURI.rfind("Release.gpg"), 12,
 +                                         "InRelease");
 +   string FinalInRelease = _config->FindDir("Dir::State::lists") + URItoFileName(InReleaseURI);
  
 -      // set the status back to , Item::Failed likes to reset it
 -      Status = pkgAcquire::Item::StatTransientNetworkError;
 -      return;
 +   if (RealFileExists(Final) || RealFileExists(FinalInRelease))
 +   {
 +      std::string downgrade_msg;
 +      strprintf(downgrade_msg, _("The repository '%s' is no longer signed."),
 +                URIDesc.c_str());
 +      if(_config->FindB("Acquire::AllowDowngradeToInsecureRepositories"))
 +      {
 +         // meh, the users wants to take risks (we still mark the packages
 +         // from this repository as unauthenticated)
 +         _error->Warning("%s", downgrade_msg.c_str());
 +         _error->Warning(_("This is normally not allowed, but the option "
 +                           "Acquire::AllowDowngradeToInsecureRepositories was "
 +                           "given to override it."));
 +         
 +      } else {
 +         _error->Error("%s", downgrade_msg.c_str());
 +         Rename(MetaIndexFile, MetaIndexFile+".FAILED");
 +       Item::Failed("Message: " + downgrade_msg, Cnf);
 +         TransactionManager->AbortTransaction();
 +         return;
 +      }
     }
  
 -   // Delete any existing sigfile when the acquire failed
 -   unlink(Final.c_str());
 +   // this ensures that any file in the lists/ dir is removed by the
 +   // transaction
 +   DestFile = GetPartialFileNameFromURI(RealURI);
 +   TransactionManager->TransactionStageRemoval(this, DestFile);
  
 -   // queue a pkgAcqMetaIndex with no sigfile
 -   new pkgAcqMetaIndex(Owner, MetaIndexURI, MetaIndexURIDesc, MetaIndexShortDesc,
 -                     "", IndexTargets, MetaIndexParser);
 +   // only allow going further if the users explicitely wants it
 +   if(_config->FindB("Acquire::AllowInsecureRepositories") == true)
 +   {
 +      // we parse the indexes here because at this point the user wanted
 +      // a repository that may potentially harm him
 +      MetaIndexParser->Load(MetaIndexFile);
 +      QueueIndexes(true);
 +   } 
 +   else 
 +   {
 +      _error->Warning("Use --allow-insecure-repositories to force the update");
 +   }
  
 -   if (Cnf->LocalOnly == true || 
 +   Item::Failed(Message,Cnf);
 +
 +   // FIXME: this is used often (e.g. in pkgAcqIndexTrans) so refactor
 +   if (Cnf->LocalOnly == true ||
         StringToBool(LookupTag(Message,"Transient-Failure"),false) == false)
 -   {      
 +   {
        // Ignore this
        Status = StatDone;
 -      Complete = false;
 -      Dequeue();
 -      return;
     }
 -   
 -   Item::Failed(Message,Cnf);
  }
                                                                        /*}}}*/
  pkgAcqMetaIndex::pkgAcqMetaIndex(pkgAcquire *Owner,                   /*{{{*/
 +                                 pkgAcqMetaBase *TransactionManager,
                                 string URI,string URIDesc,string ShortDesc,
 -                               string SigFile,
 +                                 string MetaIndexSigURI,string MetaIndexSigURIDesc, string MetaIndexSigShortDesc,
                                 const vector<IndexTarget*>* IndexTargets,
                                 indexRecords* MetaIndexParser) :
 -   Item(Owner, HashStringList()), RealURI(URI), SigFile(SigFile), IndexTargets(IndexTargets),
 -   MetaIndexParser(MetaIndexParser), AuthPass(false), IMSHit(false)
 +   pkgAcqMetaBase(Owner, IndexTargets, MetaIndexParser, HashStringList(),
 +                  TransactionManager), 
 +   RealURI(URI), URIDesc(URIDesc), ShortDesc(ShortDesc),
 +   MetaIndexSigURI(MetaIndexSigURI), MetaIndexSigURIDesc(MetaIndexSigURIDesc),
 +   MetaIndexSigShortDesc(MetaIndexSigShortDesc)
  {
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(URI);
 +   if(TransactionManager == NULL)
 +   {
 +      this->TransactionManager = this;
 +      this->TransactionManager->Add(this);
 +   }
 +
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "New pkgAcqMetaIndex with TransactionManager "
 +                << this->TransactionManager << std::endl;
 +
 +
 +   Init(URIDesc, ShortDesc);
 +}
 +                                                                      /*}}}*/
 +// pkgAcqMetaIndex::Init - Delayed constructor                                /*{{{*/
 +void pkgAcqMetaIndex::Init(std::string URIDesc, std::string ShortDesc)
 +{
 +   DestFile = GetPartialFileNameFromURI(RealURI);
  
     // Create the item
     Desc.Description = URIDesc;
     Desc.Owner = this;
     Desc.ShortDesc = ShortDesc;
 -   Desc.URI = URI;
 +   Desc.URI = RealURI;
  
     // we expect more item
     ExpectedAdditionalItems = IndexTargets->size();
 -
     QueueURI(Desc);
  }
                                                                        /*}}}*/
  // pkgAcqMetaIndex::Custom600Headers - Insert custom request headers  /*{{{*/
  // ---------------------------------------------------------------------
 -/* The only header we use is the last-modified header. */
  string pkgAcqMetaIndex::Custom600Headers() const
  {
     string Final = _config->FindDir("Dir::State::lists");
     return "\nIndex-File: true\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
  }
                                                                        /*}}}*/
 -void pkgAcqMetaIndex::Done(string Message,unsigned long long Size,HashStringList const &Hashes,       /*{{{*/
 +void pkgAcqMetaIndex::Done(string Message,unsigned long long Size,    /*{{{*/
 +                           HashStringList const &Hashes,
                           pkgAcquire::MethodConfig *Cfg)
  {
     Item::Done(Message,Size,Hashes,Cfg);
  
 -   // MetaIndexes are done in two passes: one to download the
 -   // metaindex with an appropriate method, and a second to verify it
 -   // with the gpgv method
 -
 -   if (AuthPass == true)
 +   if(CheckDownloadDone(Message, RealURI))
     {
 -      AuthDone(Message);
 +      // we have a Release file, now download the Signature, all further
 +      // verify/queue for additional downloads will be done in the
 +      // pkgAcqMetaSig::Done() code
 +      std::string MetaIndexFile = DestFile;
 +      new pkgAcqMetaSig(Owner, TransactionManager, 
 +                        MetaIndexSigURI, MetaIndexSigURIDesc,
 +                        MetaIndexSigShortDesc, MetaIndexFile, IndexTargets, 
 +                        MetaIndexParser);
  
 -      // all cool, move Release file into place
 -      Complete = true;
 +      string FinalFile = _config->FindDir("Dir::State::lists");
 +      FinalFile += URItoFileName(RealURI);
 +      TransactionManager->TransactionStageCopy(this, DestFile, FinalFile);
     }
 -   else
 -   {
 -      RetrievalDone(Message);
 -      if (!Complete)
 -         // Still more retrieving to do
 -         return;
 -
 -      if (SigFile == "")
 -      {
 -         // There was no signature file, so we are finished.  Download
 -         // the indexes and do only hashsum verification if possible
 -         MetaIndexParser->Load(DestFile);
 -         QueueIndexes(false);
 -      }
 -      else
 -      {
 -         // FIXME: move this into pkgAcqMetaClearSig::Done on the next
 -         //        ABI break
 -
 -         // if we expect a ClearTextSignature (InRelase), ensure that
 -         // this is what we get and if not fail to queue a 
 -         // Release/Release.gpg, see #346386
 -         if (SigFile == DestFile && !StartsWithGPGClearTextSignature(DestFile))
 -         {
 -            Failed(Message, Cfg);
 -            return;
 -         }
 +}
 +                                                                      /*}}}*/
 +bool pkgAcqMetaBase::CheckAuthDone(string Message, const string &RealURI)     /*{{{*/
 +{
 +   // At this point, the gpgv method has succeeded, so there is a
 +   // valid signature from a key in the trusted keyring.  We
 +   // perform additional verification of its contents, and use them
 +   // to verify the indexes we are about to download
  
 -         // There was a signature file, so pass it to gpgv for
 -         // verification
 -         if (_config->FindB("Debug::pkgAcquire::Auth", false))
 -            std::cerr << "Metaindex acquired, queueing gpg verification ("
 -                      << SigFile << "," << DestFile << ")\n";
 -         AuthPass = true;
 -         Desc.URI = "gpgv:" + SigFile;
 -         QueueURI(Desc);
 -       ActiveSubprocess = "gpgv";
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic push
 -      #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
 -#endif
 -       Mode = "gpgv";
 -#if __GNUC__ >= 4
 -      #pragma GCC diagnostic pop
 -#endif
 -       return;
 -      }
 +   if (!MetaIndexParser->Load(DestFile))
 +   {
 +      Status = StatAuthError;
 +      ErrorText = MetaIndexParser->ErrorText;
 +      return false;
     }
  
 -   if (Complete == true)
 +   if (!VerifyVendor(Message, RealURI))
     {
 -      string FinalFile = _config->FindDir("Dir::State::lists");
 -      FinalFile += URItoFileName(RealURI);
 -      if (SigFile == DestFile)
 -       SigFile = FinalFile;
 -      Rename(DestFile,FinalFile);
 -      chmod(FinalFile.c_str(),0644);
 -      DestFile = FinalFile;
 +      return false;
     }
 +
 +   if (_config->FindB("Debug::pkgAcquire::Auth", false))
 +      std::cerr << "Signature verification succeeded: "
 +                << DestFile << std::endl;
 +
 +   // Download further indexes with verification 
 +   //
 +   // it would be really nice if we could simply do
 +   //    if (IMSHit == false) QueueIndexes(true)
 +   // and skip the download if the Release file has not changed
 +   // - but right now the list cleaner will needs to be tricked
 +   //   to not delete all our packages/source indexes in this case
 +   QueueIndexes(true);
 +
 +   return true;
  }
                                                                        /*}}}*/
 -void pkgAcqMetaIndex::RetrievalDone(string Message)                   /*{{{*/
 +// pkgAcqMetaBase::QueueForSignatureVerify                            /*{{{*/
 +void pkgAcqMetaBase::QueueForSignatureVerify(const std::string &MetaIndexFile,
 +                                    const std::string &MetaIndexFileSignature)
 +{
 +   AuthPass = true;
 +   Desc.URI = "gpgv:" + MetaIndexFileSignature;
 +   DestFile = MetaIndexFile;
 +   QueueURI(Desc);
 +   SetActiveSubprocess("gpgv");
 +}
 +                                                                      /*}}}*/
 +// pkgAcqMetaBase::CheckDownloadDone                                  /*{{{*/
 +bool pkgAcqMetaBase::CheckDownloadDone(const std::string &Message,
 +                                       const std::string &RealURI)
  {
     // We have just finished downloading a Release file (it is not
     // verified yet)
     {
        Status = StatError;
        ErrorText = "Method gave a blank filename";
 -      return;
 +      return false;
     }
  
     if (FileName != DestFile)
        Local = true;
        Desc.URI = "copy:" + FileName;
        QueueURI(Desc);
 -      return;
 +      return false;
     }
  
     // make sure to verify against the right file on I-M-S hit
     {
        string FinalFile = _config->FindDir("Dir::State::lists");
        FinalFile += URItoFileName(RealURI);
 -      if (SigFile == DestFile)
 -      {
 -       SigFile = FinalFile;
 -       // constructor of pkgAcqMetaClearSig moved it out of the way,
 -       // now move it back in on IMS hit for the 'old' file
 -       string const OldClearSig = DestFile + ".reverify";
 -       if (RealFileExists(OldClearSig) == true)
 -          Rename(OldClearSig, FinalFile);
 -      }
        DestFile = FinalFile;
     }
 -   Complete = true;
 -}
 -                                                                      /*}}}*/
 -void pkgAcqMetaIndex::AuthDone(string Message)                                /*{{{*/
 -{
 -   // At this point, the gpgv method has succeeded, so there is a
 -   // valid signature from a key in the trusted keyring.  We
 -   // perform additional verification of its contents, and use them
 -   // to verify the indexes we are about to download
 -
 -   if (!MetaIndexParser->Load(DestFile))
 -   {
 -      Status = StatAuthError;
 -      ErrorText = MetaIndexParser->ErrorText;
 -      return;
 -   }
 -
 -   if (!VerifyVendor(Message))
 -   {
 -      return;
 -   }
 -
 -   if (_config->FindB("Debug::pkgAcquire::Auth", false))
 -      std::cerr << "Signature verification succeeded: "
 -                << DestFile << std::endl;
 -
 -   // do not trust any previously unverified content that we may have
 -   string LastGoodSigFile = _config->FindDir("Dir::State::lists").append("partial/").append(URItoFileName(RealURI));
 -   if (DestFile != SigFile)
 -      LastGoodSigFile.append(".gpg");
 -   LastGoodSigFile.append(".reverify");
 -   if(IMSHit == false && RealFileExists(LastGoodSigFile) == false)
 -   {
 -      for (vector <struct IndexTarget*>::const_iterator Target = IndexTargets->begin();
 -           Target != IndexTargets->end();
 -           ++Target)
 -      {
 -         // remove old indexes
 -         std::string index = _config->FindDir("Dir::State::lists") +
 -            URItoFileName((*Target)->URI);
 -         unlink(index.c_str());
 -         // and also old gzipindexes
 -         std::vector<std::string> types = APT::Configuration::getCompressionTypes();
 -         for (std::vector<std::string>::const_iterator t = types.begin(); t != types.end(); ++t)
 -         {
 -            index += '.' + (*t);
 -            unlink(index.c_str());
 -         }
 -      }
 -   }
 -
 -
 -   // Download further indexes with verification
 -   QueueIndexes(true);
  
 -   // is it a clearsigned MetaIndex file?
 -   if (DestFile == SigFile)
 -      return;
 +   // set Item to complete as the remaining work is all local (verify etc)
 +   Complete = true;
  
 -   // Done, move signature file into position
 -   string VerifiedSigFile = _config->FindDir("Dir::State::lists") +
 -      URItoFileName(RealURI) + ".gpg";
 -   Rename(SigFile,VerifiedSigFile);
 -   chmod(VerifiedSigFile.c_str(),0644);
 +   return true;
  }
                                                                        /*}}}*/
 -void pkgAcqMetaIndex::QueueIndexes(bool verify)                               /*{{{*/
 +void pkgAcqMetaBase::QueueIndexes(bool verify)                                /*{{{*/
  {
 -#if 0
 -   /* Reject invalid, existing Release files (LP: #346386) (Closes: #627642)
 -    * FIXME: Disabled; it breaks unsigned repositories without hashes */
 -   if (!verify && FileExists(DestFile) && !MetaIndexParser->Load(DestFile))
 -   {
 -      Status = StatError;
 -      ErrorText = MetaIndexParser->ErrorText;
 -      return;
 -   }
 -#endif
     bool transInRelease = false;
     {
        std::vector<std::string> const keys = MetaIndexParser->MetaKeys();
  
        if ((*Target)->IsOptional() == true)
        {
 -       if ((*Target)->IsSubIndex() == true)
 -          new pkgAcqSubIndex(Owner, (*Target)->URI, (*Target)->Description,
 -                              (*Target)->ShortDesc, ExpectedIndexHashes);
 -       else if (transInRelease == false || Record != NULL || compressedAvailable == true)
 +       if (transInRelease == false || Record != NULL || compressedAvailable == true)
         {
            if (_config->FindB("Acquire::PDiffs",true) == true && transInRelease == true &&
                MetaIndexParser->Exists((*Target)->MetaKey + ".diff/Index") == true)
 -             new pkgAcqDiffIndex(Owner, *Target, ExpectedIndexHashes, MetaIndexParser);
 +             new pkgAcqDiffIndex(Owner, TransactionManager, *Target, ExpectedIndexHashes, MetaIndexParser);
            else
 -             new pkgAcqIndexTrans(Owner, *Target, ExpectedIndexHashes, MetaIndexParser);
 +             new pkgAcqIndexTrans(Owner, TransactionManager, *Target, ExpectedIndexHashes, MetaIndexParser);
         }
         continue;
        }
           instead, but passing the required info to it is to much hassle */
        if(_config->FindB("Acquire::PDiffs",true) == true && (verify == false ||
          MetaIndexParser->Exists((*Target)->MetaKey + ".diff/Index") == true))
 -       new pkgAcqDiffIndex(Owner, *Target, ExpectedIndexHashes, MetaIndexParser);
 +       new pkgAcqDiffIndex(Owner, TransactionManager, *Target, ExpectedIndexHashes, MetaIndexParser);
        else
 -       new pkgAcqIndex(Owner, *Target, ExpectedIndexHashes, MetaIndexParser);
 +       new pkgAcqIndex(Owner, TransactionManager, *Target, ExpectedIndexHashes, MetaIndexParser);
     }
  }
                                                                        /*}}}*/
 -bool pkgAcqMetaIndex::VerifyVendor(string Message)                    /*{{{*/
 +bool pkgAcqMetaBase::VerifyVendor(string Message, const string &RealURI)/*{{{*/
  {
     string::size_type pos;
  
     return true;
  }
                                                                        /*}}}*/
 -// pkgAcqMetaIndex::Failed - no Release file present or no signature file present     /*{{{*/
 -// ---------------------------------------------------------------------
 -/* */
 -void pkgAcqMetaIndex::Failed(string Message,pkgAcquire::MethodConfig * /*Cnf*/)
 +// pkgAcqMetaIndex::Failed - no Release file present                  /*{{{*/
 +void pkgAcqMetaIndex::Failed(string Message,
 +                             pkgAcquire::MethodConfig * Cnf)
  {
 -   if (AuthPass == true)
 -   {
 -      // gpgv method failed, if we have a good signature 
 -      string LastGoodSigFile = _config->FindDir("Dir::State::lists").append("partial/").append(URItoFileName(RealURI));
 -      if (DestFile != SigFile)
 -       LastGoodSigFile.append(".gpg");
 -      LastGoodSigFile.append(".reverify");
 -
 -      if(FileExists(LastGoodSigFile))
 -      {
 -       string VerifiedSigFile = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 -       if (DestFile != SigFile)
 -          VerifiedSigFile.append(".gpg");
 -       Rename(LastGoodSigFile, VerifiedSigFile);
 -       Status = StatTransientNetworkError;
 -       _error->Warning(_("An error occurred during the signature "
 -                         "verification. The repository is not updated "
 -                         "and the previous index files will be used. "
 -                         "GPG error: %s: %s\n"),
 -                       Desc.Description.c_str(),
 -                       LookupTag(Message,"Message").c_str());
 -       RunScripts("APT::Update::Auth-Failure");
 -       return;
 -      } else if (LookupTag(Message,"Message").find("NODATA") != string::npos) {
 -       /* Invalid signature file, reject (LP: #346386) (Closes: #627642) */
 -       _error->Error(_("GPG error: %s: %s"),
 -                       Desc.Description.c_str(),
 -                       LookupTag(Message,"Message").c_str());
 -       return;
 -      } else {
 -       _error->Warning(_("GPG error: %s: %s"),
 -                       Desc.Description.c_str(),
 -                       LookupTag(Message,"Message").c_str());
 -      }
 -      // gpgv method failed 
 -      ReportMirrorFailure("GPGFailure");
 -   }
 +   pkgAcquire::Item::Failed(Message, Cnf);
 +   Status = StatDone;
  
 -   /* Always move the meta index, even if gpgv failed. This ensures
 -    * that PackageFile objects are correctly filled in */
 -   if (FileExists(DestFile)) {
 -      string FinalFile = _config->FindDir("Dir::State::lists");
 -      FinalFile += URItoFileName(RealURI);
 -      /* InRelease files become Release files, otherwise
 -       * they would be considered as trusted later on */
 -      if (SigFile == DestFile) {
 -       RealURI = RealURI.replace(RealURI.rfind("InRelease"), 9,
 -                                     "Release");
 -       FinalFile = FinalFile.replace(FinalFile.rfind("InRelease"), 9,
 -                                     "Release");
 -       SigFile = FinalFile;
 -      }
 -      Rename(DestFile,FinalFile);
 -      chmod(FinalFile.c_str(),0644);
 +   string FinalFile = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
  
 -      DestFile = FinalFile;
 -   }
 +   _error->Warning(_("The repository '%s' does not have a Release file. "
 +                     "This is deprecated, please contact the owner of the "
 +                     "repository."), URIDesc.c_str());
  
 -   // No Release file was present, or verification failed, so fall
 +   // No Release file was present so fall
     // back to queueing Packages files without verification
 -   QueueIndexes(false);
 +   // only allow going further if the users explicitely wants it
 +   if(_config->FindB("Acquire::AllowInsecureRepositories") == true)
 +   {
 +      // Done, queue for rename on transaction finished
 +      if (FileExists(DestFile)) 
 +         TransactionManager->TransactionStageCopy(this, DestFile, FinalFile);
 +
 +      // queue without any kind of hashsum support
 +      QueueIndexes(false);
 +   } else {
 +      // warn if the repository is unsinged
 +      _error->Warning("Use --allow-insecure-repositories to force the update");
 +      TransactionManager->AbortTransaction();
 +      Status = StatError;
 +      return;
 +   }
 +}
 +                                                                      /*}}}*/
 +void pkgAcqMetaIndex::Finished()                                      /*{{{*/
 +{
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "Finished: " << DestFile <<std::endl;
 +   if(TransactionManager != NULL &&
 +      TransactionManager->TransactionHasError() == false)
 +      TransactionManager->CommitTransaction();
  }
                                                                        /*}}}*/
  pkgAcqMetaClearSig::pkgAcqMetaClearSig(pkgAcquire *Owner,             /*{{{*/
                string const &MetaSigURI, string const &MetaSigURIDesc, string const &MetaSigShortDesc,
                const vector<IndexTarget*>* IndexTargets,
                indexRecords* MetaIndexParser) :
 -      pkgAcqMetaIndex(Owner, URI, URIDesc, ShortDesc, "", IndexTargets, MetaIndexParser),
 -      MetaIndexURI(MetaIndexURI), MetaIndexURIDesc(MetaIndexURIDesc), MetaIndexShortDesc(MetaIndexShortDesc),
 -      MetaSigURI(MetaSigURI), MetaSigURIDesc(MetaSigURIDesc), MetaSigShortDesc(MetaSigShortDesc)
 +   pkgAcqMetaIndex(Owner, NULL, URI, URIDesc, ShortDesc, MetaSigURI, MetaSigURIDesc,MetaSigShortDesc, IndexTargets, MetaIndexParser),
 +       MetaIndexURI(MetaIndexURI), MetaIndexURIDesc(MetaIndexURIDesc), MetaIndexShortDesc(MetaIndexShortDesc),
 +       MetaSigURI(MetaSigURI), MetaSigURIDesc(MetaSigURIDesc), MetaSigShortDesc(MetaSigShortDesc)
  {
 -   SigFile = DestFile;
 -
     // index targets + (worst case:) Release/Release.gpg
     ExpectedAdditionalItems = IndexTargets->size() + 2;
  
 -
 -   // keep the old InRelease around in case of transistent network errors
 -   string const Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 -   if (RealFileExists(Final) == true)
 -   {
 -      string const LastGoodSig = DestFile + ".reverify";
 -      Rename(Final,LastGoodSig);
 -   }
  }
                                                                        /*}}}*/
  pkgAcqMetaClearSig::~pkgAcqMetaClearSig()                             /*{{{*/
  {
 -   // if the file was never queued undo file-changes done in the constructor
 -   if (QueueCounter == 1 && Status == StatIdle && FileSize == 0 && Complete == false)
 -   {
 -      string const Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 -      string const LastGoodSig = DestFile + ".reverify";
 -      if (RealFileExists(Final) == false && RealFileExists(LastGoodSig) == true)
 -       Rename(LastGoodSig, Final);
 -   }
  }
                                                                        /*}}}*/
  // pkgAcqMetaClearSig::Custom600Headers - Insert custom request headers       /*{{{*/
  // ---------------------------------------------------------------------
 -// FIXME: this can go away once the InRelease file is used widely
  string pkgAcqMetaClearSig::Custom600Headers() const
  {
     string Final = _config->FindDir("Dir::State::lists");
     struct stat Buf;
     if (stat(Final.c_str(),&Buf) != 0)
     {
 -      Final = DestFile + ".reverify";
        if (stat(Final.c_str(),&Buf) != 0)
         return "\nIndex-File: true\nFail-Ignore: true\n";
     }
     return "\nIndex-File: true\nFail-Ignore: true\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
  }
                                                                        /*}}}*/
 +// pkgAcqMetaClearSig::Done - We got a file                           /*{{{*/
 +// ---------------------------------------------------------------------
 +void pkgAcqMetaClearSig::Done(std::string Message,unsigned long long /*Size*/,
 +                              HashStringList const &/*Hashes*/,
 +                              pkgAcquire::MethodConfig *Cnf)
 +{
 +   // if we expect a ClearTextSignature (InRelase), ensure that
 +   // this is what we get and if not fail to queue a 
 +   // Release/Release.gpg, see #346386
 +   if (FileExists(DestFile) && !StartsWithGPGClearTextSignature(DestFile))
 +   {
 +      pkgAcquire::Item::Failed(Message, Cnf);
 +      RenameOnError(NotClearsigned);
 +      TransactionManager->AbortTransaction();
 +      return;
 +   }
 +
 +   if(AuthPass == false)
 +   {
 +      if(CheckDownloadDone(Message, RealURI) == true)
 +         QueueForSignatureVerify(DestFile, DestFile);
 +      return;
 +   }
 +   else
 +   {
 +      if(CheckAuthDone(Message, RealURI) == true)
 +      {
 +         string FinalFile = _config->FindDir("Dir::State::lists");
 +         FinalFile += URItoFileName(RealURI);
 +
 +         // queue for copy in place
 +         TransactionManager->TransactionStageCopy(this, DestFile, FinalFile);
 +      }
 +   }
 +}
 +                                                                      /*}}}*/
  void pkgAcqMetaClearSig::Failed(string Message,pkgAcquire::MethodConfig *Cnf) /*{{{*/
  {
 +   Item::Failed(Message, Cnf);
 +
     // we failed, we will not get additional items from this method
     ExpectedAdditionalItems = 0;
  
     if (AuthPass == false)
     {
 -      // Remove the 'old' InRelease file if we try Release.gpg now as otherwise
 -      // the file will stay around and gives a false-auth impression (CVE-2012-0214)
 +      // Queue the 'old' InRelease file for removal if we try Release.gpg
 +      // as otherwise the file will stay around and gives a false-auth
 +      // impression (CVE-2012-0214)
        string FinalFile = _config->FindDir("Dir::State::lists");
        FinalFile.append(URItoFileName(RealURI));
 -      if (FileExists(FinalFile))
 -       unlink(FinalFile.c_str());
 +      TransactionManager->TransactionStageRemoval(this, FinalFile);
 +      Status = StatDone;
  
 -      new pkgAcqMetaSig(Owner,
 -                      MetaSigURI, MetaSigURIDesc, MetaSigShortDesc,
 +      new pkgAcqMetaIndex(Owner, TransactionManager,
                        MetaIndexURI, MetaIndexURIDesc, MetaIndexShortDesc,
 +                      MetaSigURI, MetaSigURIDesc, MetaSigShortDesc,
                        IndexTargets, MetaIndexParser);
 -      if (Cnf->LocalOnly == true ||
 -        StringToBool(LookupTag(Message, "Transient-Failure"), false) == false)
 -       Dequeue();
     }
     else
 -      pkgAcqMetaIndex::Failed(Message, Cnf);
 +   {
 +      if(CheckStopAuthentication(RealURI, Message))
 +         return;
 +
 +      _error->Warning(_("The data from '%s' is not signed. Packages "
 +                        "from that repository can not be authenticated."),
 +                      URIDesc.c_str());
 +
 +      // No Release file was present, or verification failed, so fall
 +      // back to queueing Packages files without verification
 +      // only allow going further if the users explicitely wants it
 +      if(_config->FindB("Acquire::AllowInsecureRepositories") == true)
 +      {
 +       Status = StatDone;
 +
 +         /* Always move the meta index, even if gpgv failed. This ensures
 +          * that PackageFile objects are correctly filled in */
 +         if (FileExists(DestFile))
 +         {
 +            string FinalFile = _config->FindDir("Dir::State::lists");
 +            FinalFile += URItoFileName(RealURI);
 +            /* InRelease files become Release files, otherwise
 +             * they would be considered as trusted later on */
 +            RealURI = RealURI.replace(RealURI.rfind("InRelease"), 9,
 +                                      "Release");
 +            FinalFile = FinalFile.replace(FinalFile.rfind("InRelease"), 9,
 +                                          "Release");
 +
 +            // Done, queue for rename on transaction finished
 +            TransactionManager->TransactionStageCopy(this, DestFile, FinalFile);
 +         }
 +         QueueIndexes(false);
 +      } else {
 +         // warn if the repository is unsigned
 +         _error->Warning("Use --allow-insecure-repositories to force the update");
 +         TransactionManager->AbortTransaction();
 +         Status = StatError;
 +      }
 +   }
  }
                                                                        /*}}}*/
  // AcqArchive::AcqArchive - Constructor                                       /*{{{*/
@@@ -2347,10 -2363,7 +2475,10 @@@ bool pkgAcqArchive::QueueNext(
         if ((unsigned long long)Buf.st_size > Version->Size)
            unlink(DestFile.c_str());
         else
 +       {
            PartialSize = Buf.st_size;
 +          ChangeOwnerAndPermissionOfFile("pkgAcqArchive::QueueNext", FinalFile.c_str(), "_apt", "root", 0600);
 +       }
        }
  
        // Disables download of archives - useful if no real installation follows,
@@@ -2406,20 -2419,21 +2534,20 @@@ void pkgAcqArchive::Done(string Message
        return;
     }
  
 -   Complete = true;
 -
     // Reference filename
     if (FileName != DestFile)
     {
        StoreFilename = DestFile = FileName;
        Local = true;
 +      Complete = true;
        return;
     }
 -   
 +
     // Done, move it into position
     string FinalFile = _config->FindDir("Dir::Cache::Archives");
     FinalFile += flNotDir(StoreFilename);
     Rename(DestFile,FinalFile);
 -   
 +   ChangeOwnerAndPermissionOfFile("pkgAcqArchive::Done", FinalFile.c_str(), "root", "root", 0644);
     StoreFilename = DestFile = FinalFile;
     Complete = true;
  }
@@@ -2515,10 -2529,7 +2643,10 @@@ pkgAcqFile::pkgAcqFile(pkgAcquire *Owne
        if ((Size > 0) && (unsigned long long)Buf.st_size > Size)
         unlink(DestFile.c_str());
        else
 +      {
         PartialSize = Buf.st_size;
 +       ChangeOwnerAndPermissionOfFile("pkgAcqFile", DestFile.c_str(), "_apt", "root", 0600);
 +      }
     }
  
     QueueURI(Desc);
diff --combined apt-pkg/acquire-item.h
index a3388ca3e09f2098df3ced4bf2873a804b629abe,18d72ca4097d4dd5a8335945bed5c1c5bc541b0f..0e7212fc55fd132eba5a7952a80538119471c3e3
@@@ -47,7 -47,6 +47,7 @@@ class indexRecords
  class pkgRecords;
  class pkgSourceList;
  class IndexTarget;
 +class pkgAcqMetaBase;
  
  /** \brief Represents the process by which a pkgAcquire object should {{{
   *  retrieve a file or a collection of files.
@@@ -63,8 -62,6 +63,8 @@@
   */
  class pkgAcquire::Item : public WeakPointable
  {  
 +   void *d;
 +
     protected:
     
     /** \brief The acquire object with which this item is associated. */
      *  \param Item Metadata about this item (its URI and
      *  description).
      */
 -   inline void QueueURI(ItemDesc &Item)
 -                 {Owner->Enqueue(Item);};
 +   void QueueURI(ItemDesc &Item);
  
     /** \brief Remove this item from its owner's queue. */
 -   inline void Dequeue() {Owner->Dequeue(this);};
 -   
 +   void Dequeue();
 +
     /** \brief Rename a file without modifying its timestamp.
      *
      *  Many item methods call this as their final action.
@@@ -89,7 -87,7 +89,7 @@@
      *  \param To The new name of \a From.  If \a To exists it will be
      *  overwritten.
      */
 -   void Rename(std::string From,std::string To);
 +   bool Rename(std::string From,std::string To);
  
     public:
  
         /** \brief The item was could not be downloaded because of 
        *  a transient network error (e.g. network down)
        */
 -       StatTransientNetworkError
 +       StatTransientNetworkError,
       } Status;
  
     /** \brief Contains a textual description of the error encountered
      */
     unsigned int QueueCounter;
  
 +   /** \brief TransactionManager */
 +   pkgAcqMetaBase *TransactionManager;
 +
     /** \brief The number of additional fetch items that are expected
      *  once this item is done.
      *
      */
     std::string DestFile;
  
 +   /** \brief storge name until a transaction is finished */
 +   std::string PartialFile;
 +
     /** \brief Invoked by the acquire worker when the object couldn't
      *  be fetched.
      *
     /** \return \b true if this object is being fetched from a trusted source. */
     virtual bool IsTrusted() const {return false;};
     
 -   // report mirror problems
     /** \brief Report mirror problem
      * 
      *  This allows reporting mirror failures back to a centralized
      */
     void ReportMirrorFailure(std::string FailCode);
  
 +   /** \brief Set the name of the current active subprocess
 +    *
 +    *  See also #ActiveSubprocess
 +    */
 +   void SetActiveSubprocess(const std::string &subprocess);
  
     /** \brief Initialize an item.
      *
      *  \param ExpectedHashes of the file represented by this item
      */
     Item(pkgAcquire *Owner,
 -        HashStringList const &ExpectedHashes=HashStringList());
 +        HashStringList const &ExpectedHashes=HashStringList(),
 +        pkgAcqMetaBase *TransactionManager=NULL);
  
     /** \brief Remove this item from its owner's queue by invoking
      *  pkgAcquire::Remove.
     enum RenameOnErrorState {
        HashSumMismatch,
        SizeMismatch,
 -      InvalidFormat
 +      InvalidFormat,
 +      SignatureError,
 +      NotClearsigned,
     };
  
     /** \brief Rename failed file and set error
@@@ -343,293 -328,75 +343,299 @@@ struct DiffInfo 
     /** The filename of the diff. */
     std::string file;
  
-    /** The sha1 hash of the diff. */
-    std::string sha1;
+    /** The hashes of the diff */
+    HashStringList result_hashes;
  
-    /** The size of the diff. */
-    unsigned long size;
+    /** The hashes of the file after the diff is applied */
+    HashStringList patch_hashes;
+    /** The size of the file after the diff is applied */
+    unsigned long long result_size;
+    /** The size of the diff itself */
+    unsigned long long patch_size;
  };
                                                                        /*}}}*/
 -/** \brief An item that is responsible for fetching a SubIndex                {{{
 - *
 - *  The MetaIndex file includes only records for important indexes
 - *  and records for these SubIndex files so these can carry records
 - *  for addition files like PDiffs and Translations
 - */
 -class pkgAcqSubIndex : public pkgAcquire::Item
 +                                                                      /*}}}*/
 +
 +class pkgAcqMetaBase  : public pkgAcquire::Item
  {
 +   void *d;
 +
   protected:
 -   /** \brief If \b true, debugging information will be written to std::clog. */
 -   bool Debug;
 +   std::vector<Item*> Transaction;
  
 - public:
 -   // Specialized action members
 -   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 -   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 -                   pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string DescURI() {return Desc.URI;};
 -   virtual std::string Custom600Headers() const;
 -   virtual bool ParseIndex(std::string const &IndexFile);
 +   /** \brief A package-system-specific parser for the meta-index file. */
 +   indexRecords *MetaIndexParser;
  
 -   /** \brief Create a new pkgAcqSubIndex.
 +   /** \brief The index files which should be looked up in the meta-index
 +    *  and then downloaded.
 +    */
 +   const std::vector<IndexTarget*>* IndexTargets;
 +
 +   /** \brief If \b true, the index's signature is currently being verified.
 +    */
 +   bool AuthPass;
 +
 +   // required to deal gracefully with problems caused by incorrect ims hits
 +   bool IMSHit; 
 +
 +   /** \brief Starts downloading the individual index files.
      *
 -    *  \param Owner The Acquire object that owns this item.
 +    *  \param verify If \b true, only indices whose expected hashsum
 +    *  can be determined from the meta-index will be downloaded, and
 +    *  the hashsums of indices will be checked (reporting
 +    *  #StatAuthError if there is a mismatch).  If verify is \b false,
 +    *  no hashsum checking will be performed.
 +    */
 +   void QueueIndexes(bool verify);
 +
 +
 +   /** \brief Called when a file is finished being retrieved.
      *
 -    *  \param URI The URI of the list file to download.
 +    *  If the file was not downloaded to DestFile, a copy process is
 +    *  set up to copy it to DestFile; otherwise, Complete is set to \b
 +    *  true and the file is moved to its final location.
      *
 -    *  \param URIDesc A long description of the list file to download.
 +    *  \param Message The message block received from the fetch
 +    *  subprocess.
 +    */
 +   bool CheckDownloadDone(const std::string &Message,
 +                          const std::string &RealURI);
 +
 +   /** \brief Queue the downloaded Signature for verification */
 +   void QueueForSignatureVerify(const std::string &MetaIndexFile,
 +                                const std::string &MetaIndexFileSignature);
 +
 +   /** \brief Called when authentication succeeded.
      *
 -    *  \param ShortDesc A short description of the list file to download.
 +    *  Sanity-checks the authenticated file, queues up the individual
 +    *  index files for download, and saves the signature in the lists
 +    *  directory next to the authenticated list file.
      *
 -    *  \param ExpectedHashes The list file's hashsums which are expected.
 +    *  \param Message The message block received from the fetch
 +    *  subprocess.
 +    */
 +   bool CheckAuthDone(std::string Message, const std::string &RealURI);
 +
 +   /** Check if the current item should fail at this point */
 +   bool CheckStopAuthentication(const std::string &RealURI,
 +                                const std::string &Message);
 +
 +   /** \brief Check that the release file is a release file for the
 +    *  correct distribution.
 +    *
 +    *  \return \b true if no fatal errors were encountered.
      */
 -   pkgAcqSubIndex(pkgAcquire *Owner, std::string const &URI,std::string const &URIDesc,
 -                 std::string const &ShortDesc, HashStringList const &ExpectedHashes);
 +   bool VerifyVendor(std::string Message, const std::string &RealURI);
 +   
 + public:
 +   // transaction code
 +   void Add(Item *I);
 +   void AbortTransaction();
 +   bool TransactionHasError() APT_PURE;
 +   void CommitTransaction();
 +
 +   /** \brief Stage (queue) a copy action when the transaction is commited
 +    */
 +   void TransactionStageCopy(Item *I,
 +                             const std::string &From, 
 +                             const std::string &To);
 +   /** \brief Stage (queue) a removal action when the transaction is commited
 +    */
 +   void TransactionStageRemoval(Item *I, const std::string &FinalFile);
 +
 +   pkgAcqMetaBase(pkgAcquire *Owner,
 +                  const std::vector<IndexTarget*>* IndexTargets,
 +                  indexRecords* MetaIndexParser,
 +                  HashStringList const &ExpectedHashes=HashStringList(),
 +                  pkgAcqMetaBase *TransactionManager=NULL)
 +      : Item(Owner, ExpectedHashes, TransactionManager),
 +        MetaIndexParser(MetaIndexParser), IndexTargets(IndexTargets),
 +        AuthPass(false), IMSHit(false) {};
 +};
 +
 +/** \brief An acquire item that downloads the detached signature      {{{
 + *  of a meta-index (Release) file, then queues up the release
 + *  file itself.
 + *
 + *  \todo Why protected members?
 + *
 + *  \sa pkgAcqMetaIndex
 + */
 +class pkgAcqMetaSig : public pkgAcqMetaBase
 +{
 +   void *d;
 +
 +   protected:
 +
 +   /** \brief The URI of the signature file.  Unlike Desc.URI, this is
 +    *  never modified; it is used to determine the file that is being
 +    *  downloaded.
 +    */
 +   std::string RealURI;
 +
 +   /** \brief The file we need to verify */
 +   std::string MetaIndexFile;
 +
 +   /** \brief The file we use to verify the MetaIndexFile with */
 +   std::string MetaIndexFileSignature;
 +
 +   /** \brief Long URI description used in the acquire system */
 +   std::string URIDesc;
 +
 +   /** \brief Short URI description used in the acquire system */
 +   std::string ShortDesc;
 +
 +   public:
 +   
 +   // Specialized action members
 +   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 +   virtual void Done(std::string Message,unsigned long long Size,
 +                     HashStringList const &Hashes,
 +                   pkgAcquire::MethodConfig *Cnf);
 +   virtual std::string Custom600Headers() const;
 +   virtual std::string DescURI() {return RealURI; };
 +
 +   /** \brief Create a new pkgAcqMetaSig. */
 +   pkgAcqMetaSig(pkgAcquire *Owner,
 +                 pkgAcqMetaBase *TransactionManager,
 +                 std::string URI,std::string URIDesc, std::string ShortDesc,
 +                 std::string MetaIndexFile,
 +               const std::vector<IndexTarget*>* IndexTargets,
 +               indexRecords* MetaIndexParser);
 +   virtual ~pkgAcqMetaSig();
 +};
 +                                                                      /*}}}*/
 +
 +/** \brief An item that is responsible for downloading the meta-index {{{
 + *  file (i.e., Release) itself and verifying its signature.
 + *
 + *  Once the download and verification are complete, the downloads of
 + *  the individual index files are queued up using pkgAcqDiffIndex.
 + *  If the meta-index file had a valid signature, the expected hashsums
 + *  of the index files will be the md5sums listed in the meta-index;
 + *  otherwise, the expected hashsums will be "" (causing the
 + *  authentication of the index files to be bypassed).
 + */
 +class pkgAcqMetaIndex : public pkgAcqMetaBase
 +{
 +   void *d;
 +
 +   protected:
 +   /** \brief The URI that is actually being downloaded; never
 +    *  modified by pkgAcqMetaIndex.
 +    */
 +   std::string RealURI;
 +
 +   std::string URIDesc;
 +   std::string ShortDesc;
 +
 +   /** \brief The URI of the meta-index file for the detached signature */
 +   std::string MetaIndexSigURI;
 +
 +   /** \brief A "URI-style" description of the meta-index file */
 +   std::string MetaIndexSigURIDesc;
 +
 +   /** \brief A brief description of the meta-index file */
 +   std::string MetaIndexSigShortDesc;
 +
 +   /** \brief delayed constructor */
 +   void Init(std::string URIDesc, std::string ShortDesc);
 +   
 +   public:
 +
 +   // Specialized action members
 +   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 +   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 +                   pkgAcquire::MethodConfig *Cnf);
 +   virtual std::string Custom600Headers() const;
 +   virtual std::string DescURI() {return RealURI; };
 +   virtual void Finished();
 +
 +   /** \brief Create a new pkgAcqMetaIndex. */
 +   pkgAcqMetaIndex(pkgAcquire *Owner,
 +                   pkgAcqMetaBase *TransactionManager,
 +                 std::string URI,std::string URIDesc, std::string ShortDesc,
 +                   std::string MetaIndexSigURI, std::string MetaIndexSigURIDesc, std::string MetaIndexSigShortDesc,
 +                 const std::vector<IndexTarget*>* IndexTargets,
 +                 indexRecords* MetaIndexParser);
 +};
 +                                                                      /*}}}*/
 +/** \brief An item repsonsible for downloading clearsigned metaindexes        {{{*/
 +class pkgAcqMetaClearSig : public pkgAcqMetaIndex
 +{
 +   void *d;
 +
 +   /** \brief The URI of the meta-index file for the detached signature */
 +   std::string MetaIndexURI;
 +
 +   /** \brief A "URI-style" description of the meta-index file */
 +   std::string MetaIndexURIDesc;
 +
 +   /** \brief A brief description of the meta-index file */
 +   std::string MetaIndexShortDesc;
 +
 +   /** \brief The URI of the detached meta-signature file if the clearsigned one failed. */
 +   std::string MetaSigURI;
 +
 +   /** \brief A "URI-style" description of the meta-signature file */
 +   std::string MetaSigURIDesc;
 +
 +   /** \brief A brief description of the meta-signature file */
 +   std::string MetaSigShortDesc;
 +
 +public:
 +   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 +   virtual std::string Custom600Headers() const;
 +   virtual void Done(std::string Message,unsigned long long Size,
 +                     HashStringList const &Hashes,
 +                   pkgAcquire::MethodConfig *Cnf);
 +
 +   /** \brief Create a new pkgAcqMetaClearSig. */
 +   pkgAcqMetaClearSig(pkgAcquire *Owner,
 +              std::string const &URI, std::string const &URIDesc, std::string const &ShortDesc,
 +              std::string const &MetaIndexURI, std::string const &MetaIndexURIDesc, std::string const &MetaIndexShortDesc,
 +              std::string const &MetaSigURI, std::string const &MetaSigURIDesc, std::string const &MetaSigShortDesc,
 +              const std::vector<IndexTarget*>* IndexTargets,
 +              indexRecords* MetaIndexParser);
 +   virtual ~pkgAcqMetaClearSig();
  };
                                                                        /*}}}*/
  
 +
  /** \brief Common base class for all classes that deal with fetching  {{{
             indexes
   */
  class pkgAcqBaseIndex : public pkgAcquire::Item
  {
 +   void *d;
 +
   protected:
     /** \brief Pointer to the IndexTarget data
      */
     const struct IndexTarget * Target;
 +
 +   /** \brief Pointer to the indexRecords parser */
     indexRecords *MetaIndexParser;
  
 +   /** \brief The MetaIndex Key */
 +   std::string MetaKey;
 +
 +   /** \brief The URI of the index file to recreate at our end (either
 +    *  by downloading it or by applying partial patches).
 +    */
 +   std::string RealURI;
 +
 +   bool VerifyHashByMetaKey(HashStringList const &Hashes);
 +
     pkgAcqBaseIndex(pkgAcquire *Owner,
 +                   pkgAcqMetaBase *TransactionManager,
                     struct IndexTarget const * const Target,
                     HashStringList const &ExpectedHashes,
                     indexRecords *MetaIndexParser)
 -      : Item(Owner, ExpectedHashes), Target(Target), 
 +      : Item(Owner, ExpectedHashes, TransactionManager), Target(Target), 
          MetaIndexParser(MetaIndexParser) {};
 -
  };
                                                                        /*}}}*/
  /** \brief An item that is responsible for fetching an index file of  {{{
   */
  class pkgAcqDiffIndex : public pkgAcqBaseIndex
  {
 +   void *d;
 +
   protected:
     /** \brief If \b true, debugging information will be written to std::clog. */
     bool Debug;
  
 -   /** \brief The URI of the index file to recreate at our end (either
 -    *  by downloading it or by applying partial patches).
 -    */
 -   std::string RealURI;
 -
     /** \brief The index file which will be patched to generate the new
      *  file.
      */
      */
     std::string Description;
  
 +   /** \brief If the copy step of the packages file is done
 +    */
 +   bool PackagesFileReadyInPartial;
 +
   public:
     // Specialized action members
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
      *  \param ExpectedHashes The list file's hashsums which are expected.
      */
     pkgAcqDiffIndex(pkgAcquire *Owner,
 +                   pkgAcqMetaBase *TransactionManager,
                     struct IndexTarget const * const Target,
                     HashStringList const &ExpectedHashes,
                     indexRecords *MetaIndexParser);
   */
  class pkgAcqIndexMergeDiffs : public pkgAcqBaseIndex
  {
 +   void *d;
 +
     protected:
  
     /** \brief If \b true, debugging output will be written to
      */
     bool Debug;
  
 -   /** \brief URI of the package index file that is being
 -    *  reconstructed.
 -    */
 -   std::string RealURI;
 -
     /** \brief description of the file being downloaded. */
     std::string Description;
  
      *  check if it was the last one to complete the download step
      */
     pkgAcqIndexMergeDiffs(pkgAcquire *Owner,
 +                         pkgAcqMetaBase *TransactionManager,
                           struct IndexTarget const * const Target,
                           HashStringList const &ExpectedHash,
                           indexRecords *MetaIndexParser,
   */
  class pkgAcqIndexDiffs : public pkgAcqBaseIndex
  {
 +   void *d;
 +
     private:
  
     /** \brief Queue up the next diff download.
      *  \return \b true if an applicable diff was found, \b false
      *  otherwise.
      */
-    bool QueueNextDiff();
+    APT_HIDDEN bool QueueNextDiff();
  
     /** \brief Handle tasks that must be performed after the item
      *  finishes downloading.
      *  \param allDone If \b true, the file was entirely reconstructed,
      *  and its md5sum is verified. 
      */
-    void Finish(bool allDone=false);
+    APT_HIDDEN void Finish(bool allDone=false);
  
     protected:
  
      */
     bool Debug;
  
 -   /** \brief The URI of the package index file that is being
 -    *  reconstructed.
 -    */
 -   std::string RealURI;
 -
     /** A description of the file being downloaded. */
     std::string Description;
  
      */
     std::vector<DiffInfo> available_patches;
  
-    /** Stop applying patches when reaching that sha1 */
-    std::string ServerSha1;
     /** The current status of this patch. */
     enum DiffState
       {
  
     virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
                     pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string DescURI() {return RealURI + "Index";};
 +   virtual std::string DescURI() {return RealURI + "IndexDiffs";};
  
     /** \brief Create an index diff item.
      *
      *
      *  \param ShortDesc A brief description of this item.
      *
-     *  \param ExpectedHashes The expected md5sum of the completely
+     *  \param ExpectedHashes The expected hashsums of the completely
      *  reconstructed package index file; the index file will be tested
      *  against this value when it is entirely reconstructed.
      *
-     *  \param ServerSha1 is the sha1sum of the current file on the server
-     *
      *  \param diffs The remaining diffs from the index of diffs.  They
      *  should be ordered so that each diff appears before any diff
      *  that depends on it.
      */
     pkgAcqIndexDiffs(pkgAcquire *Owner,
 +                    pkgAcqMetaBase *TransactionManager,
                      struct IndexTarget const * const Target,
                      HashStringList const &ExpectedHash,
                      indexRecords *MetaIndexParser,
-                   std::string ServerSha1,
                    std::vector<DiffInfo> diffs=std::vector<DiffInfo>());
  };
                                                                        /*}}}*/
   */
  class pkgAcqIndex : public pkgAcqBaseIndex
  {
 -   protected:
 +   void *d;
  
 -   /** \brief If \b true, the index file has been decompressed. */
 -   bool Decompression;
 +   protected:
  
 -   /** \brief If \b true, the partially downloaded file will be
 -    *  removed when the download completes.
 +   /** \brief The stages the method goes through
 +    *
 +    *  The method first downloads the indexfile, then its decompressed (or
 +    *  copied) and verified
      */
 -   bool Erase;
 +   enum AllStages {
 +      STAGE_DOWNLOAD,
 +      STAGE_DECOMPRESS_AND_VERIFY,
 +   };
 +   AllStages Stage;
  
 -   // Unused, used to be used to verify that "Packages: " header was there
 -   bool __DELME_ON_NEXT_ABI_BREAK_Verify;
 +   /** \brief Handle what needs to be done when the download is done */
 +   void StageDownloadDone(std::string Message,
 +                          HashStringList const &Hashes,
 +                          pkgAcquire::MethodConfig *Cfg);
  
 -   /** \brief The object that is actually being fetched (minus any
 -    *  compression-related extensions).
 +   /** \brief Handle what needs to be done when the decompression/copy is
 +    *         done 
      */
 -   std::string RealURI;
 +   void StageDecompressDone(std::string Message,
 +                            HashStringList const &Hashes,
 +                            pkgAcquire::MethodConfig *Cfg);
 +
 +   /** \brief If \b set, this partially downloaded file will be
 +    *  removed when the download completes.
 +    */
 +   std::string EraseFileName;
  
     /** \brief The compression-related file extensions that are being
      *  added to the downloaded file one by one if first fails (e.g., "gz bz2").
      */
 -   std::string CompressionExtension;
 +   std::string CompressionExtensions;
  
 +   /** \brief The actual compression extension currently used */
 +   std::string CurrentCompressionExtension;
  
     /** \brief Do the changes needed to fetch via AptByHash (if needed) */
     void InitByHashIfNeeded(const std::string MetaKey);
  
 -   /** \brief Get the full pathname of the final file for the given URI
 +   /** \brief Auto select the right compression to use */
 +   void AutoSelectCompression();
 +
 +   /** \brief Get the full pathname of the final file for the current URI
      */
 -   std::string GetFinalFilename(std::string const &URI,
 -                                std::string const &compExt);
 +   std::string GetFinalFilename() const;
  
     /** \brief Schedule file for verification after a IMS hit */
 -   void ReverifyAfterIMS(std::string const &FileName);
 +   void ReverifyAfterIMS();
 +
 +   /** \brief Validate the downloaded index file */
 +   bool ValidateFile(const std::string &FileName);
  
     public:
     
     // Specialized action members
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 -   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 +   virtual void Done(std::string Message,unsigned long long Size, 
 +                     HashStringList const &Hashes,
                     pkgAcquire::MethodConfig *Cnf);
     virtual std::string Custom600Headers() const;
     virtual std::string DescURI() {return Desc.URI;};
      *  fallback is ".gz" or none.
      */
     pkgAcqIndex(pkgAcquire *Owner,std::string URI,std::string URIDesc,
 -             std::string ShortDesc, HashStringList const &ExpectedHashes,
 -             std::string compressExt="");
 -   pkgAcqIndex(pkgAcquire *Owner,
 +             std::string ShortDesc, HashStringList const &ExpectedHashes);
 +   pkgAcqIndex(pkgAcquire *Owner, pkgAcqMetaBase *TransactionManager,
                 IndexTarget const * const Target,
                 HashStringList const &ExpectedHash,
                 indexRecords *MetaIndexParser);
 +               
     void Init(std::string const &URI, std::string const &URIDesc,
               std::string const &ShortDesc);
  };
   */
  class pkgAcqIndexTrans : public pkgAcqIndex
  {
 +   void *d;
 +
     public:
    
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
      *
      *  \param ShortDesc A brief description of this index file.
      */
 -   pkgAcqIndexTrans(pkgAcquire *Owner,std::string URI,std::string URIDesc,
 +   pkgAcqIndexTrans(pkgAcquire *Owner,
 +                    std::string URI,std::string URIDesc,
                    std::string ShortDesc);
 -   pkgAcqIndexTrans(pkgAcquire *Owner, IndexTarget const * const Target,
 -                  HashStringList const &ExpectedHashes, indexRecords *MetaIndexParser);
 +   pkgAcqIndexTrans(pkgAcquire *Owner,
 +                    pkgAcqMetaBase *TransactionManager,
 +                    IndexTarget const * const Target,
 +                    HashStringList const &ExpectedHashes,
 +                    indexRecords *MetaIndexParser);
  };
                                                                        /*}}}*/
  /** \brief Information about an index file. */                                /*{{{*/
  class IndexTarget
  {
 +   void *d;
 +
   public:
     /** \brief A URI from which the index file can be downloaded. */
     std::string URI;
     virtual bool IsOptional() const {
        return false;
     }
 -   virtual bool IsSubIndex() const {
 -      return false;
 -   }
  };
                                                                        /*}}}*/
  /** \brief Information about an optional index file. */                       /*{{{*/
  class OptionalIndexTarget : public IndexTarget
  {
 +   void *d;
 +
     virtual bool IsOptional() const {
        return true;
     }
  };
                                                                        /*}}}*/
 -/** \brief Information about an subindex index file. */                       /*{{{*/
 -class SubIndexTarget : public IndexTarget
 -{
 -   virtual bool IsSubIndex() const {
 -      return true;
 -   }
 -};
 -                                                                      /*}}}*/
 -/** \brief Information about an subindex index file. */                       /*{{{*/
 -class OptionalSubIndexTarget : public OptionalIndexTarget
 -{
 -   virtual bool IsSubIndex() const {
 -      return true;
 -   }
 -};
 -                                                                      /*}}}*/
 -
 -/** \brief An acquire item that downloads the detached signature      {{{
 - *  of a meta-index (Release) file, then queues up the release
 - *  file itself.
 - *
 - *  \todo Why protected members?
 - *
 - *  \sa pkgAcqMetaIndex
 - */
 -class pkgAcqMetaSig : public pkgAcquire::Item
 -{
 -   protected:
 -   /** \brief The last good signature file */
 -   std::string LastGoodSig;
 -
 -   /** \brief The URI of the signature file.  Unlike Desc.URI, this is
 -    *  never modified; it is used to determine the file that is being
 -    *  downloaded.
 -    */
 -   std::string RealURI;
 -
 -   /** \brief The URI of the meta-index file to be fetched after the signature. */
 -   std::string MetaIndexURI;
 -
 -   /** \brief A "URI-style" description of the meta-index file to be
 -    *  fetched after the signature.
 -    */
 -   std::string MetaIndexURIDesc;
 -
 -   /** \brief A brief description of the meta-index file to be fetched
 -    *  after the signature.
 -    */
 -   std::string MetaIndexShortDesc;
 -
 -   /** \brief A package-system-specific parser for the meta-index file. */
 -   indexRecords* MetaIndexParser;
 -
 -   /** \brief The index files which should be looked up in the meta-index
 -    *  and then downloaded.
 -    *
 -    *  \todo Why a list of pointers instead of a list of structs?
 -    */
 -   const std::vector<IndexTarget*>* IndexTargets;
 -
 -   public:
 -   
 -   // Specialized action members
 -   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 -   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 -                   pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string Custom600Headers() const;
 -   virtual std::string DescURI() {return RealURI; };
 -
 -   /** \brief Create a new pkgAcqMetaSig. */
 -   pkgAcqMetaSig(pkgAcquire *Owner,std::string URI,std::string URIDesc, std::string ShortDesc,
 -               std::string MetaIndexURI, std::string MetaIndexURIDesc, std::string MetaIndexShortDesc,
 -               const std::vector<IndexTarget*>* IndexTargets,
 -               indexRecords* MetaIndexParser);
 -   virtual ~pkgAcqMetaSig();
 -};
 -                                                                      /*}}}*/
 -/** \brief An item that is responsible for downloading the meta-index {{{
 - *  file (i.e., Release) itself and verifying its signature.
 - *
 - *  Once the download and verification are complete, the downloads of
 - *  the individual index files are queued up using pkgAcqDiffIndex.
 - *  If the meta-index file had a valid signature, the expected hashsums
 - *  of the index files will be the md5sums listed in the meta-index;
 - *  otherwise, the expected hashsums will be "" (causing the
 - *  authentication of the index files to be bypassed).
 - */
 -class pkgAcqMetaIndex : public pkgAcquire::Item
 -{
 -   protected:
 -   /** \brief The URI that is actually being downloaded; never
 -    *  modified by pkgAcqMetaIndex.
 -    */
 -   std::string RealURI;
 -
 -   /** \brief The file in which the signature for this index was stored.
 -    *
 -    *  If empty, the signature and the md5sums of the individual
 -    *  indices will not be checked.
 -    */
 -   std::string SigFile;
 -
 -   /** \brief The index files to download. */
 -   const std::vector<IndexTarget*>* IndexTargets;
 -
 -   /** \brief The parser for the meta-index file. */
 -   indexRecords* MetaIndexParser;
 -
 -   /** \brief If \b true, the index's signature is currently being verified.
 -    */
 -   bool AuthPass;
 -   // required to deal gracefully with problems caused by incorrect ims hits
 -   bool IMSHit; 
 -
 -   /** \brief Check that the release file is a release file for the
 -    *  correct distribution.
 -    *
 -    *  \return \b true if no fatal errors were encountered.
 -    */
 -   bool VerifyVendor(std::string Message);
 -
 -   /** \brief Called when a file is finished being retrieved.
 -    *
 -    *  If the file was not downloaded to DestFile, a copy process is
 -    *  set up to copy it to DestFile; otherwise, Complete is set to \b
 -    *  true and the file is moved to its final location.
 -    *
 -    *  \param Message The message block received from the fetch
 -    *  subprocess.
 -    */
 -   void RetrievalDone(std::string Message);
 -
 -   /** \brief Called when authentication succeeded.
 -    *
 -    *  Sanity-checks the authenticated file, queues up the individual
 -    *  index files for download, and saves the signature in the lists
 -    *  directory next to the authenticated list file.
 -    *
 -    *  \param Message The message block received from the fetch
 -    *  subprocess.
 -    */
 -   void AuthDone(std::string Message);
 -
 -   /** \brief Starts downloading the individual index files.
 -    *
 -    *  \param verify If \b true, only indices whose expected hashsum
 -    *  can be determined from the meta-index will be downloaded, and
 -    *  the hashsums of indices will be checked (reporting
 -    *  #StatAuthError if there is a mismatch).  If verify is \b false,
 -    *  no hashsum checking will be performed.
 -    */
 -   void QueueIndexes(bool verify);
 -   
 -   public:
 -   
 -   // Specialized action members
 -   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 -   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 -                   pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string Custom600Headers() const;
 -   virtual std::string DescURI() {return RealURI; };
 -
 -   /** \brief Create a new pkgAcqMetaIndex. */
 -   pkgAcqMetaIndex(pkgAcquire *Owner,
 -                 std::string URI,std::string URIDesc, std::string ShortDesc,
 -                 std::string SigFile,
 -                 const std::vector<IndexTarget*>* IndexTargets,
 -                 indexRecords* MetaIndexParser);
 -};
 -                                                                      /*}}}*/
 -/** \brief An item repsonsible for downloading clearsigned metaindexes        {{{*/
 -class pkgAcqMetaClearSig : public pkgAcqMetaIndex
 -{
 -   /** \brief The URI of the meta-index file for the detached signature */
 -   std::string MetaIndexURI;
 -
 -   /** \brief A "URI-style" description of the meta-index file */
 -   std::string MetaIndexURIDesc;
 -
 -   /** \brief A brief description of the meta-index file */
 -   std::string MetaIndexShortDesc;
 -
 -   /** \brief The URI of the detached meta-signature file if the clearsigned one failed. */
 -   std::string MetaSigURI;
 -
 -   /** \brief A "URI-style" description of the meta-signature file */
 -   std::string MetaSigURIDesc;
 -
 -   /** \brief A brief description of the meta-signature file */
 -   std::string MetaSigShortDesc;
 -
 -public:
 -   void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string Custom600Headers() const;
 -
 -   /** \brief Create a new pkgAcqMetaClearSig. */
 -   pkgAcqMetaClearSig(pkgAcquire *Owner,
 -              std::string const &URI, std::string const &URIDesc, std::string const &ShortDesc,
 -              std::string const &MetaIndexURI, std::string const &MetaIndexURIDesc, std::string const &MetaIndexShortDesc,
 -              std::string const &MetaSigURI, std::string const &MetaSigURIDesc, std::string const &MetaSigShortDesc,
 -              const std::vector<IndexTarget*>* IndexTargets,
 -              indexRecords* MetaIndexParser);
 -   virtual ~pkgAcqMetaClearSig();
 -};
 -                                                                      /*}}}*/
  /** \brief An item that is responsible for fetching a package file.   {{{
   *
   *  If the package file already exists in the cache, nothing will be
   */
  class pkgAcqArchive : public pkgAcquire::Item
  {
 +   void *d;
 +
     protected:
     /** \brief The package version being fetched. */
     pkgCache::VerIterator Version;
   */
  class pkgAcqFile : public pkgAcquire::Item
  {
 +   void *d;
 +
     /** \brief How many times to retry the download, set from
      *  Acquire::Retries.
      */
diff --combined cmdline/apt-get.cc
index 15696e19fbb1679baa31d4c4a436f1e63ab29ec0,13a85d34b003e80f267ba36607e0e7be5858ba48..8448638dbbeb99dd49e7954cf76c33c3cdf43330
@@@ -646,7 -646,9 +646,7 @@@ static bool DoDownload(CommandLine &Cmd
        return false;
  
     AcqTextStatus Stat(ScreenWidth, _config->FindI("quiet", 0));
 -   pkgAcquire Fetcher;
 -   if (Fetcher.Setup(&Stat, "", false) == false)
 -      return false;
 +   pkgAcquire Fetcher(&Stat);
  
     pkgRecords Recs(Cache);
     pkgSourceList *SrcList = Cache.GetSourceList();
@@@ -742,8 -744,9 +742,8 @@@ static bool DoSource(CommandLine &CmdL
        return false;
  
     // Create the download object
 -   AcqTextStatus Stat(ScreenWidth,_config->FindI("quiet",0));   
 -   pkgAcquire Fetcher;
 -   Fetcher.SetLog(&Stat);
 +   AcqTextStatus Stat(ScreenWidth,_config->FindI("quiet",0));
 +   pkgAcquire Fetcher(&Stat);
  
     SPtrArray<DscFile> Dsc = new DscFile[CmdL.FileSize()];
     
@@@ -1045,6 -1048,12 +1045,6 @@@ static bool DoBuildDep(CommandLine &Cmd
     if (_error->PendingError() == true)
        return false;
  
 -   // Create the download object
 -   AcqTextStatus Stat(ScreenWidth,_config->FindI("quiet",0));   
 -   pkgAcquire Fetcher;
 -   if (Fetcher.Setup(&Stat) == false)
 -      return false;
 -
     bool StripMultiArch;
     string hostArch = _config->Find("APT::Get::Host-Architecture");
     if (hostArch.empty() == false)
@@@ -1556,7 -1565,8 +1556,7 @@@ static bool DoChangelog(CommandLine &Cm
     }
  
     AcqTextStatus Stat(ScreenWidth, _config->FindI("quiet",0));
 -   if (Fetcher.Setup(&Stat, "",false) == false)
 -      return false;
 +   Fetcher.SetLog(&Stat);
  
     bool const downOnly = _config->FindB("APT::Get::Download-Only", false);
  
     {
        string changelogfile;
        if (downOnly == false)
-        changelogfile.append(tmpname).append("changelog");
+        changelogfile.append(tmpname).append("/changelog");
        else
         changelogfile.append(Ver.ParentPkg().Name()).append(".changelog");
        if (DownloadChangelog(Cache, Fetcher, Ver, changelogfile) && downOnly == false)
diff --combined debian/apt.postinst
index b8f3edbe595e9ebae2fef8405ad2d839fce2b6c3,5820db5875febf317b70dd6c73c4cfbdaa17f711..b0a5da7d85d7cb5c5ba46c65a15b63fc83be6f0c
@@@ -15,13 -15,17 +15,17 @@@ set -
  
  case "$1" in
      configure)
-       if dpkg --compare-versions "$2" lt 1.0.7; then
+       if dpkg --compare-versions "$2" lt 1.1~exp4; then
            # apt-key before 0.9.10 could leave empty keyrings around
            find /etc/apt/trusted.gpg.d/ -name '*.gpg' | while read keyring; do
                if ! test -s "$keyring"; then
                    rm -f "$keyring"
                fi
            done
+           # apt-key before 0.9.8.2 could create 0600 trusted.gpg file
+           if test -e /etc/apt/trusted.gpg ; then
+               chmod -f 0644 /etc/apt/trusted.gpg || true
+           fi
        fi
  
        if dpkg --compare-versions "$2" lt-nl 0.9.9.5; then
            fi
        fi
  
 -        # add unprivileged user for the apt methods
 -        adduser --force-badname --system -home /var/empty \
 -            --no-create-home --quiet _apt || true
 -        chown -R _apt:root \
 -            /var/lib/apt/lists \
 -            /var/cache/apt/archives
 +      # add unprivileged user for the apt methods
 +      adduser --force-badname --system -home /var/empty \
 +          --no-create-home --quiet _apt || true
 +
 +      # deal with upgrades from experimental
 +      if dpkg --compare-versions "$2" 'eq' '1.1~exp3'; then
 +          # libapt will setup partial/ at runtime
 +          chown -R root:root /var/lib/apt/lists /var/cache/apt/archives || true
 +      fi
  
          # ensure tighter permissons on the logs, see LP: #975199
          if dpkg --compare-versions "$2" lt-nl 0.9.7.7; then
index 29e5fafe62360b38dc8e328855c9c165f660aa6a,e83606fae22a93c5c6a3039ac395157608936619..0aa648fb657c734d213b76b9c7cdebdb63a92ced
@@@ -47,6 -47,6 +47,9 @@@ msgskip() { printf "${CWARNING}SKIP${CN
  msgfail() {
        if [ $# -gt 0 ]; then printf "${CFAIL}FAIL: $*${CNORMAL}\n" >&2;
        else printf "${CFAIL}FAIL${CNORMAL}\n" >&2; fi
++        if [ -n "$APT_DEBUG_TESTS" ]; then
++            bash
++        fi
        EXIT_CODE=$((EXIT_CODE+1));
  }
  
@@@ -164,10 -164,9 +167,10 @@@ addtrap() 
  
  setupenvironment() {
        TMPWORKINGDIRECTORY=$(mktemp -d)
 -      TESTDIRECTORY=$(readlink -f $(dirname $0))
 +      addtrap "cd /; rm -rf $TMPWORKINGDIRECTORY;"
        msgninfo "Preparing environment for ${CCMD}$(basename $0)${CINFO} in ${TMPWORKINGDIRECTORY}… "
  
 +      TESTDIRECTORY=$(readlink -f $(dirname $0))
          # allow overriding the default BUILDDIR location
        BUILDDIRECTORY=${APT_INTEGRATION_TESTS_BUILD_DIR:-"${TESTDIRECTORY}/../../build/bin"}
        LIBRARYPATH=${APT_INTEGRATION_TESTS_LIBRARY_PATH:-"${BUILDDIRECTORY}"}
        test -x "${BUILDDIRECTORY}/apt-get" || msgdie "You need to build tree first"
          # -----
  
 -      addtrap "cd /; rm -rf $TMPWORKINGDIRECTORY;"
        cd $TMPWORKINGDIRECTORY
        mkdir rootdir aptarchive keys
        cd rootdir
                cp "${TESTDIRECTORY}/${SOURCESSFILE}" aptarchive/Sources
        fi
        cp $(find $TESTDIRECTORY -name '*.pub' -o -name '*.sec') keys/
 +      chmod 644 $(find keys -name '*.pub' -o -name '*.sec')
        ln -s ${TMPWORKINGDIRECTORY}/keys/joesixpack.pub rootdir/etc/apt/trusted.gpg.d/joesixpack.gpg
        echo "Dir \"${TMPWORKINGDIRECTORY}/rootdir\";" > aptconfig.conf
        echo "Dir::state::status \"${TMPWORKINGDIRECTORY}/rootdir/var/lib/dpkg/status\";" >> aptconfig.conf
@@@ -715,7 -714,7 +718,7 @@@ buildaptarchivefromincoming() 
        aptftparchive -qq generate ftparchive.conf
        cd - > /dev/null
        msgdone "info"
 -      generatereleasefiles
 +      generatereleasefiles "$@"
  }
  
  buildaptarchivefromfiles() {
@@@ -830,20 -829,17 +833,20 @@@ setupflataptarchive() 
  }
  
  setupaptarchive() {
 -      buildaptarchive
 +      local NOUPDATE=0
 +      if [ "$1" = '--no-update' ]; then
 +              NOUPDATE=1
 +              shift
 +      fi
 +      buildaptarchive "$@"
        if [ -e aptarchive/dists ]; then
                setupdistsaptarchive
        else
                setupflataptarchive
        fi
 -      signreleasefiles
 -      if [ "$1" != '--no-update' ]; then
 -              msgninfo "\tSync APT's cache with the archive… "
 -              aptget update -qq
 -              msgdone "info"
 +      signreleasefiles 'Joe Sixpack' "$@"
 +      if [ "1" != "$NOUPDATE" ]; then
 +              testsuccess aptget update -o Debug::pkgAcquire::Worker=true -o Debug::Acquire::gpgv=true
        fi
  }
  
@@@ -1179,19 -1175,6 +1182,19 @@@ testfailure() 
        fi
  }
  
 +testaccessrights() {
 +      msgtest "Test that file $1 has access rights set to" "$2"
 +      if [ "$2" = "$(stat --format '%a' "$1")" ]; then
 +              msgpass
 +      else
 +              echo >&2
 +              ls -l >&2 "$1"
 +              echo -n >&2 "stat(1) reports access rights: "
 +              stat --format '%a' "$1"
 +              msgfail
 +      fi
 +}
 +
  testwebserverlaststatuscode() {
        local DOWNLOG='rootdir/tmp/webserverstatus-testfile.log'
        local STATUS='rootdir/tmp/webserverstatus-statusfile.log'
index 805ed5964c9298e1f2b78ea78c0bdec19d25a278,805ed5964c9298e1f2b78ea78c0bdec19d25a278..1beb5d8318f0b31a372fc94fa28f66c17843e8ea
@@@ -39,10 -39,10 +39,10 @@@ testrun() 
                test -e rootdir/var/lib/apt/lists/*_Translation-en.${COMPRESS} || F=1
                # there is no point in trying pdiff if we have compressed indexes
                # as we can't patch compressed files (well, we can, but what is the point?)
--              ! test -e rootdir/var/lib/apt/lists/*.IndexDiff || F=1
++              ! test -e rootdir/var/lib/apt/lists/*diff_Index || F=1
        else
                # clear the faked pdiff indexes so the glob below works
--              rm -f rootdir/var/lib/apt/lists/*.IndexDiff
++              rm -f rootdir/var/lib/apt/lists/*diff_Index
                test -e rootdir/var/lib/apt/lists/*_Packages || F=1
                test -e rootdir/var/lib/apt/lists/*_Sources || F=1
                test -e rootdir/var/lib/apt/lists/*_Translation-en || F=1
index e86963f2802960e1fda6a931fff3adc51adfc20d,0d52614291ab1f9fd8b813dcb2e62c3ae387afc4..5bad902147f4a94b4e68e56d0bee27aca6474d71
@@@ -76,8 -76,15 +76,15 @@@ SHA1-History
   9f4148e06d7faa37062994ff10d0c842d7017513 33053002 2010-08-18-2013.28
   $(sha1sum $PKGFILE | cut -d' ' -f 1) $(stat -c%s $PKGFILE) $(basename $PATCHFILE)
  SHA1-Patches:
-  7651fc0ac57cd83d41c63195a9342e2db5650257 19722 2010-08-18-0814.28
-  $(sha1sum $PATCHFILE | cut -d' ' -f 1) $(stat -c%s $PATCHFILE) $(basename $PATCHFILE)" > $PATCHINDEX
+  7651fc0ac57cd83d41c63195a9342e2db5650257 19722 2010-08-18-2013.28
+  $(sha1sum $PATCHFILE | cut -d' ' -f 1) $(stat -c%s $PATCHFILE) $(basename $PATCHFILE)
+ SHA256-Current: $(sha256sum ${PKGFILE}-new | cut -d' ' -f 1) $(stat -c%s ${PKGFILE}-new)
+ SHA256-History:
+  01ba4719c80b6fe911b091a7c05124b64eeece964e09c058ef8f9805daca546b 33053002 2010-08-18-2013.28
+  $(sha256sum $PKGFILE | cut -d' ' -f 1) $(stat -c%s $PKGFILE) $(basename $PATCHFILE)
+ SHA256-Patches:
+  e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855 19722 2010-08-18-2013.28
+  $(sha256sum $PATCHFILE | cut -d' ' -f 1) $(stat -c%s $PATCHFILE) $(basename $PATCHFILE)" > $PATCHINDEX
        generatereleasefiles '+1hour'
        signreleasefiles
        find aptarchive -name 'Packages*' -type f -delete
@@@ -87,7 -94,7 +94,7 @@@
  " aptcache show apt newstuff
  
        msgmsg "Testcase: index is already up-to-date: $*"
--      find rootdir/var/lib/apt/lists -name '*.IndexDiff' -type f -delete
++      find rootdir/var/lib/apt/lists -name '*diff_Index' -type f -delete
        testsuccess aptget update "$@"
        testequal "$(cat ${PKGFILE}-new)
  " aptcache show apt newstuff
@@@ -119,9 -126,18 +126,18 @@@ SHA1-History
   $(sha1sum ${PKGFILE} | cut -d' ' -f 1) $(stat -c%s ${PKGFILE}) $(basename ${PATCHFILE})
   $(sha1sum ${PKGFILE}-new | cut -d' ' -f 1) $(stat -c%s ${PKGFILE}-new) $(basename ${PATCHFILE2})
  SHA1-Patches:
-  7651fc0ac57cd83d41c63195a9342e2db5650257 19722 2010-08-18-0814.28
+  7651fc0ac57cd83d41c63195a9342e2db5650257 19722 2010-08-18-2013.28
   $(sha1sum $PATCHFILE | cut -d' ' -f 1) $(stat -c%s $PATCHFILE) $(basename $PATCHFILE)
-  $(sha1sum ${PATCHFILE2} | cut -d' ' -f 1) $(stat -c%s ${PATCHFILE2}) $(basename ${PATCHFILE2})" > $PATCHINDEX
+  $(sha1sum ${PATCHFILE2} | cut -d' ' -f 1) $(stat -c%s ${PATCHFILE2}) $(basename ${PATCHFILE2})
+ SHA256-Current: $(sha256sum aptarchive/Packages | cut -d' ' -f 1) $(stat -c%s aptarchive/Packages)
+ SHA256-History:
+  01ba4719c80b6fe911b091a7c05124b64eeece964e09c058ef8f9805daca546b 33053002 2010-08-18-2013.28
+  $(sha256sum $PKGFILE | cut -d' ' -f 1) $(stat -c%s $PKGFILE) $(basename $PATCHFILE)
+  $(sha256sum ${PKGFILE}-new | cut -d' ' -f 1) $(stat -c%s ${PKGFILE}-new) $(basename ${PATCHFILE2})
+ SHA256-Patches:
+  e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855 19722 2010-08-18-2013.28
+  $(sha256sum $PATCHFILE | cut -d' ' -f 1) $(stat -c%s $PATCHFILE) $(basename $PATCHFILE)
+  $(sha256sum ${PATCHFILE2} | cut -d' ' -f 1) $(stat -c%s ${PATCHFILE2}) $(basename ${PATCHFILE2})" > $PATCHINDEX
        generatereleasefiles '+2hour'
        signreleasefiles
        cp -a aptarchive/Packages Packages-future
@@@ -147,8 -163,15 +163,15 @@@ SHA1-History
   9f4148e06d7faa37062994ff10d0c842d7017513 33053002 2010-08-18-2013.28
   $(sha1sum $PKGFILE | cut -d' ' -f 1) $(stat -c%s $PKGFILE) $(basename $PATCHFILE)
  SHA1-Patches:
-  7651fc0ac57cd83d41c63195a9342e2db5650257 19722 2010-08-18-0814.28
-  $(sha1sum $PATCHFILE | cut -d' ' -f 1) $(stat -c%s $PATCHFILE) $(basename $PATCHFILE)" > $PATCHINDEX
+  7651fc0ac57cd83d41c63195a9342e2db5650257 19722 2010-08-18-2013.28
+  $(sha1sum $PATCHFILE | cut -d' ' -f 1) $(stat -c%s $PATCHFILE) $(basename $PATCHFILE)
+ SHA256-Current: $(sha256sum ${PKGFILE}-new | cut -d' ' -f 1) $(stat -c%s ${PKGFILE}-new)
+ SHA256-History:
+  01ba4719c80b6fe911b091a7c05124b64eeece964e09c058ef8f9805daca546b 33053002 2010-08-18-2013.28
+  $(sha256sum $PKGFILE | cut -d' ' -f 1) $(stat -c%s $PKGFILE) $(basename $PATCHFILE)
+ SHA256-Patches:
+  e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855 19722 2010-08-18-2013.28
+  $(sha256sum $PATCHFILE | cut -d' ' -f 1) $(stat -c%s $PATCHFILE) $(basename $PATCHFILE)" > $PATCHINDEX
        echo 'I am Mallory and I change files' >> $PATCHFILE
        cat $PATCHFILE | gzip > ${PATCHFILE}.gz
        generatereleasefiles '+1hour'
  " aptcache show apt newstuff
  }
  echo 'Debug::pkgAcquire::Diffs "true";
 +Debug::Acquire::Transaction "true";
  Debug::pkgAcquire::rred "true";' > rootdir/etc/apt/apt.conf.d/rreddebug.conf
  
  testrun -o Acquire::PDiffs::Merge=0 -o APT::Get::List-Cleanup=1
  testrun -o Acquire::PDiffs::Merge=1 -o APT::Get::List-Cleanup=1
  testrun -o Acquire::PDiffs::Merge=0 -o APT::Get::List-Cleanup=0
  testrun -o Acquire::PDiffs::Merge=1 -o APT::Get::List-Cleanup=0
+ sha256sum() {
+       echo '01ba4719c80b6fe911b091a7c05124b64eeece964e09c058ef8f9805daca546b -'
+ }
+ testrun -o Acquire::PDiffs::Merge=0 -o Acquire::ForceHash=SHA1
+ testrun -o Acquire::PDiffs::Merge=1 -o Acquire::ForceHash=SHA1
+ unset -f sha256sum
+ sha1sum() {
+       echo 'adc83b19e793491b1c6ea0fd8b46cd9f32e592fc -'
+ }
+ testrun -o Acquire::PDiffs::Merge=0 -o Acquire::ForceHash=SHA256
+ testrun -o Acquire::PDiffs::Merge=1 -o Acquire::ForceHash=SHA256