]> git.saurik.com Git - apt.git/commitdiff
parse arch-qualified Provides correctly
authorHelmut Grohne <helmut@subdivi.de>
Mon, 9 Mar 2015 17:11:10 +0000 (18:11 +0100)
committerMichael Vogt <mvo@ubuntu.com>
Fri, 22 May 2015 14:14:54 +0000 (16:14 +0200)
The underlying problem is that libapt-pkg does not correctly parse these
provides. Internally, it creates a version named "baz:i386" with
architecture amd64. Of course, such a package name is invalid and thus
this version is completely inaccessible. Thus, this bug should not cause
apt to accept a broken situation as valid. Nevertheless, it prevents
using architecture qualified depends.

Closes: 777071
apt-pkg/deb/deblistparser.cc

index 4eef66c2b7bd1d0c736e69e2bbd6987990c6c23f..3941cf2f228f7b21538c66985560fc9236ab5740 100644 (file)
@@ -817,10 +817,16 @@ bool debListParser::ParseProvides(pkgCache::VerIterator &Ver)
       while (1)
       {
         Start = ParseDepends(Start,Stop,Package,Version,Op);
+        const size_t archfound = Package.rfind(':');
         if (Start == 0)
            return _error->Error("Problem parsing Provides line");
         if (Op != pkgCache::Dep::NoOp && Op != pkgCache::Dep::Equals) {
            _error->Warning("Ignoring Provides line with non-equal DepCompareOp for package %s", Package.c_str());
+        } else if (archfound != string::npos) {
+           string OtherArch = Package.substr(archfound+1, string::npos);
+           Package = Package.substr(0, archfound);
+           if (NewProvides(Ver, Package, OtherArch, Version) == false)
+              return false;
         } else if ((Ver->MultiArch & pkgCache::Version::Foreign) == pkgCache::Version::Foreign) {
            if (NewProvidesAllArch(Ver, Package, Version) == false)
               return false;