svn commit: r1734499 - in /zookeeper/branches/branch-3.5: CHANGES.txt src/contrib/zkperl/ZooKeeper.xs

[phunt]

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