aboutsummaryrefslogtreecommitdiffstats
path: root/Bugzilla/Attachment
diff options
context:
space:
mode:
authorByron Jones <bjones@mozilla.com>2012-05-08 14:10:16 +0800
committerByron Jones <bjones@mozilla.com>2012-05-08 14:10:16 +0800
commit4f608d4d3489935d55c20e15cc2ec7d700887ed4 (patch)
treedb534127e1e9860bfecf9f88543104484cb6005f /Bugzilla/Attachment
parent2b222a696035597e471c03b3cfa858f0d64f65c9 (diff)
downloadbugs-4f608d4d3489935d55c20e15cc2ec7d700887ed4.tar
bugs-4f608d4d3489935d55c20e15cc2ec7d700887ed4.tar.gz
bugs-4f608d4d3489935d55c20e15cc2ec7d700887ed4.tar.bz2
bugs-4f608d4d3489935d55c20e15cc2ec7d700887ed4.tar.xz
bugs-4f608d4d3489935d55c20e15cc2ec7d700887ed4.zip
Bug 744015: Fix 'use of uninitialized value in string' in PatchReader
r=gerv, a=LpSolit
Diffstat (limited to 'Bugzilla/Attachment')
-rw-r--r--Bugzilla/Attachment/PatchReader.pm4
1 files changed, 3 insertions, 1 deletions
diff --git a/Bugzilla/Attachment/PatchReader.pm b/Bugzilla/Attachment/PatchReader.pm
index 3be0555c1..c79b96ed2 100644
--- a/Bugzilla/Attachment/PatchReader.pm
+++ b/Bugzilla/Attachment/PatchReader.pm
@@ -200,7 +200,9 @@ sub warn_if_interdiff_might_fail {
# Verify that the revisions in the files are the same.
foreach my $file (keys %{$old_file_list}) {
- if ($old_file_list->{$file}{old_revision} ne
+ if (exists $old_file_list->{$file}{old_revision}
+ && exists $new_file_list->{$file}{old_revision}
+ && $old_file_list->{$file}{old_revision} ne
$new_file_list->{$file}{old_revision})
{
return 'interdiff2';