[bugzilla-gnome-org-upstream/4.4] Bug 1202447: [SECURITY] The email address is not properly validated during registration if longer th
- From: Andrea Veri <av src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [bugzilla-gnome-org-upstream/4.4] Bug 1202447: [SECURITY] The email address is not properly validated during registration if longer th
- Date: Sat, 19 Sep 2015 09:19:20 +0000 (UTC)
commit be1be8cbe100a43c51984c1792151c314c65244f
Author: Byron Jones ‹:glob› <glob mozilla com>
Date: Thu Sep 10 13:27:44 2015 -0400
Bug 1202447: [SECURITY] The email address is not properly validated during registration if longer than
127 characters
r=LpSolit,a=justdave
Bugzilla/Util.pm | 12 +++++++++---
1 files changed, 9 insertions(+), 3 deletions(-)
---
diff --git a/Bugzilla/Util.pm b/Bugzilla/Util.pm
index 4bd10e1..527bae8 100644
--- a/Bugzilla/Util.pm
+++ b/Bugzilla/Util.pm
@@ -681,12 +681,18 @@ sub validate_email_syntax {
# RFC 2822 section 2.1 specifies that email addresses must
# be made of US-ASCII characters only.
# Email::Address::addr_spec doesn't enforce this.
- my $ret = ($addr =~ /$match/ && $email !~ /\P{ASCII}/ && $email =~ /^$addr_spec$/);
- if ($ret) {
+ # We set the max length to 127 to ensure addresses aren't truncated when
+ # inserted into the tokens.eventdata field.
+ if ($addr =~ /$match/
+ && $email !~ /\P{ASCII}/
+ && $email =~ /^$addr_spec$/
+ && length($email) <= 127)
+ {
# We assume these checks to suffice to consider the address untainted.
trick_taint($_[0]);
+ return 1;
}
- return $ret ? 1 : 0;
+ return 0;
}
sub check_email_syntax {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]