From: Frédéric Buclin <LpSolit@gmail.com>
Date: Mon, 19 Sep 2011 15:35:42 +0000 (+0200)
Subject: Bug 686904: Removing an invalid URL from the See Also field crashes Bugzilla
X-Git-Tag: bugzilla-4.3.1~243
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f6e2d728bc63bc8f92093d5285195abfd7e7d7dd;p=thirdparty%2Fbugzilla.git

Bug 686904: Removing an invalid URL from the See Also field crashes Bugzilla
r=timello a=LpSolit
---

diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index 2b526588ff..b2afe69721 100644
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -2886,7 +2886,8 @@ sub remove_see_also {
     # Since we remove also the url from the referenced bug,
     # we need to notify changes for that bug too.
     $removed_bug_url = $removed_bug_url->[0];
-    if ($removed_bug_url->isa('Bugzilla::BugUrl::Bugzilla::Local')
+    if ($removed_bug_url
+        and $removed_bug_url->isa('Bugzilla::BugUrl::Bugzilla::Local')
         and defined $removed_bug_url->ref_bug_url)
     {
         push @{ $self->{see_also_changes} },
@@ -3400,7 +3401,7 @@ sub reporter {
 sub see_also {
     my ($self) = @_;
     return [] if $self->{'error'};
-    if (!defined $self->{see_also}) {
+    if (!exists $self->{see_also}) {
         my $ids = Bugzilla->dbh->selectcol_arrayref(
             'SELECT id FROM bug_see_also WHERE bug_id = ?',
             undef, $self->id);