summaryrefslogtreecommitdiffstats
path: root/Bugzilla/Auth/Login
diff options
context:
space:
mode:
authorlpsolit%gmail.com <>2005-07-26 23:53:27 +0200
committerlpsolit%gmail.com <>2005-07-26 23:53:27 +0200
commitb8c9e238b76b53eb614bcad37c8e119d0ab84c31 (patch)
tree9027c358f35cfe04bde8776e92c76d4e85df2d65 /Bugzilla/Auth/Login
parentb9f1149d089594e3135a6b169ff6eef4ef4be52d (diff)
downloadbugzilla-b8c9e238b76b53eb614bcad37c8e119d0ab84c31.tar.gz
bugzilla-b8c9e238b76b53eb614bcad37c8e119d0ab84c31.tar.xz
Bug 300403: New Charts errors out, creates new 'add' user, when Env auth method is used - Patch by A. Karl Kornel <karl@kornel.name> r=wurblzap a=justdave
Diffstat (limited to 'Bugzilla/Auth/Login')
-rw-r--r--Bugzilla/Auth/Login/WWW/Env.pm8
1 files changed, 6 insertions, 2 deletions
diff --git a/Bugzilla/Auth/Login/WWW/Env.pm b/Bugzilla/Auth/Login/WWW/Env.pm
index 985393206..39bea28df 100644
--- a/Bugzilla/Auth/Login/WWW/Env.pm
+++ b/Bugzilla/Auth/Login/WWW/Env.pm
@@ -51,8 +51,12 @@ sub login {
for ($env_id, $env_email, $env_realname) { $_ ||= '' };
# make sure the email field contains only a valid email address
my $emailregexp = Param("emailregexp");
- $env_email =~ /($emailregexp)/;
- $env_email = $1;
+ if ($env_email =~ /($emailregexp)/) {
+ $env_email = $1;
+ }
+ else {
+ return undef;
+ }
# untaint the remaining values
trick_taint($env_id);
trick_taint($env_realname);