[Schmitzm-commits] r1656 - trunk/schmitzm-core/src/main/java/de/schmitzm/crypt

[scm-commit@wald.intevation.org]

Author: keeb
Date: 2011-07-29 16:14:35 +0200 (Fri, 29 Jul 2011)
New Revision: 1656

Modified:
   trunk/schmitzm-core/src/main/java/de/schmitzm/crypt/HtPasswd.java
Log:
Not allowing hashes that start with whitespace for convenience

Modified: trunk/schmitzm-core/src/main/java/de/schmitzm/crypt/HtPasswd.java
===================================================================
--- trunk/schmitzm-core/src/main/java/de/schmitzm/crypt/HtPasswd.java	2011-07-29 10:42:05 UTC (rev 1655)
+++ trunk/schmitzm-core/src/main/java/de/schmitzm/crypt/HtPasswd.java	2011-07-29 14:14:35 UTC (rev 1656)
@@ -11,8 +11,9 @@
 import org.apache.commons.lang.StringUtils;
 
 /**
- * Diese Klasse erlaubt es auf eine .htpasswd Datei lesen und schreibend zuzugreifen. Nach jeder Aktion wird der Zugriff
- * auf die Datei wieder geschlossen.
+ * Diese Klasse erlaubt es auf eine .htpasswd Datei lesen und schreibend
+ * zuzugreifen. Nach jeder Aktion wird der Zugriff auf die Datei wieder
+ * geschlossen.
  */
 public class HtPasswd {
 
@@ -37,9 +38,15 @@
 	 */
 	public void setPasswd(String username, String cleartext) throws IOException {
 
-		final String salt = StringUtils.leftPad(String.valueOf(new Random().nextInt(99)), 2);
+		String salt = StringUtils.leftPad(
+				String.valueOf(new Random().nextInt(99)), 2);
 
 		String crypted = UnixCrypt.crypt(salt, cleartext);
+		while (crypted.startsWith(" ")) {
+			salt = StringUtils.leftPad(
+					String.valueOf(new Random().nextInt(99)), 2);
+			crypted = UnixCrypt.crypt(salt, cleartext);
+		}
 
 		TreeMap<String, String> users = readUsers();