Skip to content

Commit

Permalink
Merge pull request sakaiproject#47 from baholladay/SAK-28988
Browse files Browse the repository at this point in the history
SAK-28988 Delegated Access: Concurrent hashmaps throw NPE when storing null values
  • Loading branch information
Bryan Holladay committed Jan 16, 2015
2 parents bc6a96b + 663dba4 commit 44ebc75
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1806,9 +1806,9 @@ private Map<String, AccessNode> grantAccessToSites(List<String> siteRefs, boolea
}else{
//set default to no access and override it if the user does have access
//this is so we don't have to keep looking up their access for the same site:
deniedAuthToolsMap.put(siteRef, null);
deniedPublicToolsMap.put(siteRef, null);
accessMap.put(siteRef, null);
deniedAuthToolsMap.put(siteRef, new String[0]);
deniedPublicToolsMap.put(siteRef, new String[0]);
accessMap.put(siteRef, new String[0]);
}
}
//Set user Id and hierarchy id
Expand Down

0 comments on commit 44ebc75

Please sign in to comment.