diff options
author | djm <djm> | 2013-05-16 01:09:43 +0000 |
---|---|---|
committer | djm <djm> | 2013-05-16 01:09:43 +0000 |
commit | 69ce36ee2c459e97571a2f71f5372273cd7c5fd5 (patch) | |
tree | 7a64fc5e0b4e09fed7d8ed59ffa91d9dcd1f47f5 | |
parent | 357dde443564b564b83e6eadf5981fddac616875 (diff) | |
download | openssh-69ce36ee2c459e97571a2f71f5372273cd7c5fd5.tar.gz |
- (djm) [contrib/ssh-copy-id] Fix bug that could cause "rm *" to be
executed if mktemp failed; bz#2105 ok dtucker@
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | contrib/ssh-copy-id | 9 |
2 files changed, 12 insertions, 1 deletions
@@ -1,3 +1,7 @@ +20130516 + - (djm) [contrib/ssh-copy-id] Fix bug that could cause "rm *" to be + executed if mktemp failed; bz#2105 ok dtucker@ + 20130510 - (djm) OpenBSD CVS Cherrypick - djm@cvs.openbsd.org 2013/04/11 02:27:50 diff --git a/contrib/ssh-copy-id b/contrib/ssh-copy-id index af18a192..9f2817b6 100644 --- a/contrib/ssh-copy-id +++ b/contrib/ssh-copy-id @@ -165,6 +165,9 @@ done eval set -- "$SAVEARGS" +if [ $# == 0 ] ; then + usage +fi if [ $# != 1 ] ; then printf '%s: ERROR: Too many arguments. Expecting a target hostname, got: %s\n\n' "$0" "$SAVEARGS" >&2 usage @@ -196,7 +199,11 @@ populate_new_ids() { umask 0177 local L_TMP_ID_FILE=$(mktemp ~/.ssh/ssh-copy-id_id.XXXXXXXXXX) - trap "rm -f $L_TMP_ID_FILE*" EXIT TERM INT QUIT + if test $? -ne 0 || test "x$L_TMP_ID_FILE" = "x" ; then + echo "mktemp failed" 1>&2 + exit 1 + fi + trap "rm -f $L_TMP_ID_FILE ${L_TMP_ID_FILE}.pub" EXIT TERM INT QUIT printf '%s: INFO: attempting to log in with the new key(s), to filter out any that are already installed\n' "$0" >&2 NEW_IDS=$( eval $GET_ID | { |