Re: [Ecryptfs-devel] [PATCH] ecryptfs-setup-private: don't echo passwords to screen

2008-09-08 Thread Michael Halcrow
On Wed, Aug 20, 2008 at 11:45:21PM +0100, Dustin Kirkland wrote:
 [PATCH] ecryptfs-setup-private: don't echo passwords to screen

Merged.

 This patch fixes some mostly debug code I've used for a while.  It's
 ready for prime time now, and shouldn't really be displaying these on
 standard out for should surfers to see.

 diff -upr ecryptfs-utils-53/src/utils/ecryptfs-setup-private 
 ecryptfs-utils-53.new/src/utils/ecryptfs-setup-private
 --- ecryptfs-utils-53/src/utils/ecryptfs-setup-private2008-08-20 
 23:06:09.220683144 +0100
 +++ ecryptfs-utils-53.new/src/utils/ecryptfs-setup-private2008-08-20 
 23:13:45.051913720 +0100
 @@ -155,6 +155,7 @@ if [ -z $MOUNTPASS ]; then
   # Pull 128 bits of random data from /dev/urandom, and 
 convert
   # to a string of 32 hex digits
   MOUNTPASS=`head -c 16 /dev/urandom | od -x | head -n 1 
 |sed s/^000// | sed s/\s*//g`
 + RANDOM_MOUNTPASS=1
   break
   else
   stty -echo
 @@ -171,21 +172,24 @@ if [ -z $MOUNTPASS ]; then
   done
  fi
  
 -echo
 -echo
 -echo Using username [$USER]
 -echo Using mount passphrase [$MOUNTPASS]
 -echo Using login passphrase [$LOGINPASS]
 -echo Using mount point [$MOUNTPOINT]
 -echo Using encrypted dir [$CRYPTDIR]
 -echo
 -echo This script will attempt to set up your system to mount
 -echo $MOUNTPOINT with eCryptfs automatically on login,
 -echo using your login passphrase.
 +#echo
 +#echo Using username [$USER]
 +#echo Using mount passphrase [$MOUNTPASS]
 +#echo Using login passphrase [$LOGINPASS]
 +#echo Using mount point [$MOUNTPOINT]
 +#echo Using encrypted dir [$CRYPTDIR]
 +#echo
 +#echo This script will attempt to set up your system to mount
 +#echo $MOUNTPOINT with eCryptfs automatically on login,
 +#echo using your login passphrase.
  echo
  echo 
 
 -echo YOU SHOULD RECORD THIS MOUNT PASSPHRASE AND STORE IN A SAFE LOCATION:
 -echo $MOUNTPASS
 +if [ $RANDOM_MOUNTPASS = 1 ]; then
 + echo YOU SHOULD RECORD THIS MOUNT PASSPHRASE AND STORE IN A SAFE 
 LOCATION:
 + echo $MOUNTPASS
 +else
 + echo YOU SHOULD RECORD YOUR MOUNT PASSPHRASE AND STORE IN A SAFE 
 LOCATION:
 +fi
  echo THIS WILL BE REQUIRED IF YOU NEED TO RECOVER YOUR DATA AT A LATER 
 TIME.
  echo 
 
  echo

 -
 This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
 Build the coolest Linux based applications with Moblin SDK  win great prizes
 Grand prize is a trip for two to an Open Source event anywhere in the world
 http://moblin-contest.org/redirect.php?banner_id=100url=/
 ___
 eCryptfs-devel mailing list
 eCryptfs-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/ecryptfs-devel



pgpQ4LX2bEg0N.pgp
Description: PGP signature
-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/___
eCryptfs-devel mailing list
eCryptfs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ecryptfs-devel


[Ecryptfs-devel] [PATCH] ecryptfs-setup-private: don't echo passwords to screen

2008-08-20 Thread Dustin Kirkland
[PATCH] ecryptfs-setup-private: don't echo passwords to screen

This patch fixes some mostly debug code I've used for a while.  It's
ready for prime time now, and shouldn't really be displaying these on
standard out for should surfers to see.

-- 
:-Dustin
diff -upr ecryptfs-utils-53/src/utils/ecryptfs-setup-private ecryptfs-utils-53.new/src/utils/ecryptfs-setup-private
--- ecryptfs-utils-53/src/utils/ecryptfs-setup-private	2008-08-20 23:06:09.220683144 +0100
+++ ecryptfs-utils-53.new/src/utils/ecryptfs-setup-private	2008-08-20 23:13:45.051913720 +0100
@@ -155,6 +155,7 @@ if [ -z $MOUNTPASS ]; then
 			# Pull 128 bits of random data from /dev/urandom, and convert
 			# to a string of 32 hex digits
 			MOUNTPASS=`head -c 16 /dev/urandom | od -x | head -n 1 |sed s/^000// | sed s/\s*//g`
+			RANDOM_MOUNTPASS=1
 			break
 		else
 			stty -echo
@@ -171,21 +172,24 @@ if [ -z $MOUNTPASS ]; then
 	done
 fi
 
-echo
-echo
-echo Using username [$USER]
-echo Using mount passphrase [$MOUNTPASS]
-echo Using login passphrase [$LOGINPASS]
-echo Using mount point [$MOUNTPOINT]
-echo Using encrypted dir [$CRYPTDIR]
-echo
-echo This script will attempt to set up your system to mount
-echo $MOUNTPOINT with eCryptfs automatically on login,
-echo using your login passphrase.
+#echo
+#echo Using username [$USER]
+#echo Using mount passphrase [$MOUNTPASS]
+#echo Using login passphrase [$LOGINPASS]
+#echo Using mount point [$MOUNTPOINT]
+#echo Using encrypted dir [$CRYPTDIR]
+#echo
+#echo This script will attempt to set up your system to mount
+#echo $MOUNTPOINT with eCryptfs automatically on login,
+#echo using your login passphrase.
 echo
 echo 
-echo YOU SHOULD RECORD THIS MOUNT PASSPHRASE AND STORE IN A SAFE LOCATION:
-echo $MOUNTPASS
+if [ $RANDOM_MOUNTPASS = 1 ]; then
+	echo YOU SHOULD RECORD THIS MOUNT PASSPHRASE AND STORE IN A SAFE LOCATION:
+	echo $MOUNTPASS
+else
+	echo YOU SHOULD RECORD YOUR MOUNT PASSPHRASE AND STORE IN A SAFE LOCATION:
+fi
 echo THIS WILL BE REQUIRED IF YOU NEED TO RECOVER YOUR DATA AT A LATER TIME.
 echo 
 echo
-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK  win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100url=/___
eCryptfs-devel mailing list
eCryptfs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ecryptfs-devel