opendev/gerrit
Code Issues Proposed changes

Merge "AddMembers.apply: Prevent NPE when account doesn't exist" into stable-2.9

Browse Source
This commit is contained in:
David Pursehouse
2014-08-25 00:57:15 +00:00
committed by Gerrit Code Review
parent 7f039ce625 4f1f3313a3
commit b0982edc9c
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-server/src/main/java/com/google/gerrit/server/group/AddMembers.java
View File

@@ -163,7 +163,10 @@ public class AddMembers implements RestModifyView<GroupResource, Input> {
case LDAP:
if (accountResolver.find(nameOrEmail) == null) {
// account does not exist, try to create it
return createAccountByLdap(nameOrEmail);
Account a = createAccountByLdap(nameOrEmail);
if (a != null) {
return a;
}
}
break;
default: