On Wed, Sep 26, 2018 at 06:48:07AM -0600, Todd C. Miller wrote:
> One comment inline, otherwise OK millert@

> > @@ -397,6 +397,9 @@ static int
> >  services(int argc, char *argv[])
> >  {
> >     struct servent  *se;
> > +   const char      *err;
> > +   char            *proto;
> > +   int             serv;
> 
> This should be in_port_t, not int.  I would also call the variable
> port instead of serv but the type is what is important.
Missed that one, thanks.

I'll commit it with `in_port_t port' later this evening unless someone
objects.

Reply via email to