KYLIN-2778 avoid NPE in AclService.java
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/c962feb7 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/c962feb7 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/c962feb7 Branch: refs/heads/yaho-cube-planner Commit: c962feb75605c1919b88ce5b435e814efab626de Parents: 5c5ca98 Author: shaofengshi <shaofeng...@apache.org> Authored: Mon Aug 7 22:20:08 2017 +0800 Committer: shaofengshi <shaofeng...@apache.org> Committed: Mon Aug 7 22:20:08 2017 +0800 ---------------------------------------------------------------------- .../src/main/java/org/apache/kylin/rest/service/AclService.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/c962feb7/server-base/src/main/java/org/apache/kylin/rest/service/AclService.java ---------------------------------------------------------------------- diff --git a/server-base/src/main/java/org/apache/kylin/rest/service/AclService.java b/server-base/src/main/java/org/apache/kylin/rest/service/AclService.java index ac5d098..00bda7f 100644 --- a/server-base/src/main/java/org/apache/kylin/rest/service/AclService.java +++ b/server-base/src/main/java/org/apache/kylin/rest/service/AclService.java @@ -151,9 +151,9 @@ public class AclService implements MutableAclService { for (ObjectIdentity oid : oids) { AclRecord record = aclStore.getResource(getQueryKeyById(String.valueOf(oid.getIdentifier())), AclRecord.class, SERIALIZER); - if (record != null) { + if (record != null && record.getOwnerInfo() != null) { SidInfo owner = record.getOwnerInfo(); - Sid ownerSid = (null == owner) ? null : (owner.isPrincipal() ? new PrincipalSid(owner.getSid()) : new GrantedAuthoritySid(owner.getSid())); + Sid ownerSid = owner.isPrincipal() ? new PrincipalSid(owner.getSid()) : new GrantedAuthoritySid(owner.getSid()); boolean entriesInheriting = record.isEntriesInheriting(); Acl parentAcl = null;
