There is no reason to remove this in arp.

As for ndp/ipv6, i'm not sure.
Is there anyone adding large numbers of ndp entries? Why?

/Benno


Dimitris Papastamos(s...@2f30.org) on 2015.07.25 21:11:41 +0100:
> On Sat, Jul 25, 2015 at 09:20:18PM +0200, Martin Pieuchot wrote:
> > On 13/07/15(Mon) 14:04, Dimitris Papastamos wrote:
> > > Hi,
> > > 
> > > I noticed -f in ndp(8) did nothing at all so I've enabled it and
> > > documented the file syntax in the man page.
> > 
> > If it does nothing, I'd say let's kill it.
> > 
> 
> Index: ndp.8
> ===================================================================
> RCS file: /cvs/src/usr.sbin/ndp/ndp.8,v
> retrieving revision 1.33
> diff -u -p -r1.33 ndp.8
> --- ndp.8     3 Sep 2014 10:39:41 -0000       1.33
> +++ ndp.8     25 Jul 2015 20:10:08 -0000
> @@ -44,7 +44,6 @@
>  .Op Fl H | P | R
>  .Op Fl A Ar wait
>  .Op Fl d Ar hostname
> -.Op Fl f Ar filename
>  .Op Fl i Ar interface Op Ar flag ...
>  .Op Fl s Ar nodename etheraddr Oo Ic temp Oc Op Ic proxy
>  .Op Fl V Ar rdomain
> @@ -119,9 +118,6 @@ the node has sent during the current sta
>  Erase all the NDP entries.
>  .It Fl d Ar hostname
>  Delete the specified NDP entry.
> -.It Fl f Ar filename
> -Parse the file specified by
> -.Ar filename .
>  .It Fl H
>  Harmonize consistency between the routing table and the default router
>  list; install the top entry of the list into the kernel routing table.
> Index: ndp.c
> ===================================================================
> RCS file: /cvs/src/usr.sbin/ndp/ndp.c,v
> retrieving revision 1.61
> diff -u -p -r1.61 ndp.c
> --- ndp.c     3 Jun 2015 08:10:53 -0000       1.61
> +++ ndp.c     25 Jul 2015 20:10:08 -0000
> @@ -123,7 +123,6 @@ char ntop_buf[INET6_ADDRSTRLEN];  /* inet
>  char host_buf[NI_MAXHOST];           /* getnameinfo() */
>  char ifix_buf[IFNAMSIZ];             /* if_indextoname() */
>  
> -int file(char *);
>  void getsocket(void);
>  int set(int, char **);
>  void get(char *);
> @@ -294,41 +293,6 @@ main(int argc, char *argv[])
>       exit(0);
>  }
>  
> -/*
> - * Process a file to set standard ndp entries
> - */
> -int
> -file(char *name)
> -{
> -     FILE *fp;
> -     int i, retval;
> -     char line[100], arg[5][50], *args[5];
> -
> -     if ((fp = fopen(name, "r")) == NULL) {
> -             fprintf(stderr, "ndp: cannot open %s\n", name);
> -             exit(1);
> -     }
> -     args[0] = &arg[0][0];
> -     args[1] = &arg[1][0];
> -     args[2] = &arg[2][0];
> -     args[3] = &arg[3][0];
> -     args[4] = &arg[4][0];
> -     retval = 0;
> -     while (fgets(line, sizeof(line), fp) != NULL) {
> -             i = sscanf(line, "%49s %49s %49s %49s %49s",
> -                 arg[0], arg[1], arg[2], arg[3], arg[4]);
> -             if (i < 2) {
> -                     fprintf(stderr, "ndp: bad line: %s\n", line);
> -                     retval = 1;
> -                     continue;
> -             }
> -             if (set(i, args))
> -                     retval = 1;
> -     }
> -     fclose(fp);
> -     return (retval);
> -}
> -
>  void
>  getsocket(void)
>  {
> @@ -792,7 +756,7 @@ usage(void)
>  {
>       printf("usage: ndp [-nrt] [-a | -c | -p] [-H | -P | -R] ");
>       printf("[-A wait] [-d hostname]\n");
> -     printf("\t[-f filename] [-i interface [flag ...]]\n");
> +     printf("\t[-i interface [flag ...]]\n");
>       printf("\t[-s nodename etheraddr [temp] [proxy]] ");
>       printf("[-V rdomain] [hostname]\n");
>       exit(1);
> 

-- 

Reply via email to