Skip to content

Commit e6b66b6

Browse files
committed
fix npe
1 parent 1da67f6 commit e6b66b6

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

polaris-common/polaris-model/src/main/java/com/tencent/polaris/api/utils/ApiTrieUtil.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ public static TrieNode<String> buildConfigTrieNode(String prefix, TrieNode<Strin
127127
}
128128

129129
public static boolean checkConfig(TrieNode<String> root, String config) {
130-
if (root == null || root.isEmptyChildren()) {
130+
if (root == null || root.isEmptyChildren() || StringUtils.isEmpty(config)) {
131131
return false;
132132
}
133133

polaris-common/polaris-model/src/test/java/com/tencent/polaris/api/utils/ApiTrieUtilTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ public void testCheckConfig() {
5252
assertThat(ApiTrieUtil.checkConfig(root1, "provider")).isFalse();
5353
assertThat(ApiTrieUtil.checkConfig(root1, "provider.config")).isTrue();
5454
assertThat(ApiTrieUtil.checkConfig(root1, "provider.config.nameList[1]")).isTrue();
55+
assertThat(ApiTrieUtil.checkConfig(root1, null)).isFalse();
5556

5657
TrieNode<String> root2 = ApiTrieUtil.buildConfigTrieNode("provider.conf");
5758
assertThat(ApiTrieUtil.checkConfig(root2, "provider.conf.test")).isTrue();

0 commit comments

Comments
 (0)