summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorlpsolit%gmail.com <>2007-01-17 07:50:02 +0100
committerlpsolit%gmail.com <>2007-01-17 07:50:02 +0100
commit8a33252e3ac8d46230f3bf63eaef2b717900a878 (patch)
tree9c9ce6c680fd5c34f59dfc8705979381df611bc8
parent1942f8188c24fb8b88c1fe87ca42a0e79c02b752 (diff)
downloadbugzilla-8a33252e3ac8d46230f3bf63eaef2b717900a878.tar.gz
bugzilla-8a33252e3ac8d46230f3bf63eaef2b717900a878.tar.xz
Bug 367077: Big attachments which should be stored locally are not - Patch by Frédéric Buclin <LpSolit@gmail.com> r=wurblzap a=LpSolit
-rw-r--r--Bugzilla/Attachment.pm5
1 files changed, 4 insertions, 1 deletions
diff --git a/Bugzilla/Attachment.pm b/Bugzilla/Attachment.pm
index fe1b781c9..6a798d046 100644
--- a/Bugzilla/Attachment.pm
+++ b/Bugzilla/Attachment.pm
@@ -749,7 +749,10 @@ sub insert_attachment_for_bug {
unless ($cgi->param('ispatch')) {
$class->validate_content_type($throw_error) || return 0;
}
- $data = _validate_data($throw_error, $hr_vars) || return 0;
+ $data = _validate_data($throw_error, $hr_vars);
+ # If the attachment is stored locally, $data eq ''.
+ # If an error is thrown, $data eq '0'.
+ ($data ne '0') || return 0;
$contenttype = $cgi->param('contenttype');
# These are inserted using placeholders so no need to panic