From 69b7654484c3f16d61f315275abb15cc062d190e Mon Sep 17 00:00:00 2001 From: David Kalnischkies Date: Thu, 10 Sep 2015 22:44:01 +0200 Subject: [PATCH] fix 'Dead assignment' by dropping unneeded boolean Reported-By: scan-build Git-Dch: Ignore --- apt-pkg/acquire.cc | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/apt-pkg/acquire.cc b/apt-pkg/acquire.cc index 433a2a6fa..6a800002e 100644 --- a/apt-pkg/acquire.cc +++ b/apt-pkg/acquire.cc @@ -467,9 +467,8 @@ static void CheckDropPrivsMustBeDisabled(pkgAcquire const &Fetcher) if (seteuid(pw->pw_uid) != 0) _error->Errno("seteuid", "seteuid %u failed", pw->pw_uid); - bool dropPrivs = true; for (pkgAcquire::ItemCIterator I = Fetcher.ItemsBegin(); - I != Fetcher.ItemsEnd() && dropPrivs == true; ++I) + I != Fetcher.ItemsEnd(); ++I) { std::string filename = (*I)->DestFile; if (filename.empty()) @@ -496,7 +495,6 @@ static void CheckDropPrivsMustBeDisabled(pkgAcquire const &Fetcher) if (faccessat(-1, dirname.c_str(), R_OK | W_OK | X_OK, AT_EACCESS | AT_SYMLINK_NOFOLLOW) != 0) { - dropPrivs = false; _error->WarningE("pkgAcquire::Run", _("Can't drop privileges for downloading as file '%s' couldn't be accessed by user '%s'."), filename.c_str(), SandboxUser.c_str()); _config->Set("APT::Sandbox::User", ""); -- 2.45.2