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 2013/10/02 01:45:39 UTC
svn commit: r1528272 - in /zookeeper/branches/branch-3.4: CHANGES.txt
src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java
src/java/test/data/invalidsnap/version-2/snapshot.273
src/java/test/org/apache/zookeeper/test/InvalidSnapshotTest.java
Author: phunt
Date: Tue Oct 1 23:45:39 2013
New Revision: 1528272
URL: http://svn.apache.org/r1528272
Log:
ZOOKEEPER-1770. NullPointerException in SnapshotFormatter (Germán Blanco via phunt)
Added:
zookeeper/branches/branch-3.4/src/java/test/data/invalidsnap/version-2/snapshot.273 (with props)
Modified:
zookeeper/branches/branch-3.4/CHANGES.txt
zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java
zookeeper/branches/branch-3.4/src/java/test/org/apache/zookeeper/test/InvalidSnapshotTest.java
Modified: zookeeper/branches/branch-3.4/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/branches/branch-3.4/CHANGES.txt?rev=1528272&r1=1528271&r2=1528272&view=diff
==============================================================================
--- zookeeper/branches/branch-3.4/CHANGES.txt (original)
+++ zookeeper/branches/branch-3.4/CHANGES.txt Tue Oct 1 23:45:39 2013
@@ -114,6 +114,9 @@ BUGFIXES:
ZOOKEEPER-1696. Fail to run zookeeper client on Weblogic application server.
(Jeffrey Zhong via mahadev)
+ ZOOKEEPER-1770. NullPointerException in SnapshotFormatter
+ (Germán Blanco via phunt)
+
IMPROVEMENTS:
ZOOKEEPER-1564. Allow JUnit test build with IBM Java
Modified: zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java
URL: http://svn.apache.org/viewvc/zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java?rev=1528272&r1=1528271&r2=1528272&view=diff
==============================================================================
--- zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java (original)
+++ zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java Tue Oct 1 23:45:39 2013
@@ -87,7 +87,11 @@ public class SnapshotFormatter {
synchronized(n) { // keep findbugs happy
System.out.println(name);
printStat(n.stat);
- System.out.println(" dataLength = " + n.data.length);
+ if (n.data != null) {
+ System.out.println(" dataLength = " + n.data.length);
+ } else {
+ System.out.println(" no data");
+ }
children = n.getChildren();
}
if (children != null) {
Added: zookeeper/branches/branch-3.4/src/java/test/data/invalidsnap/version-2/snapshot.273
URL: http://svn.apache.org/viewvc/zookeeper/branches/branch-3.4/src/java/test/data/invalidsnap/version-2/snapshot.273?rev=1528272&view=auto
==============================================================================
Binary file - no diff available.
Propchange: zookeeper/branches/branch-3.4/src/java/test/data/invalidsnap/version-2/snapshot.273
------------------------------------------------------------------------------
svn:mime-type = application/octet-stream
Modified: zookeeper/branches/branch-3.4/src/java/test/org/apache/zookeeper/test/InvalidSnapshotTest.java
URL: http://svn.apache.org/viewvc/zookeeper/branches/branch-3.4/src/java/test/org/apache/zookeeper/test/InvalidSnapshotTest.java?rev=1528272&r1=1528271&r2=1528272&view=diff
==============================================================================
--- zookeeper/branches/branch-3.4/src/java/test/org/apache/zookeeper/test/InvalidSnapshotTest.java (original)
+++ zookeeper/branches/branch-3.4/src/java/test/org/apache/zookeeper/test/InvalidSnapshotTest.java Tue Oct 1 23:45:39 2013
@@ -72,6 +72,17 @@ public class InvalidSnapshotTest extends
}
/**
+ * Verify the SnapshotFormatter by running it on a known file with one null data.
+ */
+ @Test
+ public void testSnapshotFormatterWithNull() throws Exception {
+ File snapDir = new File(testData, "invalidsnap");
+ File snapfile = new File(new File(snapDir, "version-2"), "snapshot.273");
+ String[] args = {snapfile.getCanonicalFile().toString()};
+ SnapshotFormatter.main(args);
+ }
+
+ /**
* test the snapshot
* @throws Exception an exception could be expected
*/