commit 91b66ce3048c3c2703b5762c1a7440fdc51755af
author Mike Frysinger <vapier@chromium.org> Sat May 16 11:49:15 2020 -0400
committer Commit Bot <commit-bot@chromium.org> Thu May 21 04:17:19 2020 +0000
tree 95e3df834a51d921659ed346562f2adb54df3ad2
parent 0063ddc95ae42bd805a46ffde1beafd0bbba3bcb

enter_chroot: delete ssh .pub key copy

This was added a long time ago for some test image logic in CL:3449,
but that has since been purged in CL:177667.  Clean up this bit of
code that doesn't help otherwise as only having access to .pub keys
in ~/.ssh doesn't grant access to anything.

BUG=chromium:306619
TEST=CQ passes

Change-Id: Ib6ccc04fb723faca922f246b4ec5f50fc7c59645
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosutils/+/2204978
Reviewed-by: Alex Klein <saklein@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>

sdk_lib/enter_chroot.sh

diff --git a/sdk_lib/enter_chroot.sh b/sdk_lib/enter_chroot.sh
index f80e371..543c8f9 100755
--- a/sdk_lib/enter_chroot.sh
+++ b/sdk_lib/enter_chroot.sh
@@ -509,15 +509,6 @@
           truncate -s 0 "${TARGET_DIR}/known_hosts"
           setup_mount "${known_hosts}" --bind "${target_ssh}/known_hosts"
         fi
-        (
-          # Only copy ~/.ssh/*.pub if they exist. Since we set
-          # nullglob, this needs to happen within a subshell.
-          shopt -s nullglob
-          files=("${SUDO_HOME}"/.ssh/*.pub)
-          if [[ ${#files[@]} -gt 0 ]]; then
-            user_cp "${files[@]}" "${TARGET_DIR}/"
-          fi
-        )
         copy_ssh_config "${TARGET_DIR}"
         chown -R ${SUDO_UID}:${SUDO_GID} "${TARGET_DIR}"