commit | d7f9244d5d2244019341157821423516a2968ac2 | [log] [tgz] |
---|---|---|
author | Hector Oswaldo Caballero <hector.caballero@ericsson.com> | Sat Dec 23 10:36:11 2017 -0500 |
committer | Hugo Arès <hugares@gmail.com> | Thu Feb 08 13:42:50 2018 +0000 |
tree | 99094d75289b535e131b0c377cccdc49a046b5bb | |
parent | 62877c12c2ceb18bf3dc9cc9679b21e9f7f426cb [diff] |
Fix NPE if change does not exist If for any reason a change does not exist in the database, a NPE was thrown when trying to get the list of patch sets. Change-Id: Id8eec2e3e03652a0db68b85a8191065b570a5e0e
diff --git a/owners-autoassign/src/main/java/com/vmware/gerrit/owners/common/GitRefListener.java b/owners-autoassign/src/main/java/com/vmware/gerrit/owners/common/GitRefListener.java index 67fdb1e..fa0a19a 100644 --- a/owners-autoassign/src/main/java/com/vmware/gerrit/owners/common/GitRefListener.java +++ b/owners-autoassign/src/main/java/com/vmware/gerrit/owners/common/GitRefListener.java
@@ -94,6 +94,9 @@ // same local thread and thus cannot be closed in this event listener. try { Change change = reviewDb.changes().get(id); + if (change == null) { + return; + } PatchList patchList = getPatchList(event, change); if (patchList != null) { PathOwners owners =