opendev/gerrit
Code Issues Proposed changes

Fix NPE in CreateAccessChange

Browse Source 
Change-Id: I8d6c9aa813918f07dd485a842db857c314b0ad06
This commit is contained in:
Alice Kober-Sotzek
2018-04-10 11:40:20 +02:00
parent de75aa0d45
commit 37c18c1891
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
java/com/google/gerrit/server/restapi/project/CreateAccessChange.java
View File

@@ -119,6 +119,7 @@ public class CreateAccessChange implements RestModifyView<ProjectResource, Proje
try (MetaDataUpdate md = metaDataUpdateUser.create(rsrc.getNameKey())) {
ProjectConfig config = ProjectConfig.read(md);
ObjectId oldCommit = config.getRevision();
String oldCommitSha1 = oldCommit == null ? null : oldCommit.getName();
setAccess.validateChanges(config, removals, additions);
setAccess.applyChanges(config, removals, additions);
@@ -141,7 +142,7 @@ public class CreateAccessChange implements RestModifyView<ProjectResource, Proje
config.commitToNewRef(
md, new PatchSet.Id(changeId, Change.INITIAL_PATCH_SET_ID).toRefName());
if (commit.name().equals(oldCommit.getName())) {
if (commit.name().equals(oldCommitSha1)) {
throw new BadRequestException("no change");
}