opuppet/puppetlabs-stdlib
4
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #413 from mhaskel/fix

Browse source 
Check for string before copying
This commit is contained in:
Travis Fields 2015-02-19 12:02:15 -08:00
commit 647e9bcbf9
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
lib/puppet/parser/functions/is_domain_name.rb
View file

@ -13,6 +13,9 @@ Returns true if the string passed to this function is a syntactically correct do
"given #{arguments.size} for 1") "given #{arguments.size} for 1")
end end
# Only allow string types
return false unless arguments[0].is_a?(String)
domain = arguments[0].dup domain = arguments[0].dup
# Limits (rfc1035, 3.1) # Limits (rfc1035, 3.1)
@ -20,9 +23,6 @@ Returns true if the string passed to this function is a syntactically correct do
label_min_length=1 label_min_length=1
label_max_length=63 label_max_length=63
# Only allow string types
return false unless domain.is_a?(String)
# Allow ".", it is the top level domain # Allow ".", it is the top level domain
return true if domain == '.' return true if domain == '.'