Fixed code duplication in T1105 sending and typo in report header

monkey/infection_monkey/exploit/sshexec.py

@@ -164,19 +164,20 @@ class SSHExploiter(HostExploiter):
                 ftp.putfo(file_obj, self._config.dropper_target_path_linux, file_size=monkeyfs.getsize(src_path),
                           callback=self.log_transfer)
                 ftp.chmod(self._config.dropper_target_path_linux, 0o777)
-                T1105Telem(ScanStatus.USED,
-                           get_interface_to_target(self.host.ip_addr[0]),
-                           self.host.ip_addr[0],
-                           src_path).send()
+                status = ScanStatus.USED
             ftp.close()
         except Exception as exc:
             LOG.debug("Error uploading file into victim %r: (%s)", self.host, exc)
-            T1105Telem(ScanStatus.SCANNED,
-                       get_interface_to_target(self.host.ip_addr[0]),
-                       self.host.ip_addr[0],
-                       src_path).send()
+            status = ScanStatus.SCANNED
+
+        T1105Telem(status,
+                   get_interface_to_target(self.host.ip_addr[0]),
+                   self.host.ip_addr[0],
+                   src_path).send()
+        if status == ScanStatus.SCANNED:
             return False
 
+
         try:
             cmdline = "%s %s" % (self._config.dropper_target_path_linux, MONKEY_ARG)
             cmdline += build_monkey_commandline(self.host, get_monkey_depth() - 1)

monkey/infection_monkey/post_breach/actions/users_custom_pba.py

@@ -82,17 +82,22 @@ class UsersPBA(PBA):
 
         pba_file_contents = ControlClient.get_pba_file(filename)
 
+        status = None
         if not pba_file_contents or not pba_file_contents.content:
             LOG.error("Island didn't respond with post breach file.")
-            T1105Telem(ScanStatus.SCANNED,
-                       WormConfiguration.current_server.split(':')[0],
-                       get_interface_to_target(WormConfiguration.current_server.split(':')[0]),
-                       filename).send()
-            return False
-        T1105Telem(ScanStatus.USED,
+            status = ScanStatus.SCANNED
+
+        if not status:
+            status = ScanStatus.USED
+
+        T1105Telem(status,
                    WormConfiguration.current_server.split(':')[0],
                    get_interface_to_target(WormConfiguration.current_server.split(':')[0]),
                    filename).send()
+
+        if status == ScanStatus.SCANNED:
+            return False
+
         try:
             with open(os.path.join(dst_dir, filename), 'wb') as written_PBA_file:
                 written_PBA_file.write(pba_file_contents.content)

monkey/monkey_island/cc/ui/src/components/attack/techniques/T1105.js

@@ -12,7 +12,7 @@ class T1105 extends React.Component {
 
   static getFilesColumns() {
     return ([{
-      Header: 'Files copied.',
+      Header: 'Files copied',
       columns: [
         {Header: 'Src. Machine', id: 'srcMachine', accessor: x => x.src, style: { 'whiteSpace': 'unset'}, width: 170 },
         {Header: 'Dst. Machine', id: 'dstMachine', accessor: x => x.dst, style: { 'whiteSpace': 'unset'}, width: 170},