[Schmitzm-commits] r1657 - branches/2.5/schmitzm-core/src/main/java/de/schmitzm/crypt

[scm-commit@wald.intevation.org]

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

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

Modified: branches/2.5/schmitzm-core/src/main/java/de/schmitzm/crypt/HtPasswd.java
===================================================================
--- branches/2.5/schmitzm-core/src/main/java/de/schmitzm/crypt/HtPasswd.java	2011-07-29 14:14:35 UTC (rev 1656)
+++ branches/2.5/schmitzm-core/src/main/java/de/schmitzm/crypt/HtPasswd.java	2011-07-29 14:14:49 UTC (rev 1657)
@@ -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();