The entire concept of PendingError() is flawed :/.

[apt.git] / apt-private / private-cacheset.cc

diff --git a/apt-private/private-cacheset.cc b/apt-private/private-cacheset.cc
index 439b844d50950e1eee6c04faeeed9853339df898..52cd22d2a56db3140c819a947d05d7ef7fb4de6a 100644 (file)
--- a/apt-private/private-cacheset.cc
+++ b/apt-private/private-cacheset.cc
@@ -30,13 +30,18 @@ bool GetLocalitySortedVersionSet(pkgCacheFile &CacheFile,
                                  Matcher &matcher,
                                  OpProgress * const progress)
 {
                                  Matcher &matcher,
                                  OpProgress * const progress)
 {

-   pkgCache *Cache = CacheFile.GetPkgCache();
-   pkgDepCache *DepCache = CacheFile.GetDepCache();
+   pkgCache * const Cache = CacheFile.GetPkgCache();
+   if (unlikely(Cache == nullptr))
+      return false;
+   if (progress != nullptr)
+      progress->SubProgress(Cache->Head().PackageCount, _("Sorting"));
+
+   pkgDepCache * const DepCache = CacheFile.GetDepCache();
+   if (unlikely(DepCache == nullptr))
+      return false;

    APT::CacheSetHelper helper(false);
 
    int Done=0;
    APT::CacheSetHelper helper(false);
 
    int Done=0;

-   if (progress != NULL)
-      progress->SubProgress(Cache->Head().PackageCount, _("Sorting"));

 
    bool const insertCurrentVer = _config->FindB("APT::Cmd::Installed", false);
    bool const insertUpgradable = _config->FindB("APT::Cmd::Upgradable", false);
 
    bool const insertCurrentVer = _config->FindB("APT::Cmd::Installed", false);
    bool const insertUpgradable = _config->FindB("APT::Cmd::Upgradable", false);


@@ -179,8 +184,8 @@ CacheSetHelperVirtuals::CacheSetHelperVirtuals(bool const ShowErrors, GlobalErro
                                                                        /*}}}*/
 
 // CacheSetHelperAPTGet - responsible for message telling from the CacheSets/*{{{*/
                                                                        /*}}}*/
 
 // CacheSetHelperAPTGet - responsible for message telling from the CacheSets/*{{{*/

-CacheSetHelperAPTGet::CacheSetHelperAPTGet(std::ostream &out) :
-   APT::CacheSetHelper{true}, out(out)
+CacheSetHelperAPTGet::CacheSetHelperAPTGet(std::ostream &pout) :
+   APT::CacheSetHelper{true}, out(pout)

 {
    explicitlyNamed = true;
 }
 {
    explicitlyNamed = true;
 }


@@ -245,8 +250,6 @@ bool CacheSetHelperAPTGet::showVirtualPackageErrors(pkgCacheFile &Cache)
                  "This may mean that the package is missing, has been obsoleted, or\n"
                  "is only available from another source\n"),Pkg.FullName(true).c_str());
 
                  "This may mean that the package is missing, has been obsoleted, or\n"
                  "is only available from another source\n"),Pkg.FullName(true).c_str());
 

-        std::string List;
-        std::string VersionsList;

         std::vector<bool> Seen(Cache.GetPkgCache()->Head().PackageCount, false);
         APT::PackageList pkglist;
         for (pkgCache::DepIterator Dep = Pkg.RevDependsList();
         std::vector<bool> Seen(Cache.GetPkgCache()->Head().PackageCount, false);
         APT::PackageList pkglist;
         for (pkgCache::DepIterator Dep = Pkg.RevDependsList();