]> git.sur5r.net Git - openldap/blobdiff - libraries/liblutil/passwd.c
Fix ldaps / TLS processing...
[openldap] / libraries / liblutil / passwd.c
index fcf70c8e6b0f72101cd6063cd24420db81e288ef..a1b3201dd8c06ace913f1acf8f62e8384e85efc2 100644 (file)
@@ -4,10 +4,13 @@
  * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
  */
 /*
- * lutil_password(credentials, password)
+ * int lutil_passwd(
+ *     const struct berval *passwd,
+ *     const struct berval *cred,
+ *     const char **schemes )
  *
- * Returns true if user supplied credentials matches
- * the stored password. 
+ * Returns true if user supplied credentials (cred) matches
+ * the stored password (passwd)
  *
  * Due to the use of the crypt(3) function 
  * this routine is NOT thread-safe.
 
 #include <stdio.h>
 #include <ac/stdlib.h>
-
 #include <ac/string.h>
-#include <ac/unistd.h>
-#include <ac/crypt.h>
 
-#include <lber.h>
+#ifdef SLAPD_SPASSWD
+#      include <sasl.h>
+#endif
 
-#include "lutil_md5.h"
-#include "lutil_sha1.h"
-#include "lutil.h"
+#ifdef SLAPD_KPASSWD
+#      include <ac/krb.h>
+#      include <ac/krb5.h>
+#endif
+
+#include <ac/param.h>
+
+#include <ac/unistd.h>
+#include <ac/crypt.h>
 
 #ifdef HAVE_SHADOW_H
 #      include <shadow.h>
 #ifdef HAVE_PWD_H
 #      include <pwd.h>
 #endif
+#ifdef HAVE_AIX_SECURITY
+#      include <userpw.h>
+#endif
+
+#include <lber.h>
+
+#include "lutil_md5.h"
+#include "lutil_sha1.h"
+#include "lutil.h"
+
+static const unsigned char crypt64[] =
+       "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890./";
 
 struct pw_scheme;
 
@@ -42,14 +62,14 @@ typedef int (*PASSWD_CHK_FUNC)(
        const struct berval *passwd,
        const struct berval *cred );
 
-typedef struct berval * (*PASSWD_GEN_FUNC) (
+typedef struct berval * (*PASSWD_HASH_FUNC) (
        const struct pw_scheme *scheme,
        const struct berval *passwd );
 
 struct pw_scheme {
        struct berval name;
        PASSWD_CHK_FUNC chk_fn;
-       PASSWD_GEN_FUNC gen_fn;
+       PASSWD_HASH_FUNC hash_fn;
 };
 
 /* password check routines */
@@ -73,53 +93,80 @@ static int chk_sha1(
        const struct berval *passwd,
        const struct berval *cred );
 
+#ifdef SLAPD_SPASSWD
+static int chk_sasl(
+       const struct pw_scheme *scheme,
+       const struct berval *passwd,
+       const struct berval *cred );
+#endif
+
+#ifdef SLAPD_KPASSWD
+static int chk_kerberos(
+       const struct pw_scheme *scheme,
+       const struct berval *passwd,
+       const struct berval *cred );
+#endif
+
+#ifdef SLAPD_CRYPT
 static int chk_crypt(
        const struct pw_scheme *scheme,
        const struct berval *passwd,
        const struct berval *cred );
 
+#if defined( HAVE_GETPWNAM ) && defined( HAVE_PW_PASSWD )
 static int chk_unix(
        const struct pw_scheme *scheme,
        const struct berval *passwd,
        const struct berval *cred );
