diff options
| author | Prudhvi Akhil Alahari <prudhvi@codeaurora.org> | 2021-05-25 20:59:51 +0530 |
|---|---|---|
| committer | Prudhvi Akhil Alahari <prudhvi@codeaurora.org> | 2021-05-25 21:26:24 +0530 |
| commit | 7a8b421f0bc2e78f75b178239f9b001f113a293e (patch) | |
| tree | be446e9c3793d2cf4d51a88d98b2345be1939360 | |
| parent | 4f04fab5e74c2c09f266072320631339affde1e7 (diff) | |
Fix to not throw NPE while accessing draft refs
This bug was introduced by I622dbbb334b5a, where notes variable was
being accessed without checking for null.
Change-Id: I192b0fb8643eb2980998d56e088aa95f16d4b7c2
| -rw-r--r-- | gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java b/gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java index 83e2a7ae03..549f88953e 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java @@ -1153,7 +1153,7 @@ public class ChangeData { draftsByUser = new HashMap<>(); if (notesMigration.readChanges()) { - for (Ref ref : commentsUtil.getDraftRefs(notes.getChangeId())) { + for (Ref ref : commentsUtil.getDraftRefs(notes().getChangeId())) { Account.Id account = Account.Id.fromRefSuffix(ref.getName()); if (account != null // Double-check that any drafts exist for this user after |