View Issue Details

IDProjectCategoryView StatusLast Update
0000288Main CAcert Websitemiscpublic2013-01-14 10:53
Reporteraanriot Assigned To 
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Fixed in Version2006 
Summary0000288: Bad usage of checkEmail
DescriptionIn includes/account.php and possibly at other places aswell the function checkEmail() is called this way:

  $checkemail = checkEmail($_REQUEST['newemail']);
  if($checkemail != true) [complain about invalid address and exit]

but the only values checkemail() would return are

  return trim(strip_tags($line));
  return true;
  return _("Failed to make a connection to the mail server");

which would normally all be true.



TagsNo tags attached.
Reviewed by
Test Instructions

Activities

duane

2006-08-13 05:40

developer   ~0000364

changed checkEmail function to return "OK", updated to check return value != "OK"

Issue History

Date Modified Username Field Change
2006-08-11 09:30 bluec New Issue
2006-08-13 05:40 duane Status new => solved?
2006-08-13 05:40 duane Fixed in Version => production
2006-08-13 05:40 duane Resolution open => fixed
2006-08-13 05:40 duane Assigned To => duane
2006-08-13 05:40 duane Note Added: 0000364
2007-10-24 06:21 evaldo Reporter bluec => aanriot
2007-10-24 06:21 evaldo Assigned To duane =>
2007-10-24 06:21 evaldo Status solved? => closed
2013-01-14 10:53 Werner Dworak Fixed in Version => 2006