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
----
----
---
