Is this a typo in if_tap.c?

Kevin Lo kevlo at FreeBSD.org
Thu Nov 22 14:10:42 UTC 2012


On 2012/11/22 17:39, David Xu wrote:
> When I was trying to create a second tap device, kernel crashed.
> Is this patch correct ?
>
>
> Index: sys/net/if_tap.c
> ===================================================================
> --- sys/net/if_tap.c    (revision 243397)
> +++ sys/net/if_tap.c    (working copy)
> @@ -186,7 +186,7 @@
>      /* Find any existing device, or allocate new unit number. */
>      i = clone_create(&tapclones, &tap_cdevsw, &unit, &dev, 0);
>      if (i) {
> -        dev = make_dev(&tap_cdevsw, 0, UID_ROOT, GID_WHEEL, 0600,
> +        dev = make_dev(&tap_cdevsw, unit, UID_ROOT, GID_WHEEL, 0600,
>              "%s%d", tapname, unit);
>      }
>

The patch looks right to me.

     Kevin


More information about the freebsd-current mailing list