--- loncom/interface/loncommon.pm	2006/06/22 13:02:22	1.384
+++ loncom/interface/loncommon.pm	2006/06/22 17:34:40	1.385
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # a pile of common routines
 #
-# $Id: loncommon.pm,v 1.384 2006/06/22 13:02:22 raeburn Exp $
+# $Id: loncommon.pm,v 1.385 2006/06/22 17:34:40 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -3315,6 +3315,8 @@ table.LC_mail_list tr.LC_mail_other {
 table.LC_mail_list tr.LC_mail_other:hover {
   background-color: $mail_other_hover;
 }
+
+
 END
 }
 
@@ -3999,8 +4001,11 @@ sub get_secgrprole_info {
                   };
     if ($needroles) {
         $rolehash = {'all' => 'all'};
-        my %dumphash = &Apache::lonnet::dump('nohist_userroles',$cdom,$cnum);
-        foreach my $item (keys(%dumphash)) {
+        my %user_roles = &Apache::lonnet::dump('nohist_userroles',$cdom,$cnum);
+	if (&Apache::lonnet::error(%user_roles)) {
+	    undef(%user_roles);
+	}
+        foreach my $item (keys(%user_roles)) {
             my ($role)=split(/\:/,$item,2);
             if ($role eq 'cr') { next; }
             if ($role =~ /^cr/) {