Update services/secondary-storage/controller/src/main/java/org/apache/cloudstack/secondarystorage/SecondaryStorageManagerImpl.java

DaanHoogland · web-flow · commit 03c259865a4f · 2025-01-14T10:30:29.000+01:00
diff --git a/services/secondary-storage/controller/src/main/java/org/apache/cloudstack/secondarystorage/SecondaryStorageManagerImpl.java b/services/secondary-storage/controller/src/main/java/org/apache/cloudstack/secondarystorage/SecondaryStorageManagerImpl.java
@@ -407,7 +407,7 @@ private List<String> getAllowedInternalSiteCidrs() {
                     s_logger.warn(String.format("Invalid CIDR %s in %s", cidr, SecStorageAllowedInternalDownloadSites.key()));
                 }
                 allowedCidrs.add(NetUtils.getCleanIp4Cidr(cidr));
-            } else if (NetUtils.isValidIp4(cidr)) {
+            } else if (NetUtils.isValidIp4(cidr) && !cidr.startsWith("0.0.0.0")) {
                 s_logger.warn(String.format("Ip address is not a valid CIDR; %s consider using %s/32", cidr, cidr));
                 allowedCidrs.add(cidr);
             }

Original file line number	Diff line number	Diff line change
`@@ -407,7 +407,7 @@ private List<String> getAllowedInternalSiteCidrs() {`
`407`	`407`	`s_logger.warn(String.format("Invalid CIDR %s in %s", cidr, SecStorageAllowedInternalDownloadSites.key()));`
`408`	`408`	`}`
`409`	`409`	`allowedCidrs.add(NetUtils.getCleanIp4Cidr(cidr));`
`410`		`- } else if (NetUtils.isValidIp4(cidr)) {`
	`410`	`+ } else if (NetUtils.isValidIp4(cidr) && !cidr.startsWith("0.0.0.0")) {`
`411`	`411`	`s_logger.warn(String.format("Ip address is not a valid CIDR; %s consider using %s/32", cidr, cidr));`
`412`	`412`	`allowedCidrs.add(cidr);`
`413`	`413`	`}`