From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bernhard Held <berny156@gmx.de>
To: development@lists.ipfire.org
Subject:
 Re: Aw: [PATCH v3 3/4] proxy.cgi: strip newline from subnet parameters
Date: Thu, 15 Feb 2018 07:38:12 +0100
Message-ID: <197846243.26579.1518676693154@communicator.strato.de>
In-Reply-To:
 <trinity-5f1782a4-6f7e-438a-910c-ecdb6a3ade18-1518650060684@3c-app-gmx-bs52>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============3673739817941436744=="
List-Id: <development.lists.ipfire.org>

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

> Bernhard Bitsch <Bernhard.Bitsch(a)gmx.de> hat am 15. Februar 2018 um 00:14=
 geschrieben:
> newlines have to be stripped from each line.
> Therefore the code should read:
>   				close(SUBNETS);
>  			}
>  =20
>   			foreach (@templist)
>   			{
> 			        chomp;
>   				@temp =3D split(/\//);

Hm, I see now that I misinterpreted your last comment regarding this patch.

However, v3 is still correct, because if you chomp a list, each element is ch=
omped.

Regards,
Bernhard

> - Bernhard
>=20
> > Gesendet: Mittwoch, 14. Februar 2018 um 20:35 Uhr
> > Von: "Bernhard Held" <berny156(a)gmx.de>
> > An: development(a)lists.ipfire.org
> > Betreff: [PATCH v3 3/4] proxy.cgi: strip newline from subnet parameters
> >
> > The subnets in @templist end with newlines. Theses have to be removed
> > before printing and comparison with other subnets.
> > ---
> >  html/cgi-bin/proxy.cgi | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >=20
> > diff --git a/html/cgi-bin/proxy.cgi b/html/cgi-bin/proxy.cgi
> > index 6aa14e15a..d565ffbdc 100644
> > --- a/html/cgi-bin/proxy.cgi
> > +++ b/html/cgi-bin/proxy.cgi
> > @@ -3061,6 +3061,8 @@ END
> >  				close(SUBNETS);
> >  			}
> > =20
> > +			chomp(@templist);
> > +
> >  			foreach (@templist)
> >  			{
> >  				@temp =3D split(/\//);
> > @@ -3069,7 +3071,6 @@ END
> >  					($temp[0] ne $netsettings{'BLUE_NETADDRESS'}) && ($temp[1] ne $nets=
ettings{'BLUE_NETMASK'})
> >  					)
> >  				{
> > -					chomp $temp[1];
> >  					print FILE " ||\n     (isInNet(myIpAddress(), \"$temp[0]\", \"$temp=
[1]\"))";
> >  				}
> >  			}
> > --=20
> > 2.16.1
> >=20
> >

--===============3673739817941436744==--