GitHub user brettKK opened a pull request: https://github.com/apache/zookeeper/pull/495
ZOOKEEPER-3007:Potential NPE in ReferenceCountedACLCache#deserialize @LJ1043041006 found a potential NPE in ZK ---- callee BinaryInputArchive#startVector will return null: ``` // code placeholder public Index startVector(String tag) throws IOException { int len = readInt(tag); if (len == -1) { return null; } ``` ----- and caller ReferenceCountedACLCache#deserialize call it without null check ``` // code placeholder Index j = ia.startVector("acls"); while (!j.done()) { ACL acl = new ACL(); acl.deserialize(ia, "acl"); } ``` You can merge this pull request into a Git repository by running: $ git pull https://github.com/brettKK/zookeeper master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zookeeper/pull/495.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #495 ---- ---- ---