You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by ph...@apache.org on 2016/03/11 07:34:35 UTC
svn commit: r1734499 - in /zookeeper/branches/branch-3.5: CHANGES.txt
src/contrib/zkperl/ZooKeeper.xs
Author: phunt
Date: Fri Mar 11 06:34:35 2016
New Revision: 1734499
URL: http://svn.apache.org/viewvc?rev=1734499&view=rev
Log:
ZOOKEEPER-2133 zkperl: Segmentation fault if getting a node with null value (Botond Hejj via phunt)
Modified:
zookeeper/branches/branch-3.5/CHANGES.txt
zookeeper/branches/branch-3.5/src/contrib/zkperl/ZooKeeper.xs
Modified: zookeeper/branches/branch-3.5/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/branches/branch-3.5/CHANGES.txt?rev=1734499&r1=1734498&r2=1734499&view=diff
==============================================================================
--- zookeeper/branches/branch-3.5/CHANGES.txt (original)
+++ zookeeper/branches/branch-3.5/CHANGES.txt Fri Mar 11 06:34:35 2016
@@ -108,6 +108,9 @@ BUGFIXES:
ZOOKEEPER-2379: recent commit broke findbugs qabot check
(rakeshr via cnauroth)
+ ZOOKEEPER-2133 zkperl: Segmentation fault if getting a node with
+ null value (Botond Hejj via phunt)
+
IMPROVEMENTS:
ZOOKEEPER-2270: Allow MBeanRegistry to be overridden for better unit tests
Modified: zookeeper/branches/branch-3.5/src/contrib/zkperl/ZooKeeper.xs
URL: http://svn.apache.org/viewvc/zookeeper/branches/branch-3.5/src/contrib/zkperl/ZooKeeper.xs?rev=1734499&r1=1734498&r2=1734499&view=diff
==============================================================================
--- zookeeper/branches/branch-3.5/src/contrib/zkperl/ZooKeeper.xs (original)
+++ zookeeper/branches/branch-3.5/src/contrib/zkperl/ZooKeeper.xs Fri Mar 11 06:34:35 2016
@@ -1713,7 +1713,7 @@ zk_get(zkh, path, ...)
old_watch, new_watch);
}
- if (ret == ZOK) {
+ if (ret == ZOK && buf_len != -1) {
ST(0) = sv_newmortal();
#ifdef SV_HAS_TRAILING_NUL
buf[buf_len] = '\0';