commit d554232bb6751372ec10d8aaae8d82ec044adb73
author Brad Bishop <bradleyb@fuzziesquirrel.com> Thu Jun 02 19:56:23 2022 -0400
committer Brad Bishop <bradleyb@fuzziesquirrel.com> Wed Jun 08 12:23:46 2022 -0400
tree 1c9dd78614e7692aca07d07bb48b065734852bca
parent cd7bf5290ea884aff3fdbd81494c5952d39ac002

mapper: remove service deny list feature

This feature has existed for years, but remains unused.  Drop the dead
code.

Change-Id: I1b0b6f7fee0da30a3e36e1d151e70e1039fe2c7b
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

src/main.cpp

diff --git a/src/main.cpp b/src/main.cpp
index 12969b1..86ef055 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -23,7 +23,6 @@
 AssociationMaps associationMaps;
 
 static AllowDenyList serviceAllowList;
-static AllowDenyList serviceDenyList;
 
 void updateOwners(sdbusplus::asio::connection* conn,
                   boost::container::flat_map<std::string, std::string>& owners,
@@ -264,7 +263,7 @@
 #endif
     sdbusplus::asio::object_server& objectServer)
 {
-    if (needToIntrospect(processName, serviceAllowList, serviceDenyList))
+    if (needToIntrospect(processName, serviceAllowList))
     {
         std::shared_ptr<InProgressIntrospect> transaction =
             std::make_shared<InProgressIntrospect>(systemBus, io, processName,
@@ -326,8 +325,7 @@
 #endif
             for (const std::string& processName : processNames)
             {
-                if (needToIntrospect(processName, serviceAllowList,
-                                     serviceDenyList))
+                if (needToIntrospect(processName, serviceAllowList))
                 {
                     startNewIntrospect(systemBus, io, interfaceMap, processName,
                                        assocMaps,
@@ -658,7 +656,6 @@
         std::make_shared<sdbusplus::asio::connection>(io);
 
     splitArgs(options["service-namespaces"], serviceAllowList);
-    splitArgs(options["service-blacklists"], serviceDenyList);
 
     sdbusplus::asio::object_server server(systemBus);
 
@@ -693,7 +690,7 @@
                     std::chrono::steady_clock::now());
 #endif
                 // New daemon added
-                if (needToIntrospect(name, serviceAllowList, serviceDenyList))
+                if (needToIntrospect(name, serviceAllowList))
                 {
                     nameOwners[newOwner] = name;
                     startNewIntrospect(systemBus.get(), io, interfaceMap, name,
@@ -721,7 +718,7 @@
             {
                 return; // only introspect well-known
             }
-            if (needToIntrospect(wellKnown, serviceAllowList, serviceDenyList))
+            if (needToIntrospect(wellKnown, serviceAllowList))
             {
                 processInterfaceAdded(interfaceMap, objPath, interfacesAdded,
                                       wellKnown, associationMaps, server);