opendev/gerrit
Code Issues Proposed changes

Merge branch 'stable-2.8'

Browse Source 
* stable-2.8:
  ChangeScreen2: Use font-size 8pt on buttons
  ChangeScreen2: Use consistent borders on header buttons
  SideBySide2: Disable blinking cursor
  ChangeScreen2: Fix click inside of messages
  ChangeScreen2: Prevent loading the same revision again
  Use rev-parse to find gitdir when generating commit-msg hook hint
  canReadCommit: move check for READ on refs/* on top
This commit is contained in:
Shawn Pearce 2013-11-21 19:27:29 -08:00
commit 100b09dd43
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-server/src/main/java/com/google/gerrit/server/project/ProjectControl.java
View File

@ -490,6 +490,10 @@ public class ProjectControl {
}
public boolean canReadCommit(RevWalk rw, RevCommit commit) {
if (controlForRef("refs/*").canPerform(Permission.READ)) {
return true;
}
Project.NameKey projName = state.getProject().getNameKey();
try {
Repository repo = repoManager.openRepository(projName);
@ -521,6 +525,6 @@ public class ProjectControl {
commit.name(), projName.get());
log.error(msg, e);
}
return controlForRef("refs/*").canPerform(Permission.READ);
return false;
}
}