diff options
| author | lpsolit%gmail.com <> | 2007-01-17 07:50:02 +0100 |
|---|---|---|
| committer | lpsolit%gmail.com <> | 2007-01-17 07:50:02 +0100 |
| commit | 8a33252e3ac8d46230f3bf63eaef2b717900a878 (patch) | |
| tree | 9c9ce6c680fd5c34f59dfc8705979381df611bc8 | |
| parent | 1942f8188c24fb8b88c1fe87ca42a0e79c02b752 (diff) | |
| download | bugzilla-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.pm | 5 |
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 |