Parcourir la source

rename StopAuthentication -> CheckStopAuthentication and make it protected

Michael Vogt il y a 11 ans
Parent
commit
2d0a7bb434
2 fichiers modifiés avec 11 ajouts et 9 suppressions
  1. 7 4
      apt-pkg/acquire-item.cc
  2. 4 5
      apt-pkg/acquire-item.h

+ 7 - 4
apt-pkg/acquire-item.cc

@@ -1502,9 +1502,12 @@ void pkgAcqMetaBase::TransactionStageRemoval(Item *I,
 // AcqMetaBase::GenerateAuthWarning - Check gpg authentication error	/*{{{*/
 // ---------------------------------------------------------------------
 /* */
-bool pkgAcqMetaBase::StopAuthentication(const std::string &RealURI,
-                                        const std::string &Message)
+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))
@@ -1626,7 +1629,7 @@ void pkgAcqMetaSig::Failed(string Message,pkgAcquire::MethodConfig *Cnf)/*{{{*/
    string Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 
    // check if we need to fail at this point 
-   if (AuthPass == true && StopAuthentication(RealURI, Message))
+   if (AuthPass == true && CheckStopAuthentication(RealURI, Message))
          return;
 
    // FIXME: meh, this is not really elegant
@@ -2154,7 +2157,7 @@ void pkgAcqMetaClearSig::Failed(string Message,pkgAcquire::MethodConfig *Cnf) /*
    }
    else
    {
-      if(StopAuthentication(RealURI, Message))
+      if(CheckStopAuthentication(RealURI, Message))
          return;
 
       _error->Warning(_("The data from '%s' is not signed. Packages "

+ 4 - 5
apt-pkg/acquire-item.h

@@ -413,6 +413,10 @@ class pkgAcqMetaBase  : public pkgAcquire::Item
     */
    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.
     *
@@ -436,11 +440,6 @@ class pkgAcqMetaBase  : public pkgAcquire::Item
     */
    void TransactionStageRemoval(Item *I, const std::string &FinalFile);
 
-   /** Check if the current item should fail at this point */
-   bool StopAuthentication(const std::string &RealURI,
-                           const std::string &Message);
-
-
    pkgAcqMetaBase(pkgAcquire *Owner,
                   const std::vector<IndexTarget*>* IndexTargets,
                   indexRecords* MetaIndexParser,