Bug 190197 AnyEntryGroups() is broken in globals.pl; call from enter_bug.cgi breaks...

patch by jon@vmware.com r=justdave a=justdave

diff --git a/globals.pl b/globals.pl
index 5ad624f9a8d3564919cf6f21a40a33a53bba6d66..8162b9d21f2248aad5d7bc67032a9740a3f50f1d 100644 (file)
--- a/globals.pl
+++ b/globals.pl
@@ -474,12 +474,15 @@ sub AnyEntryGroups {
     $query .= " AND product_id = $product_id" if ($product_id);
     $query .= " LIMIT 1";
     SendSQL($query);
-    $::CachedAnyEntryGroups{$product_id} = MoreSQLData();
-    FetchSQLData();
-    PopGlobalSQLState();
-    return $::CachedAnyEntryGroups{$product_id};
+    if (MoreSQLData()) {
+       $::CachedAnyEntryGroups{$product_id} = MoreSQLData();
+       FetchSQLData();
+       PopGlobalSQLState();
+       return $::CachedAnyEntryGroups{$product_id};
+    } else {
+       return undef;
+    }
 }
-
 #
 # This function checks if there are any default groups defined.
 # If so, then groups may have to be changed when bugs move from