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:33:27 UTC

svn commit: r1734498 - in /zookeeper/trunk: CHANGES.txt src/contrib/zkperl/ZooKeeper.xs

Author: phunt
Date: Fri Mar 11 06:33:27 2016
New Revision: 1734498

URL: http://svn.apache.org/viewvc?rev=1734498&view=rev
Log:
ZOOKEEPER-2133 zkperl: Segmentation fault if getting a node with null value (Botond Hejj via phunt)

Modified:
    zookeeper/trunk/CHANGES.txt
    zookeeper/trunk/src/contrib/zkperl/ZooKeeper.xs

Modified: zookeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/trunk/CHANGES.txt?rev=1734498&r1=1734497&r2=1734498&view=diff
==============================================================================
--- zookeeper/trunk/CHANGES.txt (original)
+++ zookeeper/trunk/CHANGES.txt Fri Mar 11 06:33:27 2016
@@ -260,6 +260,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-1660 Documentation for Dynamic Reconfiguration (Reed Wanderman-Milne via shralex)  
 

Modified: zookeeper/trunk/src/contrib/zkperl/ZooKeeper.xs
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/contrib/zkperl/ZooKeeper.xs?rev=1734498&r1=1734497&r2=1734498&view=diff
==============================================================================
--- zookeeper/trunk/src/contrib/zkperl/ZooKeeper.xs (original)
+++ zookeeper/trunk/src/contrib/zkperl/ZooKeeper.xs Fri Mar 11 06:33:27 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';