You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@zookeeper.apache.org by "Julian Chandra Sutadi (Jira)" <ji...@apache.org> on 2026/03/24 10:04:00 UTC
[jira] [Updated] (ZOOKEEPER-5032) StatsTrack(byte[]) throws NullPointerException when quota node data is null
[ https://issues.apache.org/jira/browse/ZOOKEEPER-5032?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Julian Chandra Sutadi updated ZOOKEEPER-5032:
---------------------------------------------
Description:
While conducting testing on the ZooKeeper 3.9.5 release, a possible NullPointerException was identified in the StatsTrack(byte[]) constructor. When the byte[] parameter is null, passing it directly to new String(stat, StandardCharsets.UTF_8) causes an NPE. A null byte array can occur when node.data is null for a quota znode.
*Fix*
Delegate to the existing StatsTrack(String) constructor with a null check:
{code:java}
public StatsTrack(byte[] stat) {
this(stat == null ? null : new String(stat, StandardCharsets.UTF_8));
// invokes constructor overload below
}
public StatsTrack(String stat) {
this.stats.clear();
if (stat == null || stat.length() == 0) {
return;
}
...
}{code}
*Testing*
A regression test testConstructorWithNullByteArray is added to StatsTrackTest. StatsTrackTest is also migrated from JUnit 4 to JUnit 5 to align with the rest of the test suite, as it was previously not being executed by the surefire runner
was:
While conducting testing on the ZooKeeper 3.9.5 release, a possible NullPointerException was identified in the StatsTrack(byte[]) constructor. When the byte[] parameter is null, passing it directly to new String(stat, StandardCharsets.UTF_8) causes an NPE. A null byte array can occur when node.data is null for a quota znode.
*Fix*
Delegate to the existing StatsTrack(String) constructor with a null check:
{code:java}
public StatsTrack(byte[] stat) {
this(stat == null ? null : new String(stat, StandardCharsets.UTF_8));
// invokes constructor overload below
}
public StatsTrack(String stat) {
this.stats.clear();
if (stat == null || stat.length() == 0) {
return;
}
...
}{code}
*Testing*
A regression test testConstructorWithNullByteArray is added to StatsTrackTest. StatsTrackTest is also migrated from JUnit 4 to JUnit 5 to align with the rest of the test suite, as it was previously not being executed by the surefire runner
> StatsTrack(byte[]) throws NullPointerException when quota node data is null
> ---------------------------------------------------------------------------
>
> Key: ZOOKEEPER-5032
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-5032
> Project: ZooKeeper
> Issue Type: Bug
> Components: quota
> Affects Versions: 3.9.5
> Reporter: Julian Chandra Sutadi
> Priority: Major
>
> While conducting testing on the ZooKeeper 3.9.5 release, a possible NullPointerException was identified in the StatsTrack(byte[]) constructor. When the byte[] parameter is null, passing it directly to new String(stat, StandardCharsets.UTF_8) causes an NPE. A null byte array can occur when node.data is null for a quota znode.
> *Fix*
> Delegate to the existing StatsTrack(String) constructor with a null check:
> {code:java}
> public StatsTrack(byte[] stat) {
> this(stat == null ? null : new String(stat, StandardCharsets.UTF_8));
> // invokes constructor overload below
> }
> public StatsTrack(String stat) {
> this.stats.clear();
> if (stat == null || stat.length() == 0) {
> return;
> }
> ...
> }{code}
> *Testing*
> A regression test testConstructorWithNullByteArray is added to StatsTrackTest. StatsTrackTest is also migrated from JUnit 4 to JUnit 5 to align with the rest of the test suite, as it was previously not being executed by the surefire runner
--
This message was sent by Atlassian Jira
(v8.20.10#820010)