From: Michael Vogt <mvo@debian.org>
Date: Tue, 8 Jul 2014 09:27:01 +0000 (+0200)
Subject: Do not clean "/" in pkgAcquire::Clean/pkgArchiveCleaner
X-Git-Tag: 1.0.6~14
X-Git-Url: https://git.saurik.com/apt.git/commitdiff_plain/10ecfe4f8a617ab285abd99d67917ae332ed2e4a

Do not clean "/" in pkgAcquire::Clean/pkgArchiveCleaner

Having "/" here is most likely a user configuration error and
may cause removal of import symlinks like /vmlinuz

Closes: #753531
---

diff --git a/apt-pkg/acquire.cc b/apt-pkg/acquire.cc
index a187a00ae..057bc24cd 100644
--- a/apt-pkg/acquire.cc
+++ b/apt-pkg/acquire.cc
@@ -486,6 +486,9 @@ bool pkgAcquire::Clean(string Dir)
    if (DirectoryExists(Dir) == false)
       return true;
 
+   if(Dir == "/")
+      return _error->Error(_("Clean of %s is not supported"), Dir.c_str());
+
    DIR *D = opendir(Dir.c_str());   
    if (D == 0)
       return _error->Errno("opendir",_("Unable to read %s"),Dir.c_str());
diff --git a/apt-pkg/clean.cc b/apt-pkg/clean.cc
index 0ee3b765d..37128e9aa 100644
--- a/apt-pkg/clean.cc
+++ b/apt-pkg/clean.cc
@@ -34,7 +34,10 @@
 bool pkgArchiveCleaner::Go(std::string Dir,pkgCache &Cache)
 {
    bool CleanInstalled = _config->FindB("APT::Clean-Installed",true);
-      
+
+   if(Dir == "/")
+      return _error->Error(_("Clean of %s is not supported"), Dir.c_str());
+
    DIR *D = opendir(Dir.c_str());
    if (D == 0)
       return _error->Errno("opendir",_("Unable to read %s"),Dir.c_str());