X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=update.cgi;h=13d847b1a984ac6d79803c7092508db4d1da72f8;hb=e9a3d262aa18e6469b9372741210014f51b0c9a7;hp=c42eddce278ea1c97a0552464fca34b5cdf00ea7;hpb=3bce39d94939aaf999189e4aaed805858a996153;p=mirror%2Fuserdir-ldap-cgi.git diff --git a/update.cgi b/update.cgi index c42eddc..13d847b 100755 --- a/update.cgi +++ b/update.cgi @@ -13,6 +13,7 @@ use Digest::HMAC_SHA1 qw(hmac_sha1_hex); use Util; use English; use URI::Escape; +use Crypt::PasswdMD5; use Net::LDAP qw(LDAP_SUCCESS LDAP_PROTOCOL_ERROR); my %config = &Util::ReadConfigFile; @@ -206,7 +207,7 @@ if (!($query->param('doupdate'))) { # Actually update stuff... - my ($newpassword, $newstaddress); + my ($newpassword, $newstaddress, $newwebpassword); # Good god, why would we want to do that here? it breaks password setting # etc, and it doesn't prevent people from setting eveil stuff in ldap @@ -236,6 +237,23 @@ if (!($query->param('doupdate'))) { &Util::UpdateAuthToken($authtoken, $query->param('newpass')); } + if ($query->param('newwebpass') && $query->param('newwebpassvrfy')) { + if ($query->param('newwebpass') ne $query->param('newwebpassvrfy')) { + # passwords don't match... + &Util::HTMLError("The passwords you specified do not match. Please go back and try again."); + } + + my ($r, $msg) = &Util::checkPasswordQuality($query->param('newwebpass'), undef, [@ldapinfo_for_pwcheck]); + if ($r) { + &Util::HTMLError("Password check failed: $msg. Please go back and try again."); + } + + # create a md5 crypted password + $newwebpassword = apache_md5_crypt($query->param('newwebpass'), &Util::CreateCryptSalt()); + + &Util::LDAPUpdate($ldap, $editdn, 'webPassword', $newwebpassword); + } + $newstaddress = $query->param('staddress'); $newstaddress =~ s/\n/\$/m;