View Issue Details

IDProjectCategoryView StatusLast Update
0015341phplist applicationInterface - Frontendpublic19-04-10 19:18
Reporterdhartford 
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Product Version2.10.10 
Target Version2.10.12Fixed in Version2.10.12 
Summary0015341: security - forgotpassword value not checked/eval'd
Description/lists/admin when entering value to send an email for 'forgot password', the value is not checked.

Fix included in additional info.
Additional Information/lists/admin/index.php, under the if isset($_REQUEST["forgotpassword"])....

#====php 5.1.6 tested fix - filter_var only works on installs with php > 5.2

$parsedforgotpassword = $_REQUEST["forgotpassword"];

$email_regex = '^[a-zA-Z0-9._-]+@[a-zA-Z0-9._-]+\.([a-zA-Z]{2,4})$';

if(!eregi($email_regex, $parsedforgotpassword)){

   logEvent(sprintf('Invalid forgotpassword email entered from %s.', $_SERVER['REMOTE_ADDR']));

   $page="login";

   $msg="invalid email supplied";

}else{

....normal code....

} #end of email validation check
TagsNo tags attached.

Activities

michiel

19-04-10 19:18

manager   ~0050928

we already have the is_email function for that, so used that one