summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Daniel Peykov <peykov@google.com> 2024-05-15 21:39:50 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2024-05-15 21:39:50 +0000
commit88a077b7dedcc7fd38bd01ecbe4af94590d223d5 (patch)
tree61df9d600ca5b504827eb9a75f47f2935986f6c2
parent3a542f996da8e2722f2cb46f3326a19dcde5ec76 (diff)
parent0c3f6981d42435b10493c80a46fcb6f3d5290987 (diff)
Merge "Fix potential NPE when dumping child nodes" into main
-rw-r--r--cmds/uiautomator/library/core-src/com/android/uiautomator/core/AccessibilityNodeInfoDumper.java10
1 files changed, 7 insertions, 3 deletions
diff --git a/cmds/uiautomator/library/core-src/com/android/uiautomator/core/AccessibilityNodeInfoDumper.java b/cmds/uiautomator/library/core-src/com/android/uiautomator/core/AccessibilityNodeInfoDumper.java
index 488292d68620..f726361effd6 100644
--- a/cmds/uiautomator/library/core-src/com/android/uiautomator/core/AccessibilityNodeInfoDumper.java
+++ b/cmds/uiautomator/library/core-src/com/android/uiautomator/core/AccessibilityNodeInfoDumper.java
@@ -292,13 +292,17 @@ public class AccessibilityNodeInfoDumper {
int childCount = node.getChildCount();
for (int x = 0; x < childCount; x++) {
AccessibilityNodeInfo childNode = node.getChild(x);
-
+ if (childNode == null) {
+ continue;
+ }
if (!safeCharSeqToString(childNode.getContentDescription()).isEmpty()
- || !safeCharSeqToString(childNode.getText()).isEmpty())
+ || !safeCharSeqToString(childNode.getText()).isEmpty()) {
return true;
+ }
- if (childNafCheck(childNode))
+ if (childNafCheck(childNode)) {
return true;
+ }
}
return false;
}