Merge branch '4.19' into 4.20

This commit is contained in:
Daan Hoogland 2025-01-15 17:51:31 +01:00
commit bf1ff10a90
2 changed files with 11 additions and 5 deletions

View File

@ -400,10 +400,13 @@ public class SecondaryStorageManagerImpl extends ManagerBase implements Secondar
} }
String[] cidrs = _allowedInternalSites.split(","); String[] cidrs = _allowedInternalSites.split(",");
for (String cidr : cidrs) { for (String cidr : cidrs) {
if (NetUtils.isValidIp4Cidr(cidr) || NetUtils.isValidIp4(cidr) || !cidr.startsWith("0.0.0.0")) { if (NetUtils.isValidIp4Cidr(cidr) && !cidr.startsWith("0.0.0.0")) {
if (NetUtils.getCleanIp4Cidr(cidr).equals(cidr)) { if (! NetUtils.getCleanIp4Cidr(cidr).equals(cidr)) {
logger.warn(String.format("Invalid CIDR %s in %s", cidr, SecStorageAllowedInternalDownloadSites.key())); logger.warn("Invalid CIDR {} in {}", cidr, SecStorageAllowedInternalDownloadSites.key());
} }
allowedCidrs.add(NetUtils.getCleanIp4Cidr(cidr));
} else if (NetUtils.isValidIp4(cidr) && !cidr.startsWith("0.0.0.0")) {
logger.warn("Ip address is not a valid CIDR; {} consider using {}/32", cidr, cidr);
allowedCidrs.add(cidr); allowedCidrs.add(cidr);
} }
} }

View File

@ -3167,9 +3167,12 @@ public class VirtualMachineMO extends BaseMO {
int deviceCount = 0; int deviceCount = 0;
int ideDeviceUnitNumber = -1; int ideDeviceUnitNumber = -1;
if (devices != null && devices.size() > 0) { if (devices != null) {
for (VirtualDevice device : devices) { for (VirtualDevice device : devices) {
if (device instanceof VirtualDisk && (controllerKey == device.getControllerKey())) { if (device.getControllerKey() == null || device.getControllerKey() != controllerKey) {
continue;
}
if (device instanceof VirtualDisk || device instanceof VirtualCdrom) {
deviceCount++; deviceCount++;
ideDeviceUnitNumber = device.getUnitNumber(); ideDeviceUnitNumber = device.getUnitNumber();
} }