Skip to content

Commit b786f42

Browse files
authored
Merge branch '4.19' into no-zone
2 parents 33c7722 + 0d5047b commit b786f42

File tree

45 files changed

+926
-706
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

45 files changed

+926
-706
lines changed

api/src/main/java/org/apache/cloudstack/api/command/user/firewall/CreateFirewallRuleCmd.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,8 @@
1919
import java.util.ArrayList;
2020
import java.util.List;
2121

22+
import org.apache.commons.collections.CollectionUtils;
23+
import org.apache.commons.lang3.StringUtils;
2224
import org.apache.log4j.Logger;
2325

2426
import org.apache.cloudstack.acl.RoleType;
@@ -103,14 +105,13 @@ public String getProtocol() {
103105

104106
@Override
105107
public List<String> getSourceCidrList() {
106-
if (cidrlist != null) {
108+
if (CollectionUtils.isNotEmpty(cidrlist) && !(cidrlist.size() == 1 && StringUtils.isBlank(cidrlist.get(0)))) {
107109
return cidrlist;
108110
} else {
109-
List<String> oneCidrList = new ArrayList<String>();
111+
List<String> oneCidrList = new ArrayList<>();
110112
oneCidrList.add(NetUtils.ALL_IP4_CIDRS);
111113
return oneCidrList;
112114
}
113-
114115
}
115116

116117
// ///////////////////////////////////////////////////

0 commit comments

Comments
 (0)