[accumulo] branch main updated: Protect against NPE in FateCommand.printTx (#2738)

[mmiller]

This is an automated email from the ASF dual-hosted git repository.

mmiller pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new 3d17dd766d Protect against NPE in FateCommand.printTx (#2738)
3d17dd766d is described below

commit 3d17dd766d769a3a055bfc7d923e081589e92363
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Thu May 26 13:21:23 2022 -0400

    Protect against NPE in FateCommand.printTx (#2738)
    
    Closes #2736
---
 .../main/java/org/apache/accumulo/shell/commands/FateCommand.java    | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/shell/src/main/java/org/apache/accumulo/shell/commands/FateCommand.java 
b/shell/src/main/java/org/apache/accumulo/shell/commands/FateCommand.java
index fcdb70bb0a..b470474c42 100644
--- a/shell/src/main/java/org/apache/accumulo/shell/commands/FateCommand.java
+++ b/shell/src/main/java/org/apache/accumulo/shell/commands/FateCommand.java
@@ -219,9 +219,8 @@ public class FateCommand extends Command {
       ZooReaderWriter zk, ServiceLock.ServiceLockPath tableLocksPath, String[] 
args, CommandLine cl,
       boolean printStatus) throws InterruptedException, KeeperException, 
IOException {
     // Parse transaction ID filters for print display
-    Set<Long> filterTxid = null;
-    if (args.length >= 2) {
-      filterTxid = new HashSet<>(args.length);
+    Set<Long> filterTxid = new HashSet<>();
+    if (args != null && args.length >= 2) {
       for (int i = 1; i < args.length; i++) {
         Long val = parseTxid(args[i]);
         filterTxid.add(val);