[
https://issues.apache.org/jira/browse/ZOOKEEPER-2978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16366324#comment-16366324
]
ASF GitHub Bot commented on ZOOKEEPER-2978:
-------------------------------------------
Github user afine commented on the issue:
https://github.com/apache/zookeeper/pull/461
@achimbab Thanks for the patch.
I'm wondering if you ran into this bug while running ZooKeeper? This code
is pretty old and should never actually be executed (for reasons described
here:
https://github.com/apache/zookeeper/pull/461/files#diff-e08c1cfd3802a0c3156847175ab8e24bR514).
I would be happy to merge in this fix but please rename the JIRA and the
pull request to something more descriptive (as I do not have permissions to
change the name of a Pull Request). Something like "fix potential null pointer
exception when deleting node".
> missing list
> ------------
>
> Key: ZOOKEEPER-2978
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2978
> Project: ZooKeeper
> Issue Type: Bug
> Components: java client
> Affects Versions: 3.4.11
> Reporter: achimbab
> Priority: Trivial
> Fix For: 3.4.12
>
>
> At line 518, 'existWatches.remove(clientPath)' is null because watches for
> clientPath is already removed.
> https://github.com/apache/zookeeper/pull/461/commits/a6044af23ae1096a8c5305633320fa139cf730b2
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
