Closed (fixed)
Project:
Siteminder
Version:
6.x-1.0-alpha1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
18 May 2010 at 23:54 UTC
Updated:
13 Aug 2010 at 17:13 UTC
Jump to comment: Most recent file
A white screen of death (WSOD) happens when new users come to the site as a result of the following code:
line:78
// If $account still does not contain a uid property, fall back to denying access.
// Execution should never get here. If it does, it's an uncaught edge-case.
else {
drupal_access_denied;
exit();
}
Since the email header field is not required, a condition can arise where an admin can create and go-live with a configuration that supplies a blank email address. At that point, when new users come to the site, they will receive a white screen because the following line is missing parens:
line:81
drupal_access_denied;
should be:
line:81
drupal_access_denied();
Note: This can also happen if the email header is supplied but the value is blank.
I'm going to pull a copy from cvs & supply this as a patch later on.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | bugfixes.patch | 915 bytes | netw3rker |
Comments
Comment #1
netw3rker commentedas promised, here's the patch for this issue. it also cleans up a foreach() error that is thrown if the siteminder_profile module is not enabled and configured.
-Chris
Comment #2
Ian Ward commentedChris, looks good. One thing related to the naming of the patch in the future - check out "Name your patch" on http://drupal.org/patch/submit
Comment #3
netw3rker commentedcommited to 6.x-1.0-dev