diff options
| author | lpsolit%gmail.com <> | 2005-07-26 23:53:27 +0200 |
|---|---|---|
| committer | lpsolit%gmail.com <> | 2005-07-26 23:53:27 +0200 |
| commit | b8c9e238b76b53eb614bcad37c8e119d0ab84c31 (patch) | |
| tree | 9027c358f35cfe04bde8776e92c76d4e85df2d65 | |
| parent | b9f1149d089594e3135a6b169ff6eef4ef4be52d (diff) | |
| download | bugzilla-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
| -rw-r--r-- | Bugzilla/Auth/Login/WWW/Env.pm | 8 |
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); |