Index | Thread | Search

From:
Alexander Bluhm <bluhm@openbsd.org>
Subject:
Re: ldpd: get rid of inet_aton
To:
tech <tech@openbsd.org>
Date:
Wed, 21 Aug 2024 22:04:01 +0200

Download raw body.

Thread
On Wed, Aug 21, 2024 at 09:59:49PM +0200, Alexander Bluhm wrote:
> On Wed, Aug 21, 2024 at 10:43:11AM +0200, Florian Obser wrote:
> > OK?
> 
> OK bluhm@

Ah, wait.  Should this be if(inet_pton() != 1) ?

> > diff --git parse.y parse.y
> > index 91606fb6c10..e3c15d41508 100644
> > --- parse.y
> > +++ parse.y
> > @@ -203,7 +203,7 @@ string		: string STRING	{
> >  		;
> >
> >  routerid	: STRING {
> > -			if (!inet_aton($1, &$$)) {
> > +			if (!inet_pton(AF_INET, $1, &$$)) {
> >  				yyerror("%s: error parsing router id", $1);
> >  				free($1);
> >  				YYERROR;
> >
> > --
> > In my defence, I have been left unsupervised.