diff options
author | David Kalnischkies <david@kalnischkies.de> | 2021-09-12 16:08:52 +0200 |
---|---|---|
committer | David Kalnischkies <david@kalnischkies.de> | 2021-09-13 16:08:52 +0200 |
commit | 2b0369a5d1673d9e40f2af4db7677b040a26ee58 (patch) | |
tree | d1c588de1ad6534fc942dd117002e0dda1de15a1 /apt-pkg | |
parent | 883a12310a4130370965eab0a710a2c8fae6cc09 (diff) |
Read and work with canonical file-URIs from sources.lists
We allow file (and other file-based methods) URIs to either be given
as file:///path or as file:/path, but in various places of the acquire
system we perform string comparisons on URIs which do not handle this
expecting the canonical representation produced by our URI code.
That used to be hidden by us quoting and dequoting the URIs in the
system, but as we don't do this anymore we have to be a bit more careful
on input.
Ideally we would do less of these comparisons, but for now lets be
content with inserting a canonicalisation early on to prevent hangs in
the acquire system.
Diffstat (limited to 'apt-pkg')
-rw-r--r-- | apt-pkg/sourcelist.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/apt-pkg/sourcelist.cc b/apt-pkg/sourcelist.cc index cd7dbce9c..0ac59fcfc 100644 --- a/apt-pkg/sourcelist.cc +++ b/apt-pkg/sourcelist.cc @@ -85,12 +85,12 @@ bool pkgSourceList::Type::FixupURI(string &URI) const if (URI.find(':') == string::npos) return false; - URI = SubstVar(URI,"$(ARCH)",_config->Find("APT::Architecture")); - + URI = ::URI{SubstVar(URI, "$(ARCH)", _config->Find("APT::Architecture"))}; + // Make sure that the URI is / postfixed - if (URI[URI.size() - 1] != '/') - URI += '/'; - + if (URI.back() != '/') + URI.push_back('/'); + return true; } /*}}}*/ |