Bug 367077: Big attachments which should be stored locally are not - Patch by FrÃ...

diff --git a/Bugzilla/Attachment.pm b/Bugzilla/Attachment.pm
index fe1b781c9a3af370c1f5a0c0c4e3757e837b8349..6a798d046e62fd7b8222dda8503426f608214a79 100644 (file)
--- 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