opendev/gerrit
Code Issues Proposed changes

Fix NPE in ChangeUtil

Browse Source 
Revert will run into NPE caused by [1].

[1] Line 250 of ChangeUtil.java
in https://gerrit-review.googlesource.com/#/c/40001

Change-Id: I527f675847f2e7c3c15259e6fbe1ad5b82d39faf
This commit is contained in:
Bruce Zu
2012-12-04 18:15:55 +08:00
parent 5178a7d2b9
commit 5b6797b317
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-server/src/main/java/com/google/gerrit/server/ChangeUtil.java
View File

@@ -247,7 +247,8 @@ public class ChangeUtil {
changeToRevert.getDest()); changeToRevert.getDest());
change.setTopic(changeToRevert.getTopic()); change.setTopic(changeToRevert.getTopic());
PatchSet.Id id = nextPatchSetId(git, change.currentPatchSetId()); PatchSet.Id id =
new PatchSet.Id(change.getId(), Change.INITIAL_PATCH_SET_ID);
final PatchSet ps = new PatchSet(id); final PatchSet ps = new PatchSet(id);
ps.setCreatedOn(change.getCreatedOn()); ps.setCreatedOn(change.getCreatedOn());
ps.setUploader(change.getOwner()); ps.setUploader(change.getOwner());