From 70b9769bd5b5f097b5644bf16f5a424a2667b40c Mon Sep 17 00:00:00 2001 From: Abhinandan Prateek Date: Tue, 2 Aug 2011 15:22:49 +0530 Subject: [PATCH] putting a check for domain name list --- .../com/cloud/configuration/ConfigurationManagerImpl.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java index 40f640e5ee6..d25cf86f4ba 100755 --- a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java +++ b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java @@ -1187,6 +1187,13 @@ public class ConfigurationManagerImpl implements ConfigurationManager, Configura // add the domain prefix list to details if not null if (domainSuffixList != null){ + for(String dom : domainSuffixList){ + if (!NetUtils.verifyDomainName(dom)) { + throw new InvalidParameterValueException( + "Invalid network domain suffixes. Total length shouldn't exceed 190 chars. Each domain label must be between 1 and 63 characters long, can contain ASCII letters 'a' through 'z', the digits '0' through '9', " + + "and the hyphen ('-'); can't start or end with \"-\""); + } + } newDetails.put("dns.suffixes", StringUtils.join(domainSuffixList, ",")); }