+#endif
+#endif
 
 
-/* password generation routines */
-static struct berval *gen_sha1(
+/* password hash routines */
+static struct berval *hash_sha1(
        const struct pw_scheme *scheme,
        const struct berval *passwd );
 
-static struct berval *gen_ssha1(
+static struct berval *hash_ssha1(
        const struct pw_scheme *scheme,
        const struct berval *passwd );
 
-static struct berval *gen_smd5(
+static struct berval *hash_smd5(
        const struct pw_scheme *scheme,
        const struct berval *passwd );
 
-static struct berval *gen_md5(
+static struct berval *hash_md5(
        const struct pw_scheme *scheme,
        const struct berval *passwd );
 
-static struct berval *gen_crypt(
+#ifdef SLAPD_CRYPT
+static struct berval *hash_crypt(
        const struct pw_scheme *scheme,
        const struct berval *passwd );
+#endif
 
 
 static const struct pw_scheme pw_schemes[] =
 {
-       { {sizeof("{SSHA}")-1, "{SSHA}"},       chk_ssha1, gen_ssha1 },
-       { {sizeof("{SHA}")-1, "{SHA}"},         chk_sha1, gen_sha1 },
+       { {sizeof("{SSHA}")-1, "{SSHA}"},       chk_ssha1, hash_ssha1 },
+       { {sizeof("{SHA}")-1, "{SHA}"},         chk_sha1, hash_sha1 },
 
-       { {sizeof("{SMD5}")-1, "{SMD5}"},       chk_smd5, gen_smd5 },
-       { {sizeof("{MD5}")-1, "{MD5}"},         chk_md5, gen_md5 },
+       { {sizeof("{SMD5}")-1, "{SMD5}"},       chk_smd5, hash_smd5 },
+       { {sizeof("{MD5}")-1, "{MD5}"},         chk_md5, hash_md5 },
 
-#ifdef SLAPD_CRYPT
-       { {sizeof("{CRYPT}")-1, "{CRYPT}"},     chk_crypt, gen_crypt },
+#ifdef SLAPD_SPASSWD
+       { {sizeof("{SASL}")-1, "{SASL}"}, chk_sasl, NULL },
+#endif
+
+#ifdef SLAPD_KPASSWD
+       { {sizeof("{KERBEROS}")-1, "{KERBEROS}"}, chk_kerberos, NULL },
 #endif
-# if defined( HAVE_GETSPNAM ) \
-  || ( defined( HAVE_GETPWNAM ) && defined( HAVE_PW_PASSWD ) )
+
+#ifdef SLAPD_CRYPT
+       { {sizeof("{CRYPT}")-1, "{CRYPT}"},     chk_crypt, hash_crypt },
+# if defined( HAVE_GETPWNAM ) && defined( HAVE_PW_PASSWD )
        { {sizeof("{UNIX}")-1, "{UNIX}"},       chk_unix, NULL },
+# endif
 #endif
 
 #ifdef SLAPD_CLEARTEXT
@@ -247,16 +294,48 @@ lutil_passwd(
 
 }
 
-struct berval * lutil_passwd_generate(
+struct berval * lutil_passwd_generate( ber_len_t len )
+{
+       struct berval *pw;
+
+       if( len < 1 ) return NULL;
+
+       pw = ber_memalloc( sizeof( struct berval ) );
+       if( pw == NULL ) return NULL;
+
+       pw->bv_len = len;
+       pw->bv_val = ber_memalloc( len + 1 );
+
+       if( pw->bv_val == NULL ) {
+               ber_memfree( pw );
+               return NULL;
+       }
+
+       if( lutil_entropy( pw->bv_val, pw->bv_len) < 0 ) {
+               ber_bvfree( pw );
+               return NULL; 
+       }
+
+       for( len = 0; len < pw->bv_len; len++ ) {
+               pw->bv_val[len] = crypt64[
+                       pw->bv_val[len] % (sizeof(crypt64)-1) ];
+       }
+
+       pw->bv_val[len] = '\0';
+       
+       return pw;
+}
+
+struct berval * lutil_passwd_hash(
        const struct berval * passwd,
        const char * method )
 {
        const struct pw_scheme *sc = get_scheme( method );
 
        if( sc == NULL ) return NULL;
-       if( ! sc->gen_fn ) return NULL;
+       if( ! sc->hash_fn ) return NULL;
 
-       return (sc->gen_fn)( sc, passwd );
+       return (sc->hash_fn)( sc, passwd );
 }
 
 static struct berval * pw_string(
@@ -329,15 +408,16 @@ static struct berval * pw_string64(
                string.bv_val, string.bv_len,
                &b64->bv_val[sc->name.bv_len], b64len );
 
-       b64->bv_val[b64->bv_len] = '\0';
-
        if( salt ) ber_memfree( string.bv_val );
-
+       
        if( rc < 0 ) {
                ber_bvfree( b64 );
                return NULL;
        }
 
+       /* recompute length */
+       b64->bv_len = sc->name.bv_len + rc;
+       assert( strlen(b64->bv_val) == b64->bv_len );
        return b64;
 }
 
@@ -353,7 +433,7 @@ static int chk_ssha1(
        int rc;
        unsigned char *orig_pass = NULL;
  
-       /* base64 un-encode password */
+       /* decode base64 password */
        orig_pass = (unsigned char *) ber_memalloc( (size_t) (
                LUTIL_BASE64_DECODE_LEN(passwd->bv_len) + 1) );
 
@@ -363,7 +443,7 @@ static int chk_ssha1(
 
        if(rc < 0) {
                ber_memfree(orig_pass);
-               return 1;
+               return -1;
        }
  
        /* hash credentials with salt */
@@ -378,7 +458,7 @@ static int chk_ssha1(
        /* compare */
        rc = memcmp((char *)orig_pass, (char *)SHA1digest, sizeof(SHA1digest));
        ber_memfree(orig_pass);
-       return rc;
+       return rc ? 1 : 0;
 }
 
 static int chk_sha1(
@@ -401,7 +481,7 @@ static int chk_sha1(
 
        if( rc != sizeof(SHA1digest) ) {
                ber_memfree(orig_pass);
-               return 1;
+               return -1;
        }
  
        /* hash credentials with salt */
@@ -413,7 +493,7 @@ static int chk_sha1(
        /* compare */
        rc = memcmp((char *)orig_pass, (char *)SHA1digest, sizeof(SHA1digest));
        ber_memfree(orig_pass);
-       return rc;
+       return rc ? 1 : 0;
 }
 
 static int chk_smd5(
@@ -435,7 +515,7 @@ static int chk_smd5(
        rc = lutil_b64_pton(passwd->bv_val, orig_pass, passwd->bv_len);
        if ( rc < 0 ) {
                ber_memfree(orig_pass);
-               return 1;
+               return -1;
        }
 
        /* hash credentials with salt */
@@ -450,7 +530,7 @@ static int chk_smd5(
        /* compare */
        rc = memcmp((char *)orig_pass, (char *)MD5digest, sizeof(MD5digest));
        ber_memfree(orig_pass);
-       return rc;
+       return rc ? 1 : 0;
 }
 
 static int chk_md5(
@@ -472,7 +552,7 @@ static int chk_md5(
        rc = lutil_b64_pton(passwd->bv_val, orig_pass, passwd->bv_len);
        if ( rc != sizeof(MD5digest) ) {
                ber_memfree(orig_pass);
-               return 1;
+               return -1;
        }
 
        /* hash credentials with salt */
@@ -484,70 +564,353 @@ static int chk_md5(
        /* compare */
        rc = memcmp((char *)orig_pass, (char *)MD5digest, sizeof(MD5digest));
        ber_memfree(orig_pass);
-       return rc;
+       return rc ? 1 : 0;
 }
 
-#ifdef SLAPD_CRYPT
-static int chk_crypt(
+#ifdef SLAPD_SPASSWD
+#ifdef HAVE_CYRUS_SASL
+sasl_conn_t *lutil_passwd_sasl_conn = NULL;
+#endif
+
+static int chk_sasl(
        const struct pw_scheme *sc,
        const struct berval * passwd,
        const struct berval * cred )
 {
-       return strcmp(passwd, crypt(cred, passwd));
+       int i;
+       int rtn;
+
+       for( i=0; i<cred->bv_len; i++) {
+               if(cred->bv_val[i] == '\0') {
+                       return 1;       /* NUL character in password */
+               }
+       }
+
+       if( cred->bv_val[i] != '\0' ) {
+               return 1;       /* cred must behave like a string */
+       }
+
+       for( i=0; i<passwd->bv_len; i++) {
+               if(passwd->bv_val[i] == '\0') {
+                       return 1;       /* NUL character in password */
+               }
+       }
+
+       if( passwd->bv_val[i] != '\0' ) {
+               return 1;       /* passwd must behave like a string */
+       }
+
+       rtn = 1;
+
+#ifdef HAVE_CYRUS_SASL
+       if( lutil_passwd_sasl_conn != NULL ) {
+               const char *errstr = NULL;
+               int sc;
+
+               sc = sasl_checkpass( lutil_passwd_sasl_conn,
+                       passwd->bv_val, passwd->bv_len,
+                       cred->bv_val, cred->bv_len,
+                       &errstr );
+
+               rtn = ( sc != SASL_OK );
+       }
+#endif
+
+       return rtn;
 }
+#endif
 
-# if defined( HAVE_GETSPNAM ) \
-  || ( defined( HAVE_GETPWNAM ) && defined( HAVE_PW_PASSWD ) )
-static int chk_unix(
+#ifdef SLAPD_KPASSWD
+static int chk_kerberos(
        const struct pw_scheme *sc,
        const struct berval * passwd,
        const struct berval * cred )
 {
        int i;
-       char *pw;
+       int rtn;
 
        for( i=0; i<cred->bv_len; i++) {
                if(cred->bv_val[i] == '\0') {
                        return 1;       /* NUL character in password */
                }
        }
+
        if( cred->bv_val[i] != '\0' ) {
                return 1;       /* cred must behave like a string */
        }
 
-#  ifdef HAVE_GETSPNAM
+       for( i=0; i<passwd->bv_len; i++) {
+               if(passwd->bv_val[i] == '\0') {
+                       return 1;       /* NUL character in password */
+               }
+       }
+
+       if( passwd->bv_val[i] != '\0' ) {
+               return 1;       /* passwd must behave like a string */
+       }
+
+       rtn = 1;
+
+#ifdef HAVE_KRB5 /* HAVE_HEIMDAL_KRB5 */
+       {
+/* Portions:
+ * Copyright (c) 1997, 1998, 1999 Kungliga Tekniska H\xf6gskolan
+ * (Royal Institute of Technology, Stockholm, Sweden).
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ *
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * 3. Neither the name of the Institute nor the names of its contributors
+ *    may be used to endorse or promote products derived from this software
+ *    without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE INSTITUTE AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED.  IN NO EVENT SHALL THE INSTITUTE OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
+
+               krb5_context context;
+               krb5_error_code ret;
+               krb5_creds creds;
+               krb5_get_init_creds_opt get_options;
+               krb5_verify_init_creds_opt verify_options;
+               krb5_principal client, server;
+#ifdef notdef
+               krb5_preauthtype pre_auth_types[] = {KRB5_PADATA_ENC_TIMESTAMP};
+#endif
+
+               ret = krb5_init_context( &context );
+               if (ret) {
+                       return 1;
+               }
+
+#ifdef notdef
+               krb5_get_init_creds_opt_set_preauth_list(&get_options,
+                       pre_auth_types, 1);
+#endif
+
+               krb5_get_init_creds_opt_init( &get_options );
+
+               krb5_verify_init_creds_opt_init( &verify_options );
+       
+               ret = krb5_parse_name( context, passwd->bv_val, &client );
+
+               if (ret) {
+                       krb5_free_context( context );
+                       return 1;
+               }
+
+               ret = krb5_get_init_creds_password( context,
+                       &creds, client, cred->bv_val, NULL,
+                       NULL, 0, NULL, &get_options );
+
+               if (ret) {
+                       krb5_free_principal( context, client );
+                       krb5_free_context( context );
+                       return 1;
+               }
+
+               {
+                       char host[MAXHOSTNAMELEN];
+
+                       if( gethostname( host, MAXHOSTNAMELEN ) != 0 ) {
+                               krb5_free_principal( context, client );
+                               krb5_free_context( context );
+                               return 1;
+                       }
+
+                       ret = krb5_sname_to_principal( context,
+                               host, "ldap", KRB5_NT_SRV_HST, &server );
+               }
+
+               if (ret) {
+                       krb5_free_principal( context, client );
+                       krb5_free_context( context );
+                       return 1;
+               }
+
+               ret = krb5_verify_init_creds( context,
+                       &creds, server, NULL, NULL, &verify_options );
+
+               krb5_free_principal( context, client );
+               krb5_free_principal( context, server );
+               krb5_free_creds_contents( context, &creds );
+               krb5_free_context( context );
+
+               rtn = !!ret;
+       }
+#elif  defined(HAVE_KRB4)
        {
-               struct spwd *spwd = getspnam(p);
+               /* Borrowed from Heimdal kpopper */
+/* Portions:
+ * Copyright (c) 1989 Regents of the University of California.
+ * All rights reserved.  The Berkeley software License Agreement
+ * specifies the terms and conditions for redistribution.
+ */
+
+               int status;
+               char lrealm[REALM_SZ];
+               char tkt[MAXHOSTNAMELEN];
+
+               status = krb_get_lrealm(lrealm,1);
+               if (status == KFAILURE) {
+                       return 1;
+               }
+
+               snprintf(tkt, sizeof(tkt), "%s_slapd.%u",
+                       TKT_ROOT, (unsigned)getpid());
+               krb_set_tkt_string (tkt);
+
+               status = krb_verify_user( passwd->bv_val, "", lrealm,
+                       cred->bv_val, 1, "ldap");
+
+               dest_tkt(); /* no point in keeping the tickets */
+
+               return status == KFAILURE;
+       }
+#endif
+
+       return rtn;
+}
+#endif /* SLAPD_KPASSWD */
+
+#ifdef SLAPD_CRYPT
+static int chk_crypt(
+       const struct pw_scheme *sc,
+       const struct berval * passwd,
+       const struct berval * cred )
+{
+       char *cr;
+       int i;
+
+       for( i=0; i<cred->bv_len; i++) {
+               if(cred->bv_val[i] == '\0') {
+                       return 1;       /* NUL character in password */
+               }
+       }
+
+       if( cred->bv_val[i] != '\0' ) {
+               return -1;      /* cred must behave like a string */
+       }
+
+       if( passwd->bv_len < 2 ) {
+               return -1;      /* passwd must be at least two characters long */
+       }
 
-               if(spwd == NULL) {
-                       return 1;       /* not found */
+       for( i=0; i<passwd->bv_len; i++) {
+               if(passwd->bv_val[i] == '\0') {
+                       return -1;      /* NUL character in password */
                }
+       }
 
-               pw = spwd->sp_pwdp;
+       if( passwd->bv_val[i] != '\0' ) {
+               return -1;      /* passwd must behave like a string */
+       }
+
+       cr = crypt( cred->bv_val, passwd->bv_val );
+
+       if( cr == NULL || cr[0] == '\0' ) {
+               /* salt must have been invalid */
+               return -1;
+       }
+
+       return strcmp( passwd->bv_val, cr ) ? 1 : 0;
+}
+
+# if defined( HAVE_GETPWNAM ) && defined( HAVE_PW_PASSWD )
+static int chk_unix(
+       const struct pw_scheme *sc,
+       const struct berval * passwd,
+       const struct berval * cred )
+{
+       int i;
+       char *pw,*cr;
+
+       for( i=0; i<cred->bv_len; i++) {
+               if(cred->bv_val[i] == '\0') {
+                       return -1;      /* NUL character in password */
+               }
+       }
+       if( cred->bv_val[i] != '\0' ) {
+               return -1;      /* cred must behave like a string */
+       }
+
+       for( i=0; i<passwd->bv_len; i++) {
+               if(passwd->bv_val[i] == '\0') {
+                       return -1;      /* NUL character in password */
+               }
+       }
+
+       if( passwd->bv_val[i] != '\0' ) {
+               return -1;      /* passwd must behave like a string */
        }
 
-#  else
        {
-               struct passwd *pwd = getpwnam(p);
+               struct passwd *pwd = getpwnam(passwd->bv_val);
 
                if(pwd == NULL) {
-                       return 1;       /* not found */
+                       return -1;      /* not found */
                }
 
                pw = pwd->pw_passwd;
        }
+#  ifdef HAVE_GETSPNAM
+       {
+               struct spwd *spwd = getspnam(passwd->bv_val);
+
+               if(spwd != NULL) {
+                       pw = spwd->sp_pwdp;
+               }
+       }
+#  endif
+#  ifdef HAVE_AIX_SECURITY
+       {
+               struct userpw *upw = getuserpw(passwd->bv_val);
+
+               if (upw != NULL) {
+                       pw = upw->upw_passwd;
+               }
+       }
 #  endif
 
-       if( pw == NULL || *pw == '\0' ) return 1;
+       if( pw == NULL || pw[0] == '\0' || pw[1] == '\0' ) {
+               /* password must must be at least two characters long */
+               return -1;
+       }
+
+       cr = crypt(cred->bv_val, pw);
 
-       return strcmp(pw, crypt(cred->bv_val, pw));
+       if( cr == NULL || cr[0] == '\0' ) {
+               /* salt must have been invalid */
+               return -1;
+       }
+
+       return strcmp(pw, cr) ? 1 : 0;
 
 }
 # endif
 #endif
 
-/* PASSWORD CHECK ROUTINES */
-static struct berval *gen_ssha1(
+/* PASSWORD GENERATION ROUTINES */
+
+static struct berval *hash_ssha1(
        const struct pw_scheme *scheme,
        const struct berval  *passwd )
 {
@@ -576,7 +939,7 @@ static struct berval *gen_ssha1(
        return pw_string64( scheme, &digest, &salt);
 }
 
-static struct berval *gen_sha1(
+static struct berval *hash_sha1(
        const struct pw_scheme *scheme,
        const struct berval  *passwd )
 {
@@ -594,7 +957,7 @@ static struct berval *gen_sha1(
        return pw_string64( scheme, &digest, NULL);
 }
 
-static struct berval *gen_smd5(
+static struct berval *hash_smd5(
        const struct pw_scheme *scheme,
        const struct berval  *passwd )
 {
@@ -623,7 +986,7 @@ static struct berval *gen_smd5(
        return pw_string64( scheme, &digest, &salt );
 }
 
-static struct berval *gen_md5(
+static struct berval *hash_md5(
        const struct pw_scheme *scheme,
        const struct berval  *passwd )
 {
@@ -645,15 +1008,23 @@ static struct berval *gen_md5(
 }
 
 #ifdef SLAPD_CRYPT
-static struct berval *gen_crypt(
+static struct berval *hash_crypt(
        const struct pw_scheme *scheme,
        const struct berval *passwd )
 {
-       static const unsigned char crypt64[] =
-               "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890./";
+       struct berval hash;
+       unsigned char salt[3];
+       int i;
+
+       for( i=0; i<passwd->bv_len; i++) {
+               if(passwd->bv_val[i] == '\0') {
+                       return NULL;    /* NUL character in password */
+               }
+       }
 
-       char *hash = NULL;
-       unsigned char salt[2];
+       if( passwd->bv_val[i] != '\0' ) {
+               return NULL;    /* passwd must behave like a string */
+       }
 
        if( lutil_entropy( salt, sizeof(salt)) < 0 ) {
                return NULL; 
@@ -661,11 +1032,18 @@ static struct berval *gen_crypt(
 
        salt[0] = crypt64[ salt[0] % (sizeof(crypt64)-1) ];
        salt[1] = crypt64[ salt[1] % (sizeof(crypt64)-1) ];
+       salt[2] = '\0';
+
+       hash.bv_val = crypt( passwd->bv_val, salt );
 
-       hash = crypt( passwd, salt );
+       if( hash.bv_val == NULL ) return NULL;
 
-       if( hash = NULL ) return NULL;
+       hash.bv_len = strlen( hash.bv_val );
+
+       if( hash.bv_len == 0 ) {
+               return NULL;
+       }
 
-       return pw_string( scheme, hash );
+       return pw_string( scheme, &hash );
 }
 #endif