Bug 744015: Fix 'use of uninitialized value in string' in PatchReader

r=gerv, a=LpSolit

Bug 744015: Fix 'use of uninitialized value in string' in PatchReader
4f608d4d · Byron Jones · 2b222a69 · 4f608d4d
Commit 4f608d4d authored May 08, 2012 by Byron Jones
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

PatchReader.pm Bugzilla/Attachment/PatchReader.pm +3 -1

No files found.
--- 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';