adds binaries in path to collection

osxcollector/osxcollector.py

@@ -881,6 +881,8 @@ def __enter__(self):
         def __exit__(self, type, value, traceback):
             del Logger.Extra.extras[self.key]
 
+PATH_ENVIRONMENT_NAME = "PATH"
+
 
 class Collector(object):
 
@@ -913,6 +915,7 @@ def collect(self, section_list=None):
             ('safari', self._collect_safari),
             ('accounts', self._collect_accounts),
             ('mail', self._collect_mail),
+            ('executables', self._collect_binary_names_in_path),
             ('full_hash', self._collect_full_hash)
         ]
 
@@ -1205,6 +1208,22 @@ def _collect_system_info(self):
         }
         Logger.log_dict(record)
 
+    def _collect_binary_names_in_path(self):
+        """Collect the names of executable binaries in the PATH environment"""
+        exe_files = []
+
+        def is_exe(fpath):
+            return os.path.isfile(fpath) and os.access(fpath, os.X_OK)
+
+        if PATH_ENVIRONMENT_NAME in os.environ:
+            for bin_dir in os.environ[PATH_ENVIRONMENT_NAME].split(os.pathsep):
+                for root_dir, dirs, files in os.walk(bin_dir):
+                    for the_file in files:
+                        file_path = os.path.join(root_dir, the_file)
+                        if is_exe(file_path):
+                            exe_files.append(file_path)
+        Logger.log_dict({"executable_files": exe_files})
+
     def _collect_startup(self):
         """Log the different LauchAgents and LaunchDaemons"""
 

tests/osxcollector_collector_test.py

@@ -94,6 +94,27 @@ def test_log_packages_in_dir(self):
         self.collector._log_packages_in_dir('tests/data/packages/')
         self.mock_log_dict.assert_called_with(expected)
 
+    def test_collect_binary_names_in_path(self):
+        expected = ['/usr/bin/ls', '/usr/bin/pwd']
+
+        with patch(
+            'os.walk', autospec=True,
+            return_value=[
+                ['/usr', ('test',), ('bin/ls', 'bin/pwd', 'bin/tmp',)]
+            ]
+        ), patch(
+            'os.path.isfile', autospec=True,
+            side_effect=[True, True, False, True, True, False]
+        ), patch(
+            'os.access', autospec=True,
+            side_effect=[True, True, True, True, True, True]
+        ), patch.dict(
+            'os.environ', {'PATH': '/usr/bin'}
+        ):
+            self.collector._collect_binary_names_in_path()
+            self.mock_log_dict.assert_called_once_with({'executable_files':
+                                                        expected})
+
     def test_log_startup_items(self):
         list_of_files_in_dir = ['StartupParameters.plist']
         plist = {