From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH v2] general-functions.pl: Update to fix bug #12428
Date: Mon, 04 Jan 2021 10:47:04 +0000
Message-ID: <268EC958-D9CE-4828-821F-3116B26C2114@ipfire.org>
In-Reply-To: <20210102125445.3389-1-ahb.ipfire@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0870369282318499640=="
List-Id: <development.lists.ipfire.org>

--===============0870369282318499640==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hello Adolf,

Thank you for submitting this patched.

I merged it, but it probably should have been a small series of patches. The =
changes do not really have a connection and it would have been easier to revi=
ew the changes, or potentially revert one of them if something goes wrong lat=
er.

Just for next time :) But keep up the great work.

-Michael

> On 2 Jan 2021, at 12:54, Adolf Belka <ahb.ipfire(a)gmail.com> wrote:
>=20
> - Patch of general-functions.pl for implementation of fix provided
> 	by Bernhard Bitsch in bug #12428.
> 	Had to be modified as that fix gave a failure for single character hostnam=
es.
> 	Updated version prevents spaces being put into hostnames and works for sin=
gle
> 	character hostnames
> - Updated subroutine validfqdn to apply consistent rules for hostname & dom=
ain name
> 	portions of fqdn
> - Minor updates for consistency across validhostname, validdomainname & val=
idfqdn
> - Patch implemented into testbed system and confirmed working for hostnames=
, domain names
> 	and FQDN's.
>=20
> Signed-off-by: Adolf Belka <ahb.ipfire(a)gmail.com>
> ---
> config/cfgroot/general-functions.pl | 51 ++++++++++++++++-------------
> 1 file changed, 29 insertions(+), 22 deletions(-)
>=20
> diff --git a/config/cfgroot/general-functions.pl b/config/cfgroot/general-f=
unctions.pl
> index 9be1e7708..a6656ccf5 100644
> --- a/config/cfgroot/general-functions.pl
> +++ b/config/cfgroot/general-functions.pl
> @@ -635,12 +635,12 @@ sub validhostname
> 	# Checks a hostname against RFC1035
>         my $hostname =3D $_[0];
>=20
> -	# Each part should be at least two characters in length
> +	# Hostname should be at least one character in length
> 	# but no more than 63 characters
> 	if (length ($hostname) < 1 || length ($hostname) > 63) {
> 		return 0;}
> 	# Only valid characters are a-z, A-Z, 0-9 and -
> -	if ($hostname !~ /^[a-zA-Z0-9-\s]*$/) {
> +	if ($hostname !~ /^[a-zA-Z0-9-]*$/) {
> 		return 0;}
> 	# First character can only be a letter or a digit
> 	if (substr ($hostname, 0, 1) !~ /^[a-zA-Z0-9]*$/) {
> @@ -655,46 +655,53 @@ sub validdomainname
> {
> 	my $part;
>=20
> -	# Checks a domain name against RFC1035
> +	# Checks a domain name against RFC1035 and RFC2181
>         my $domainname =3D $_[0];
> -	my @parts =3D split (/\./, $domainname);	# Split hostname at the '.'
> +	my @parts =3D split (/\./, $domainname);	# Split domain name at the '.'
>=20
> 	foreach $part (@parts) {
> -		# Each part should be no more than 63 characters in length
> +		# Each part should be at least one character in length
> +		# but no more than 63 characters
> 		if (length ($part) < 1 || length ($part) > 63) {
> 			return 0;}
> 		# Only valid characters are a-z, A-Z, 0-9, _ and -
> 		if ($part !~ /^[a-zA-Z0-9_-]*$/) {
> -			return 0;
> -		}
> +			return 0;}
> 	}
> 	return 1;
> }
>=20
> sub validfqdn
> {
> -	my $part;
> -
> -	# Checks a fully qualified domain name against RFC1035
> +	# Checks a fully qualified domain name against RFC1035 and RFC2181
>         my $fqdn =3D $_[0];
> -	my @parts =3D split (/\./, $fqdn);	# Split hostname at the '.'
> +	my @parts =3D split (/\./, $fqdn);	# Split FQDN at the '.'
> 	if (scalar(@parts) < 2) {		# At least two parts should
> 		return 0;}			# exist in a FQDN
> 						# (i.e.hostname.domain)
> -	foreach $part (@parts) {
> +
> +	for (my $index=3D0; $index < scalar(@parts); $index++) {
> 		# Each part should be at least one character in length
> 		# but no more than 63 characters
> -		if (length ($part) < 1 || length ($part) > 63) {
> -			return 0;}
> -		# Only valid characters are a-z, A-Z, 0-9 and -
> -		if ($part !~ /^[a-zA-Z0-9-]*$/) {
> -			return 0;}
> -		# First character can only be a letter or a digit
> -		if (substr ($part, 0, 1) !~ /^[a-zA-Z0-9]*$/) {
> -			return 0;}
> -		# Last character can only be a letter or a digit
> -		if (substr ($part, -1, 1) !~ /^[a-zA-Z0-9]*$/) {
> +		if (length ($parts[$index]) < 1 || length ($parts[$index]) > 63) {
> 			return 0;}
> +		if ($index eq 0) {	=09
> +			# This is the hostname part
> +			# Only valid characters are a-z, A-Z, 0-9 and -
> +			if ($parts[$index] !~ /^[a-zA-Z0-9-]*$/) {
> +				return 0;}
> +			# First character can only be a letter or a digit
> +			if (substr ($parts[$index], 0, 1) !~ /^[a-zA-Z0-9]*$/) {
> +				return 0;}
> +			# Last character can only be a letter or a digit
> +			if (substr ($parts[$index], -1, 1) !~ /^[a-zA-Z0-9]*$/) {
> +				return 0;}
> +		} else{			=09
> +			# This is the domain part
> +			# Only valid characters are a-z, A-Z, 0-9, _ and -
> +			if ($parts[$index] !~ /^[a-zA-Z0-9_-]*$/) {
> +				return 0;}
> +		}
> 	}
> 	return 1;
> }
> --=20
> 2.30.0
>=20


--===============0870369282318499640==--