tree ebf79bc6fb59ed9e21c89b3353f974902a62576e
parent bbfe8e5bb36d46ad1204b1d47a394a3896c9c10e
author Nguyen Tuan Khang Phan <nguyen.tuan.khang.phan@ericsson.com> 1593788564 -0400
committer Nguyen Tuan Khang Phan <nguyen.tuan.khang.phan@ericsson.com> 1593892469 +0000

GroupList: handle null pointer when name is null

If PluginConfig.getGroup(GroupReference.extractGroupName(configValue))
[1] is called with configValue without "group" prefix, null will be
passed in to GroupList.byName(name) [2] as name and null pointer will
happen. Changing order of comparison will result in the same
functionality and null pointer avoidance.

[1] https://gerrit.googlesource.com/gerrit/+/refs/heads/master/java/com/google/gerrit/server/config/PluginConfig.java
[2] https://gerrit.googlesource.com/gerrit/+/refs/heads/master/java/com/google/gerrit/server/project/GroupList.java

Change-Id: I0545d7786d71041342651d6bc2ceec96b6fcfd1c